scci / security-employee-tracker

app/Http/ViewComposers/ActionItemsComposer.php

Doc Comments +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
 
 
     /**
-     * @param $calculatedDays
+     * @param integer $calculatedDays
      * @param $user
      * @param $builtUser
      */

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/Mail/SendNewsEmail.php

Doc Comments +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
     /**
      * Create a new message instance.
      *
-     * @return void
+     * @return Mailable
      */
     public function __construct(News $news)
     {
@@ -26,7 +26,7 @@ 
     /**
      * Build the message.
      *
-     * @return Illuminate\Mail\Mailable
+     * @return SendNewsEmail
      */
     public function build()
     {

Indentation +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
      */
     public function __construct(News $news)
     {
-       $this->news = $news;
+        $this->news = $news;
     }
 
     /**

app/Providers/AppServiceProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
 
     /**
      * @param $user
-     * @param $ignoreList
+     * @param string[] $ignoreList
      * @param $log
      */
     private function buildLogComment($user, $ignoreList, $log)

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
     public function boot()
     {
         // If the save() function is used on the User model, we will generate a new log note.
-        User::updating(function ($user) {
+        User::updating(function($user) {
             $this->createUserLog($user);
         });
     }
@@ -51,7 +51,7 @@ 
 
             $log['author_id'] = Auth::user() ? Auth::user()->id : 1;
             $log['user_id'] = $user->id;
-            if( !empty($log['comment']) ) {
+            if (!empty($log['comment'])) {
                 Log::create($log);
             }
         }

app/TrainingUser.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
      */
     public function scopeActiveUsers($query)
     {
-        return $query->whereHas('user', function ($q) {
+        return $query->whereHas('user', function($q) {
             $q->where('status', 'active');
         });
     }

app/Listeners/ResolveLdap.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      */
     public function handle()
     {
-        if(config('auth.guards.web.provider') == 'adldap') {
+        if (config('auth.guards.web.provider') == 'adldap') {
             $this->addMissingUsers();
         }
     }

app/Listeners/EmailTraining.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
         Mail::send(
             'emails.training',
             ['user' => $user, 'training' => $training, 'due_date' => $dueDate, 'trainingUser' => $trainingUser],
-            function ($m) use ($user, $training) {
+            function($m) use ($user, $training) {
                 $m->to($user->email, $user->userFullName)->subject($training->name . " was assigned to you.");
 
                 //ATTACH FILES

app/User.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     /**
      * @var array
      */
-    protected $hidden = array('username','password');
+    protected $hidden = array('username', 'password');
 
     public function supervisor()
     {

app/Handlers/Duty/DutyGroups.php

Indentation +5 added lines, -5 removed lines

@@ -115,11 +115,11 @@
     {
         $row = '';
         foreach ($entry['group'] as $user) {
-			if(Gate::allows('view')) {
-				$row .= "<a href='" . url('user', $user->id) . "'>" . $user->userFullName . "</a> & ";
-			} else {
-				$row .= $user->userFullName . " & ";
-			}
+            if(Gate::allows('view')) {
+                $row .= "<a href='" . url('user', $user->id) . "'>" . $user->userFullName . "</a> & ";
+            } else {
+                $row .= $user->userFullName . " & ";
+            }
         }
         $row = rtrim($row, '& ');
         return $row;

Spacing +6 added lines, -6 removed lines

@@ -13,7 +13,7 @@ 
 
     public function __construct(Duty $duty)
     {
-        parent::__construct($duty);;
+        parent::__construct($duty); ;
     }
 
     public function HTMLOutput()
@@ -36,7 +36,7 @@ 
 
     public function emailOutput()
     {
-        $collection = $this->list->map(function ($value, $key) {
+        $collection = $this->list->map(function($value, $key) {
             return [
                 'users' => $value['group'],
                 'date' => $value['date']
@@ -71,7 +71,7 @@ 
         $newList = new Collection();
         $count = $this->list->count();
 
-        for ($i = 0; $i < $count ; $i++)
+        for ($i = 0; $i < $count; $i++)
         {
             $newList->push([
                 'date' => $dates[$i],
@@ -92,9 +92,9 @@ 
             ->orderBy('date', 'ASC')
             ->get();
 
-        foreach($dutySwaps as $swap)
+        foreach ($dutySwaps as $swap)
         {
-            foreach($this->list as $key => $entry)
+            foreach ($this->list as $key => $entry)
             {
                 if ($swap->date == $entry['date']) {
                     $this->list[$key] = [
@@ -115,7 +115,7 @@ 
     {
         $row = '';
         foreach ($entry['group'] as $user) {
-			if(Gate::allows('view')) {
+			if (Gate::allows('view')) {
 				$row .= "<a href='" . url('user', $user->id) . "'>" . $user->userFullName . "</a> & ";
 			} else {
 				$row .= $user->userFullName . " & ";

app/Handlers/Duty/DutyDates.php

Spacing +4 added lines, -4 removed lines

@@ -40,9 +40,9 @@ 
     {
         if ($cycle == 'monthly') {
             return Carbon::now()->startOfMonth();
-        }else if($cycle == 'weekly') {
+        } else if ($cycle == 'weekly') {
             return Carbon::now()->startOfWeek();
-        }else if($cycle == 'daily') {
+        } else if ($cycle == 'daily') {
             return Carbon::now()->startOfDay();
         } else {
             Log::error('Duty has an invalid cycle name. Must use monthly, weekly or daily');
@@ -54,9 +54,9 @@ 
     {
         if ($cycle == 'monthly') {
             return $date->addMonth();
-        }else if($cycle == 'weekly') {
+        } else if ($cycle == 'weekly') {
             return $date->addWeek();
-        }else if($cycle == 'daily') {
+        } else if ($cycle == 'daily') {
             return $date->addDay();
         } else {
             Log::error('Duty has an invalid cycle name. Must use monthly, weekly or daily');

Braces +4 added lines, -4 removed lines

@@ -40,9 +40,9 @@ 
     {
         if ($cycle == 'monthly') {
             return Carbon::now()->startOfMonth();
-        }else if($cycle == 'weekly') {
+        } else if($cycle == 'weekly') {
             return Carbon::now()->startOfWeek();
-        }else if($cycle == 'daily') {
+        } else if($cycle == 'daily') {
             return Carbon::now()->startOfDay();
         } else {
             Log::error('Duty has an invalid cycle name. Must use monthly, weekly or daily');
@@ -54,9 +54,9 @@ 
     {
         if ($cycle == 'monthly') {
             return $date->addMonth();
-        }else if($cycle == 'weekly') {
+        } else if($cycle == 'weekly') {
             return $date->addWeek();
-        }else if($cycle == 'daily') {
+        } else if($cycle == 'daily') {
             return $date->addDay();
         } else {
             Log::error('Duty has an invalid cycle name. Must use monthly, weekly or daily');