GCTC-NTGC / TalentCloud

app/Http/Requests/SkillCrudRequest.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
     public function rules() : array
     {
         return [
-            'name' => 'required|unique_translation:skills,name' . (isset($this->id) ? ",{$this->id}" : ''),
+            'name' => 'required|unique_translation:skills,name'.(isset($this->id) ? ",{$this->id}" : ''),
             'description' => 'required',
             'skill_type_id' => 'exists:skill_types,id'
         ];

app/Http/Controllers/ApplicantProfileController.php

Spacing +2 added lines, -2 removed lines

@@ -132,7 +132,7 @@ 
         $validator->validate($request->all());
 
         foreach ($questions as $question) {
-            $answerName = $this->answerFormInputName . '.' . $question->id;
+            $answerName = $this->answerFormInputName.'.'.$question->id;
             if ($request->has($answerName)) {
                 $answer = ApplicantProfileAnswer::where(
                     [
@@ -142,7 +142,7 @@ 
                 )->first();
                 if ($answer == null) {
                     $answer = new ApplicantProfileAnswer();
-                    $answer->applicant_id =$applicant->id;
+                    $answer->applicant_id = $applicant->id;
                     $answer->applicant_profile_question_id = $question->id;
                 }
                 $answer->answer = $request->input($answerName);

app/Services/Validation/Rules/PolyExistsRule.php

Spacing +2 added lines, -2 removed lines

@@ -41,11 +41,11 @@
             $type = Relation::getMorphedModel($type);
         }
 
-        if (! class_exists($type)) {
+        if (!class_exists($type)) {
             return false;
         }
 
-        return ! empty(resolve($type)->find($value));
+        return !empty(resolve($type)->find($value));
     }
 
     /**

app/Services/Validation/Rules/TwitterHandleRule.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);
     }
 
     /**

app/Services/Validation/ApplicationValidator.php

Spacing +1 added lines, -1 removed lines

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

routes/web.php

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/Providers/AuthServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
      */
     protected function defineGates(): void
     {
-        Gate::define('view-assessment-plan', function ($user, $jobPoster) {
+        Gate::define('view-assessment-plan', function($user, $jobPoster) {
             return $user->isAdmin() ||
                 $user->isManager() && $jobPoster->manager->user_id === $user->id;
         });

app/Providers/AppServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
      */
     public function register()
     {
-        $this->app->singleton(WhichPortal::class, function ($app) {
+        $this->app->singleton(WhichPortal::class, function($app) {
             return new WhichPortal();
         });
     }

app/Http/Controllers/JobController.php

Spacing +2 added lines, -2 removed lines

@@ -126,12 +126,12 @@
         // TODO: replace route('manager.show',manager.id) in templates with link using slug.
         $criteria = [
             'essential' => $jobPoster->criteria->filter(
-                function ($value, $key) {
+                function($value, $key) {
                     return $value->criteria_type->name == 'essential';
                 }
             ),
             'asset' => $jobPoster->criteria->filter(
-                function ($value, $key) {
+                function($value, $key) {
                     return $value->criteria_type->name == 'asset';
                 }
             ),