scci / security-employee-tracker

app/Http/Controllers/TrainingUserController.php

Spacing +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
         $trainingUser = TrainingUser::findOrFail($trainingUserID);
 
         //disable the due by field unless admin.
-        if(Gate::denies('edit')) {
+        if (Gate::denies('edit')) {
             $disabled = 'disabled';
         } else {
             $disabled = '';
@@ -89,7 +89,7 @@ 
     public function destroy($userID, $trainingUserID)
     {
         TrainingUser::findOrFail($trainingUserID)->delete();
-        Storage::deleteDirectory('traininguser_'.$trainingUserID);
+        Storage::deleteDirectory('traininguser_' . $trainingUserID);
 
         return Redirect::back();
     }

app/Http/Controllers/Auth/LoginController.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
     {
         $this->middleware('guest', ['except' => 'logout']);
 
-        if(config('auth.guards.web.provider') == 'adldap') {
+        if (config('auth.guards.web.provider') == 'adldap') {
             Adldap::connect();
         }
     }

app/Http/Controllers/UserController.php

Spacing +9 added lines, -9 removed lines

@@ -27,7 +27,7 @@ 
         $this->authorize('view');
 
         $users = User::with([
-            'assignedTrainings' => function ($q) {
+            'assignedTrainings' => function($q) {
                 $q->whereNull('completed_date')
                     ->whereBetween('due_date', [Carbon::now()->subYear(), Carbon::now()->addWeeks(4)]);
             },
@@ -60,7 +60,7 @@ 
         $data['status'] = 'active';
         $user = User::create($data);
 
-        if( array_key_exists('groups', $data)) {
+        if (array_key_exists('groups', $data)) {
             settype($data['groups'], "array");
             $user->groups()->sync($data['groups']);
         }
@@ -74,7 +74,7 @@ 
      */
     public function show($userId)
     {
-        $user = User::with(['subordinates' => function ($query) {
+        $user = User::with(['subordinates' => function($query) {
             $query->active();
         }, 'supervisor', 'groups', 'duties', 'attachments'])
             ->findOrFail($userId);
@@ -91,14 +91,14 @@ 
         $travels = $user->travels()->with('author', 'attachments')->get();
 
         $logs = [];
-        if(Gate::allows('view')) {
+        if (Gate::allows('view')) {
             $logs = $user->logs()->orderBy('created_at', 'desc')->get();
         }
 
         $this->previousAndNextUsers($user, $previous, $next);
 
         //This mess is just so that we can output the Security Check list or show none. Mainly just to show none.
-        $duties = Duty::whereHas('users', function ($q) use ($userId){
+        $duties = Duty::whereHas('users', function($q) use ($userId){
             $q->where('id', $userId);
         })->orWhereHas('groups.users', function($q) use ($userId) {
             $q->where('id', $userId);
@@ -136,14 +136,14 @@ 
         $user->update($data);
 
         //Handle user groups
-        if( !array_key_exists('groups', $data)) {
+        if (!array_key_exists('groups', $data)) {
             $data['groups'] = [];
         }
         $user->groups()->sync($data['groups']);
 
         //Handled closed area access (MUST come AFTER syncing groups).
-        if( array_key_exists('access', $data)) {
-            foreach($data['access'] as $group_id => $accessLevel) {
+        if (array_key_exists('access', $data)) {
+            foreach ($data['access'] as $group_id => $accessLevel) {
                 $user->groups()->updateExistingPivot($group_id, ['access' => $accessLevel]);
             }
         }
@@ -158,7 +158,7 @@ 
      */
     public function destroy($userId)
     {
-        Storage::deleteDirectory('user_'.$userId);
+        Storage::deleteDirectory('user_' . $userId);
         User::findOrFail($userId)->delete();
 
         return "success";

app/Http/Controllers/TravelController.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
     public function destroy($userID, $travelID)
     {
         Travel::findOrFail($travelID)->delete();
-        Storage::deleteDirectory('travel_'.$travelID);
+        Storage::deleteDirectory('travel_' . $travelID);
 
         return Redirect::back();
     }

app/Http/Controllers/DutySwapController.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
         $type = 'SET\\' . $data['type'];
         $dutyID = intval($data['duty']);
 
-        ( new DutyList($dutyID) )->processSwapRequest($dates, $IDs, $type);
+        (new DutyList($dutyID))->processSwapRequest($dates, $IDs, $type);
 
         return redirect()->action('DutyController@show', $dutyID);
 

app/Http/ViewComposers/ActionItemsComposer.php

Spacing +5 added lines, -5 removed lines

@@ -33,13 +33,13 @@ 
     private function getDueTraining()
     {
 
-        return Training::with(['users', 'assignedUsers' => function ($query) {
+        return Training::with(['users', 'assignedUsers' => function($query) {
             //filter the assignedusers we get back
             $query->ActiveUsers()
                 ->whereNull('completed_date')
                 ->where('due_date', '<=', Carbon::now());
             }])
-            ->whereHas('assignedUsers', function ($q) {
+            ->whereHas('assignedUsers', function($q) {
                 //filter the training we get.
                 $q->ActiveUsers()
                     ->whereNull('completed_date')
@@ -53,9 +53,9 @@ 
      */
     private function getExpiringVisits()
     {
-        return User::with(['visits' => function ($query) {
+        return User::with(['visits' => function($query) {
             $query->whereBetween('expiration_date', [Carbon::now(), Carbon::now()->addWeek()]);
-        }])->whereHas('visits', function ($q) {
+        }])->whereHas('visits', function($q) {
             $q->whereBetween('expiration_date', [Carbon::now(), Carbon::now()->addWeek()]);
         })->Active()->get();
     }
@@ -90,7 +90,7 @@ 
             $years = 5;
         } else if ($user->access_level == 'S' || $user->clearance == 'S') {
             $years = 10;
-        } else if($user->clearance = 'TS') {
+        } else if ($user->clearance = 'TS') {
             $years = 5;
         }
 

app/Providers/AuthServiceProvider.php

Spacing +7 added lines, -7 removed lines

@@ -32,34 +32,34 @@
         $this->registerPolicies($gate);
 
         //user can edit this. AKA, they are an admin.
-        $gate->define('edit', function ($user) {
+        $gate->define('edit', function($user) {
             return $this->isAdmin($user);
         });
 
         //user has view rights
-        $gate->define('view', function ($user) {
+        $gate->define('view', function($user) {
             return $this->isViewer($user);
         });
 
         //Let admin update note & user update training note.
-        $gate->define('update_record', function ($user, $record) {
+        $gate->define('update_record', function($user, $record) {
             return ($user->id == $record->user_id || $this->isAdmin($user));
         });
 
         // primarily used to set javascript variable.
-        $gate->define('update_self', function ($user, $page) {
+        $gate->define('update_self', function($user, $page) {
             return $user->id == $page->id && !$this->isAdmin($user);
         });
 
-        $gate->define('show_user', function ($user, $page) {
+        $gate->define('show_user', function($user, $page) {
             return ($this->isViewer($user) || $user->id === $page->id);
         });
 
-        $gate->define('show_note', function ($user, $page) {
+        $gate->define('show_note', function($user, $page) {
             return ($this->isAdmin($user) || Note::findOrFail($page->id)->user()->id === $user->id);
         });
         
-        $gate->define('show_published_news', function ($user, $news) {
+        $gate->define('show_published_news', function($user, $news) {
             return ($this->isViewer($user) || 
                     ($news->publish_date <= Carbon::today() &&
                       ($news->expire_date >= Carbon::today() || 

app/Providers/BroadcastServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
         /*
          * Authenticate the user's personal channel...
          */
-        Broadcast::channel('App.User.*', function ($user, $userId) {
+        Broadcast::channel('App.User.*', function($user, $userId) {
             return (int) $user->id === (int) $userId;
         });
     }

app/Providers/RouteServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
         Route::group([
             'middleware' => 'web',
             'namespace' => $this->namespace,
-        ], function ($router) {
+        ], function($router) {
             require base_path('routes/web.php');
         });
     }
@@ -62,7 +62,7 @@ 
             'middleware' => 'api',
             'namespace' => $this->namespace,
             'prefix' => 'api',
-        ], function ($router) {
+        ], function($router) {
             require base_path('routes/api.php');
         });
     }