]> BookStack Code Mirror - bookstack/commitdiff
Cleaned up some unessasary facade usage
authorDan Brown <redacted>
Sat, 5 Sep 2015 11:29:47 +0000 (12:29 +0100)
committerDan Brown <redacted>
Sat, 5 Sep 2015 11:29:47 +0000 (12:29 +0100)
app/Http/Controllers/Controller.php
app/Http/Controllers/UserController.php
app/Http/routes.php
app/Services/SocialAuthService.php

index 2db96ccf7a60fab8cdc49017db4635ba88e57674..80c4c5526f4a5b37a519345efecc37375462413d 100644 (file)
@@ -30,16 +30,16 @@ abstract class Controller extends BaseController
     public function __construct()
     {
         // Get a user instance for the current user
-        $user = Auth::user();
+        $user = auth()->user();
         if (!$user) {
             $user = User::getDefault();
         }
         // Share variables with views
-        view()->share('signedIn', Auth::check());
+        view()->share('signedIn', auth()->check());
         view()->share('currentUser', $user);
         // Share variables with controllers
         $this->currentUser = $user;
-        $this->signedIn = Auth::check();
+        $this->signedIn = auth()->check();
     }
 
     /**
index b04dedf1671125199ec6a7476abe13975b383a85..306641e71de4caa432206b42459ebeee23f70112 100644 (file)
@@ -64,7 +64,7 @@ class UserController extends Controller
         ]);
 
         $user = $this->user->fill($request->all());
-        $user->password = Hash::make($request->get('password'));
+        $user->password = bcrypt($request->get('password'));
         $user->save();
 
         $user->attachRoleId($request->get('role'));
@@ -120,7 +120,7 @@ class UserController extends Controller
         if ($request->has('password') && $request->get('password') != '') {
             //dd('cat');
             $password = $request->get('password');
-            $user->password = Hash::make($password);
+            $user->password = bcrypt($password);
         }
         $user->save();
         return redirect('/users');
index b50cd6fab804079f29db4a12205911c325bed405..97908ff4842aab4a6d2bad9253d149ece65a9e18 100644 (file)
@@ -43,6 +43,7 @@ Route::group(['middleware' => 'auth'], function () {
 
     });
 
+    // Uploads
     Route::post('/upload/image', 'ImageController@upload');
 
     // Users
@@ -75,7 +76,6 @@ Route::group(['middleware' => 'auth'], function () {
     Route::get('/settings', 'SettingController@index');
     Route::post('/settings', 'SettingController@update');
 
-
 });
 
 // Login using social authentication
index 814f96af602dcccec26811e109a08fb20e447b4f..fda39819ddb17cc266a49a99dac5df864360de4e 100644 (file)
@@ -60,8 +60,8 @@ class SocialAuthService
         // Get any attached social accounts or users
         $socialAccount = $this->socialAccount->where('driver_id', '=', $socialId)->first();
         $user = $this->userRepo->getByEmail($socialUser->getEmail());
-        $isLoggedIn = \Auth::check();
-        $currentUser = \Auth::user();
+        $isLoggedIn = auth()->check();
+        $currentUser = auth()->user();
 
         // When a user is not logged in but a matching SocialAccount exists,
         // Log the user found on the SocialAccount into the application.
@@ -115,7 +115,7 @@ class SocialAuthService
 
     private function logUserIn($user)
     {
-        \Auth::login($user);
+        auth()->login($user);
         return redirect('/');
     }
 
@@ -183,9 +183,10 @@ class SocialAuthService
      */
     public function detachSocialAccount($socialDriver)
     {
-        \Auth::user()->socialAccounts()->where('driver', '=', $socialDriver)->delete();
+        session();
+        auth()->user()->socialAccounts()->where('driver', '=', $socialDriver)->delete();
         \Session::flash('success', $socialDriver . ' account successfully detached');
-        return redirect(\Auth::user()->getEditUrl());
+        return redirect(auth()->user()->getEditUrl());
     }
 
 }
\ No newline at end of file