GCTC-NTGC / TalentCloud

app/Utilities/HandleNullState.php

Indentation +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
     public function getFunctions(): array
     {
         return [
-          new Twig_SimpleFunction('handleNullState', [$this, 'handleNullState']),
+            new Twig_SimpleFunction('handleNullState', [$this, 'handleNullState']),
         ];
     }
 

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
     public function getFunctions(): array
     {
         return [
-          new Twig_SimpleFunction('handleNullState', [$this, 'handleNullState']),
+          new Twig_SimpleFunction ('handleNullState', [$this, 'handleNullState']),
         ];
     }
 

app/Models/Criteria.php

Indentation +2 added lines, -2 removed lines

@@ -28,8 +28,8 @@
  * @property string $level_description The localized description of the skill level (accounts for skill type).
  *
  *  Localized Properties:
-  * @property string $description
-  * @property string $specificity
+ * @property string $description
+ * @property string $specificity
  */
 class Criteria extends BaseModel
 {

Spacing +8 added lines, -8 removed lines

@@ -65,27 +65,27 @@ 
 
     public function criteria_type() //phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\Lookup\CriteriaType::class);
+        return $this->belongsTo (\App\Models\Lookup\CriteriaType::class);
     }
 
     public function job_poster() //phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\JobPoster::class);
+        return $this->belongsTo (\App\Models\JobPoster::class);
     }
 
     public function skill()
     {
-        return $this->belongsTo(\App\Models\Skill::class);
+        return $this->belongsTo (\App\Models\Skill::class);
     }
 
     public function skill_level() //phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\Lookup\SkillLevel::class);
+        return $this->belongsTo (\App\Models\Lookup\SkillLevel::class);
     }
 
     public function criteria_translations() //phpcs:ignore
     {
-        return $this->hasMany(\App\Models\Lookup\CriteriaTypeTranslation::class);
+        return $this->hasMany (\App\Models\Lookup\CriteriaTypeTranslation::class);
     }
 
     /**
@@ -95,7 +95,7 @@ 
      */
     public function assessments() // phpcs:ignore
     {
-        return $this->hasMany(\App\Models\Assessment::class, 'criterion_id');
+        return $this->hasMany (\App\Models\Assessment::class, 'criterion_id');
     }
 
     /**
@@ -107,7 +107,7 @@ 
     {
         $level = $this->skill_level->name;
         $type = $this->skill->skill_type->name;
-        return Lang::get("common/lookup/skill_level.$level.$type.name");
+        return Lang::get ("common/lookup/skill_level.$level.$type.name");
     }
 
     /**
@@ -119,6 +119,6 @@ 
     {
         $level = $this->skill_level->name;
         $type = $this->skill->skill_type->name;
-        return Lang::get("common/lookup/skill_level.$level.$type.description");
+        return Lang::get ("common/lookup/skill_level.$level.$type.description");
     }
 }

app/Models/Lookup/Frequency.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
  * @property \Jenssegers\Date\Date $created_at
  * @property \Jenssegers\Date\Date $updated_at
  */
-class Frequency extends BaseModel {
+class Frequency extends BaseModel{
 
     protected $fillable = [];
 

app/Http/Controllers/Api/CriteriaController.php

Indentation +5 added lines, -5 removed lines

@@ -12,11 +12,11 @@
 class CriteriaController extends Controller
 {
 /**
-     * Converts a Criteria to the shape sent and recieved by the api.
-     *
-     * @param Criteria $model
-     * @return void
-     */
+ * Converts a Criteria to the shape sent and recieved by the api.
+ *
+ * @param Criteria $model
+ * @return void
+ */
     public function toApiArray(Criteria $model)
     {
         return array_merge($model->toArray(), $model->getTranslationsArray());

Spacing +29 added lines, -29 removed lines

@@ -19,7 +19,7 @@ 
      */
     public function toApiArray(Criteria $model)
     {
-        return array_merge($model->toArray(), $model->getTranslationsArray());
+        return array_merge ($model->toArray (), $model->getTranslationsArray ());
     }
 
     /**
@@ -31,8 +31,8 @@ 
     public function indexByJob(JobPoster $jobPoster)
     {
         $toApiArray = array($this, 'toApiArray');
-        $criteriaAray = Criteria::where('job_poster_id', $jobPoster->id)->get()->map($toApiArray);
-        return response()->json($criteriaAray);
+        $criteriaAray = Criteria::where ('job_poster_id', $jobPoster->id)->get ()->map ($toApiArray);
+        return response ()->json ($criteriaAray);
     }
 
     /**
@@ -46,39 +46,39 @@ 
     {
         $toApiArray = array($this, 'toApiArray');
 
-        $newCriteria = collect($request->input()); // TODO: switch to validated
+        $newCriteria = collect ($request->input ()); // TODO: switch to validated
         $oldCriteria = $jobPoster->criteria;
 
         $updatedIds = [];
 
         // First, delete old criteria that weren't resubmitted, and update those that were
         foreach ($oldCriteria as $criteria) {
-            $newData = $newCriteria->firstWhere('id', $criteria['id']);
+            $newData = $newCriteria->firstWhere ('id', $criteria['id']);
             if ($newData) {
                 $updatedIds[] = $criteria->id;
-                $this->updateCriteria($criteria, $newData);
+                $this->updateCriteria ($criteria, $newData);
             } else {
-                $this->deleteCriteria($criteria);
+                $this->deleteCriteria ($criteria);
             }
         }
 
         $isUnsaved = function ($criteria, $savedIds): bool {
-            return !array_key_exists('id', $criteria) || !in_array($criteria['id'], $savedIds);
+            return !array_key_exists ('id', $criteria) || !in_array ($criteria['id'], $savedIds);
         };
 
         // Now, save any new criteria that remain
         foreach ($newCriteria as $criteriaData) {
-            if ($isUnsaved($criteriaData, $updatedIds)) {
-                $criteria = new Criteria();
+            if ($isUnsaved ($criteriaData, $updatedIds)) {
+                $criteria = new Criteria ();
                 $criteria->job_poster_id = $jobPoster->id;
-                $fillableData = collect($criteriaData)->except(['id', 'job_poster_id'])->toArray();
-                $criteria->fill($fillableData);
-                $this->createCriteria($criteria);
+                $fillableData = collect ($criteriaData)->except (['id', 'job_poster_id'])->toArray ();
+                $criteria->fill ($fillableData);
+                $this->createCriteria ($criteria);
             }
         }
 
-        $criteriaAray = Criteria::where('job_poster_id', $jobPoster->id)->get()->map($toApiArray);
-        return response()->json($criteriaAray);
+        $criteriaAray = Criteria::where ('job_poster_id', $jobPoster->id)->get ()->map ($toApiArray);
+        return response ()->json ($criteriaAray);
     }
 
     /**
@@ -89,13 +89,13 @@ 
      */
     protected function createCriteria(Criteria $criteria)
     {
-        $criteria->save();
+        $criteria->save ();
 
-        $notification = $this->makeAssessmentPlanNotification(
+        $notification = $this->makeAssessmentPlanNotification (
             'CREATE',
             $criteria
         );
-        $notification->save();
+        $notification->save ();
 
         return $criteria;
     }
@@ -113,19 +113,19 @@ 
         if ($oldCriteria->skill_level_id != $newData['skill_level_id']
             || $oldCriteria->skill_id != $newData['skill_id']
         ) {
-            $notification = $this->makeAssessmentPlanNotification(
+            $notification = $this->makeAssessmentPlanNotification (
                 'UPDATE',
                 $oldCriteria,
                 $newData['skill_id'],
                 $newData['skill_level_id'],
                 $newData['criteria_type_id']
             );
-            $notification->save();
+            $notification->save ();
         }
         // Get just the data that can be changed
-        $fillableData = collect($newData)->except(['id', 'job_poster_id'])->toArray();
-        $oldCriteria->fill($fillableData);
-        $oldCriteria->save();
+        $fillableData = collect ($newData)->except (['id', 'job_poster_id'])->toArray ();
+        $oldCriteria->fill ($fillableData);
+        $oldCriteria->save ();
     }
 
     /**
@@ -136,15 +136,15 @@ 
      */
     protected function deleteCriteria(Criteria $criteria): void
     {
-        $notification = $notification = $this->makeAssessmentPlanNotification(
+        $notification = $notification = $this->makeAssessmentPlanNotification (
             'DELETE',
             $criteria
         );
-        $notification->save();
+        $notification->save ();
 
         // Delete assessments related to this criteria.
-        Assessment::where('criterion_id', $criteria->id)->delete();
-        $criteria->delete();
+        Assessment::where ('criterion_id', $criteria->id)->delete ();
+        $criteria->delete ();
     }
 
     /**
@@ -163,8 +163,8 @@ 
         $newSkillId = null,
         $newSkillLevelId = null,
         $newCriteriaTypeId = null
-    ) {
-        $notification = new AssessmentPlanNotification();
+    ){
+        $notification = new AssessmentPlanNotification ();
         $notification->job_poster_id = $criteria->job_poster_id;
         $notification->type = $type;
         $notification->criteria_id = $criteria->id;

routes/backpack/custom.php

Spacing +9 added lines, -9 removed lines

@@ -5,16 +5,16 @@
 // --------------------------
 // This route file is loaded automatically by Backpack\Base.
 // Routes you generate using Backpack\Generators will be placed here.
-Route::group([
-    'prefix' => config('backpack.base.route_prefix', 'admin'),
-    'middleware' => ['web', config('backpack.base.middleware_key', 'admin')],
+Route::group ([
+    'prefix' => config ('backpack.base.route_prefix', 'admin'),
+    'middleware' => ['web', config ('backpack.base.middleware_key', 'admin')],
     'namespace' => 'App\Http\Controllers\Admin',
 ], function () : void {
     // Custom admin routes.
-    Route::crud('skill', 'SkillCrudController');
-    Route::crud('job-poster', 'JobPosterCrudController');
-    Route::crud('user', 'UserCrudController');
-    Route::crud('manager', 'ManagerCrudController');
-    Route::crud('department', 'DepartmentCrudController');
-    Route::crud('classification', 'ClassificationCrudController');
+    Route::crud ('skill', 'SkillCrudController');
+    Route::crud ('job-poster', 'JobPosterCrudController');
+    Route::crud ('user', 'UserCrudController');
+    Route::crud ('manager', 'ManagerCrudController');
+    Route::crud ('department', 'DepartmentCrudController');
+    Route::crud ('classification', 'ClassificationCrudController');
 });

app/Notifications/ResetPasswordNotification.php

Spacing +6 added lines, -6 removed lines

@@ -57,11 +57,11 @@
     public function toMail($notifiable)
     {
         return (new MailMessage)
-            ->subject(Lang::get('common/notifications/password_reset.subject'))
-            ->greeting(Lang::get('common/notifications/password_reset.greeting'))
-            ->line(Lang::get('common/notifications/password_reset.line_1'))
-            ->action(Lang::get('common/notifications/password_reset.action'), url(route('password.reset', $this->token, false)))
-            ->line(Lang::get('common/notifications/password_reset.line_2'))
-            ->salutation(Lang::get('common/notifications/password_reset.salutation', ['name' => config('mail.from.name')]));
+            ->subject (Lang::get ('common/notifications/password_reset.subject'))
+            ->greeting (Lang::get ('common/notifications/password_reset.greeting'))
+            ->line (Lang::get ('common/notifications/password_reset.line_1'))
+            ->action (Lang::get ('common/notifications/password_reset.action'), url (route ('password.reset', $this->token, false)))
+            ->line (Lang::get ('common/notifications/password_reset.line_2'))
+            ->salutation (Lang::get ('common/notifications/password_reset.salutation', ['name' => config ('mail.from.name')]));
     }
 }

app/CRUD/TalentCloudCrudTrait.php

Spacing +8 added lines, -8 removed lines

@@ -23,17 +23,17 @@
     {
         // create an instance of the model to be able to get the table name
         $instance = new static();
-        $conn = DB::connection($instance->getConnectionName());
-        $table = Config::get('database.connections.'.Config::get('database.default').'.pr e fix').$instance->getTable();
+        $conn = DB::connection ($instance->getConnectionName ());
+        $table = Config::get ('database.connections.'.Config::get ('database.default').'.pr e fix').$instance->getTable ();
         // MongoDB columns are alway nullable
-        if ($conn->getConfig()['driver'] === 'mongodb') {
+        if ($conn->getConfig ()['driver'] === 'mongodb') {
             return true;
         }
         // register the enum, json, jsonb, and citext column types, because Doctrine doesn't support it
-        $conn->getDoctrineSchemaManager()->getDatabasePlatform()->registerDoctrineTypeMapping('enum', 'string');
-        $conn->getDoctrineSchemaManager()->getDatabasePlatform()->registerDoctrineTypeMapping('json', 'json_array');
-        $conn->getDoctrineSchemaManager()->getDatabasePlatform()->registerDoctrineTypeMapping('jsonb', 'json_array');
-        $conn->getDoctrineSchemaManager()->getDatabasePlatform()->registerDoctrineTypeMapping('citext', 'string');
-        return !$conn->getDoctrineColumn($table, $column_name)->getNotnull();
+        $conn->getDoctrineSchemaManager ()->getDatabasePlatform ()->registerDoctrineTypeMapping ('enum', 'string');
+        $conn->getDoctrineSchemaManager ()->getDatabasePlatform ()->registerDoctrineTypeMapping ('json', 'json_array');
+        $conn->getDoctrineSchemaManager ()->getDatabasePlatform ()->registerDoctrineTypeMapping ('jsonb', 'json_array');
+        $conn->getDoctrineSchemaManager ()->getDatabasePlatform ()->registerDoctrineTypeMapping ('citext', 'string');
+        return !$conn->getDoctrineColumn ($table, $column_name)->getNotnull ();
     }
 }

app/Listeners/LogApplicationRetrieved.php

Spacing +4 added lines, -4 removed lines

@@ -30,18 +30,18 @@
     {
         $application = $event->application;
 
-        if (Auth::check()) {
-            $user = Auth::user();
+        if (Auth::check ()) {
+            $user = Auth::user ();
 
             //Don't bother logging when an applicant views their own application
             if ($application->applicant->user->id != $user->id) {
                 $applicationText = "{id=$application->id}";
                 $userText = "{id=$user->id, email=$user->email}";
-                Log::notice("Application viewed: application $applicationText viewed by user $userText.");
+                Log::notice ("Application viewed: application $applicationText viewed by user $userText.");
             }
         } else {
             $applicationText = "{id=$application->id}";
-            Log::notice("Application retrieved: application $applicationText retrieved with no user logged in.");
+            Log::notice ("Application retrieved: application $applicationText retrieved with no user logged in.");
         }
     }
 }

app/Listeners/LogSuccessfulLogin.php

Spacing +1 added lines, -1 removed lines

@@ -29,6 +29,6 @@
     public function handle(Login $event)
     {
         $user = $event->user;
-        Log::notice("Login by user {id=".$user->id.", email=".$user->email.", role=".$user->user_role->name."}");
+        Log::notice ("Login by user {id=".$user->id.", email=".$user->email.", role=".$user->user_role->name."}");
     }
 }