GCTC-NTGC / TalentCloud

app/Policies/ApplicationPolicy.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
         $authManager = ($user->isManager() &&
             $jobApplication->job_poster->manager->user->is($user));
 
-        return $authApplicant||$authManager;
+        return $authApplicant || $authManager;
     }
 
     /**

database/seeds/DevSeeder.php

Spacing +2 added lines, -2 removed lines

@@ -60,7 +60,7 @@ 
 
         factory(JobPoster::class, 3)->state('published')->create([
             'manager_id' => $managerUser->manager->id
-        ])->each(function ($job) : void {
+        ])->each(function($job) : void {
             $job->job_applications()->saveMany(factory(JobApplication::class, 5))->create([
                 'job_poster_id' => $job->id
             ]);
@@ -74,7 +74,7 @@ 
         });
         factory(JobPoster::class, 3)->state('closed')->create([
             'manager_id' => $managerUser->manager->id
-        ])->each(function ($job) : void {
+        ])->each(function($job) : void {
             $job->job_applications()->saveMany(factory(JobApplication::class, 5))->create([
                 'job_poster_id' => $job->id
             ]);

routes/backpack/custom.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
     'prefix' => config('backpack.base.route_prefix', 'admin'),
     'middleware' => ['web', config('backpack.base.middleware_key', 'admin')],
     'namespace' => 'App\Http\Controllers\Admin',
-], function () : void {
+], function() : void {
     // Custom admin routes.
     Route::crud('skill', 'SkillCrudController');
     Route::crud('job-poster', 'JobPosterCrudController');

routes/web.php

Indentation +1 added lines, -1 removed lines

@@ -480,7 +480,7 @@
         ->middleware('can:update,jobPoster');
 
 
-     Route::get('jobs/{jobPoster}/criteria', 'Api\CriteriaController@indexByJob')
+        Route::get('jobs/{jobPoster}/criteria', 'Api\CriteriaController@indexByJob')
         ->where('jobPoster', '[0-9]+')
         ->middleware('can:view,jobPoster');
     Route::put('jobs/{jobPoster}/criteria', 'Api\CriteriaController@batchUpdate')

Spacing +15 added lines, -15 removed lines

@@ -18,10 +18,10 @@ 
         'prefix' => LaravelLocalization::setLocale(),
         'middleware' => ['localeSessionRedirect', 'localizationRedirect', 'localeViewPath']
     ],
-    function () : void {
+    function() : void {
         /** ADD ALL LOCALIZED ROUTES INSIDE THIS GROUP **/
 
-        Route::group(['prefix' => 'demo'], function () : void {
+        Route::group(['prefix' => 'demo'], function() : void {
 
             /* Temporary Blog Index */
             Route::view('blog', 'common/blog-index')->middleware('localOnly')->name('blog');
@@ -51,7 +51,7 @@ 
         });
 
 
-        Route::group(['prefix' => config('app.applicant_prefix')], function () : void {
+        Route::group(['prefix' => config('app.applicant_prefix')], function() : void {
 
             /* Home */
             Route::get('/', 'HomepageController@applicant')->name('home');
@@ -64,7 +64,7 @@ 
                 ->name('jobs.show');
 
             /* Require being logged in */
-            Route::middleware(['auth'])->group(function () : void {
+            Route::middleware(['auth'])->group(function() : void {
                 /* Managers */
                 Route::get('managers/{manager}', 'ManagerProfileController@show')
                     ->middleware('can:view,manager')
@@ -72,7 +72,7 @@ 
             });
 
             /* Require being logged in as applicant */
-            Route::middleware(['auth', 'role:applicant'])->group(function () : void {
+            Route::middleware(['auth', 'role:applicant'])->group(function() : void {
 
             // Application permissions are handled within the controller instead of with middleware
                 /* Applications */
@@ -207,9 +207,9 @@ 
 
         Route::group([
             'prefix' => config('app.manager_prefix'),
-        ], function (): void {
+        ], function(): void {
 
-            Route::middleware(['finishManagerRegistration'])->group(function (): void {
+            Route::middleware(['finishManagerRegistration'])->group(function(): void {
 
                 /* Home */
                 Route::get('/', 'HomepageController@manager')->name('manager.home');
@@ -225,7 +225,7 @@ 
                     'ManagerProfileController@faq'
                 )->name('manager.faq.section');
 
-                Route::middleware(['auth', 'role:manager'])->group(function (): void {
+                Route::middleware(['auth', 'role:manager'])->group(function(): void {
 
                     Route::get('profile', 'ManagerProfileController@editAuthenticated')->name('manager.profile');
 
@@ -322,7 +322,7 @@ 
             });
 
             // These routes must be excluded from the finishManagerRegistration middleware to avoid an infinite loop of redirects
-            Route::middleware(['auth', 'role:manager'])->group(function (): void {
+            Route::middleware(['auth', 'role:manager'])->group(function(): void {
                 Route::get('first-visit', 'Auth\FirstVisitController@showFirstVisitManagerForm')
                     ->name('manager.first_visit');
                 Route::post('finish_registration', 'Auth\FirstVisitController@finishManagerRegistration')
@@ -348,7 +348,7 @@ 
         /* AJAX calls =============================================================== */
 
         /* Require being logged in */
-        Route::middleware(['auth'])->group(function () : void {
+        Route::middleware(['auth'])->group(function() : void {
 
             Route::delete('courses/{course}', 'CourseController@destroy')
                 ->middleware('can:delete,course')
@@ -414,7 +414,7 @@ 
                 'prefix' => 'admin',
                 'middleware' => ['auth', 'role:admin']
             ],
-            function (): void {
+            function(): void {
                 /* Edit Job */
                 Route::get('jobs/{jobPoster}/edit', 'JobController@edit')
                     ->where('jobPoster', '[0-9]+')
@@ -435,7 +435,7 @@ 
         'prefix' => 'admin',
         'middleware' => ['auth', 'role:admin']
     ],
-    function (): void {
+    function(): void {
         // This page is non-localized, because the middleware that redirects to localized pages changes POSTs to GETs and messes up the request.
         Route::post('jobs/create/as-manager/{manager}', 'JobController@createAsManager')
             ->middleware('can:create,App\Models\JobPoster')
@@ -449,7 +449,7 @@ 
 
 
 /** API routes - currently using same default http auth, but not localized */
-Route::group(['prefix' => 'api'], function (): void {
+Route::group(['prefix' => 'api'], function(): void {
     // Protected by a gate in the controller, instead of policy middleware
     Route::get('jobs/{jobPoster}/assessment-plan', 'AssessmentPlanController@getForJob');
     // Public, not protected by policy or gate
@@ -494,7 +494,7 @@ 
         ->name('api.jobs.submit');
     Route::resource('jobs', 'Api\JobApiController')->only([
         'show', 'store', 'update'
-    ])->names([ // Specify custom names because default names collied with existing routes.
+    ])->names([// Specify custom names because default names collied with existing routes.
         'show' => 'api.jobs.show',
         'store' => 'api.jobs.store',
         'update' => 'api.jobs.update'
@@ -502,7 +502,7 @@ 
 
     Route::resource('managers', 'Api\ManagerApiController')->only([
         'show', 'update'
-    ])->names([ // Specify custom names because default names collied with existing routes
+    ])->names([// Specify custom names because default names collied with existing routes
         'show' => 'api.managers.show',
         'update' => 'api.managers.update'
     ]);

app/Services/Validation/ApplicationValidator.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
     protected function addNestedValidatorRules($nestedAttribute, $validatorRules, $rules = [])
     {
         // prepend the attribute name of each validator rule with the nested attribute name
-        $newRules = $this->arrayMapKeys(function ($key) use ($nestedAttribute) {
+        $newRules = $this->arrayMapKeys(function($key) use ($nestedAttribute) {
                 return implode('.', [$nestedAttribute, $key]);
         },
             $validatorRules);

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 +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
             }
         }
 
-        $isUnsaved = function ($criteria, $savedIds): bool {
+        $isUnsaved = function($criteria, $savedIds): bool {
             return !array_key_exists('id', $criteria) || !in_array($criteria['id'], $savedIds);
         };
 

app/Models/JobPoster.php

Spacing +2 added lines, -2 removed lines

@@ -366,7 +366,7 @@ 
      */
     public function submitted_applications() // phpcs:ignore
     {
-        return $this->hasMany(\App\Models\JobApplication::class)->whereDoesntHave('application_status', function ($query): void {
+        return $this->hasMany(\App\Models\JobApplication::class)->whereDoesntHave('application_status', function($query): void {
             $query->where('name', 'draft');
         });
     }
@@ -555,7 +555,7 @@ 
     public function getClassificationMessageAttribute()
     {
         if ($this->classification_id !== null && $this->classification_level !== null) {
-            return $this->classification->key . '-0' . $this->classification_level;
+            return $this->classification->key.'-0'.$this->classification_level;
         }
         return null;
     }

app/Models/Manager.php

Spacing +1 added lines, -1 removed lines

@@ -187,7 +187,7 @@
     public function getFullNameAttribute(): string
     {
         if ($this->user !== null) {
-            return $this->user->first_name . ' ' . $this->user->last_name;
+            return $this->user->first_name.' '.$this->user->last_name;
         }
         return '';
     }

app/Services/Validation/Rules/LinkedInUrlRule.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      */
     public function passes($attribute, $value)
     {
-        return preg_match('/' . self::PATTERN . '/', $value);
+        return preg_match('/'.self::PATTERN.'/', $value);
     }
 
     /**