Passed
Push — dev5 ( 2deb8a...4b89af )
by Ron
06:07
created
app/Http/Controllers/Admin/UserController.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -31,8 +31,8 @@  discard block
 block discarded – undo
31 31
     public function __construct()
32 32
     {
33 33
         $this->middleware('auth')->except('initializeUser', 'submitInitializeUser');
34
-        $this->middleware(function ($request, $next) {
35
-            $this->user = auth()->user();  //  TODO - is this correct????
34
+        $this->middleware(function($request, $next) {
35
+            $this->user = auth()->user(); //  TODO - is this correct????
36 36
             $this->authorize('hasAccess', 'Manage Users');
37 37
             return $next($request);
38 38
         });
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
     {
102 102
         //  Validate the new user form
103 103
         $request->validate([
104
-            'role'       => 'required|numeric',  //  TODO - add validation rule - is in user roles table
104
+            'role'       => 'required|numeric', //  TODO - add validation rule - is in user roles table
105 105
             'username'   => 'required|unique:users|regex:/^[a-zA-Z0-9_]*$/',
106 106
             'first_name' => 'required',
107 107
             'last_name'  => 'required',
@@ -172,7 +172,7 @@  discard block
 block discarded – undo
172 172
                 {
173 173
                     $query->latest()->limit(1);
174 174
                 }])->get()->makeVisible('user_id'));
175
-                $route    = 'admin.user.edit';
175
+                $route = 'admin.user.edit';
176 176
                 break;
177 177
             default:
178 178
                 abort(404);
@@ -198,10 +198,10 @@  discard block
 block discarded – undo
198 198
         $user  = new UserResource(User::find($id));
199 199
 
200 200
         $roleArr = [];
201
-        foreach ($roles as $role) {
202
-            if ($role->role_id == 1 && Auth::user()->role_id != 1) {
201
+        foreach($roles as $role) {
202
+            if($role->role_id == 1 && Auth::user()->role_id != 1) {
203 203
                 continue;
204
-            } else if ($role->role_id == 2 && Auth::user()->role_id > 1) {
204
+            } else if($role->role_id == 2 && Auth::user()->role_id > 1) {
205 205
                 continue;
206 206
             } else {
207 207
                 // $roleArr[$role->role_id] = $role->name;
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
             }
213 213
         }
214 214
 
215
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by User ID-' . Auth::user()->user_id);
215
+        Log::debug('Route '.Route::currentRouteName().' visited by User ID-'.Auth::user()->user_id);
216 216
         return view('admin.userEdit', [
217 217
             'roles' => $roleArr,
218 218
             'user'  => $user->makeVisible(['user_id', 'username']),
Please login to merge, or discard this patch.