GCTC-NTGC / TalentCloud

database/factories/JobPosterQuestionFactory.php

Spacing +2 added lines, -2 removed lines

@@ -5,9 +5,9 @@
 
 $faker_fr = Faker\Factory::create('fr');
 
-$factory->define(JobPosterQuestion::class, function (Faker\Generator $faker) use ($faker_fr) {
+$factory->define(JobPosterQuestion::class, function(Faker\Generator $faker) use ($faker_fr) {
     return [
-        'job_poster_id' => function () {
+        'job_poster_id' => function() {
             return factory(JobPoster::class)->create()->id;
         },
         'description:en' => $faker->sentence(),

database/factories/RatingGuideAnswerFactory.php

Spacing +2 added lines, -2 removed lines

@@ -5,9 +5,9 @@
 use App\Models\Skill;
 use App\Models\Criteria;
 
-$factory->define(RatingGuideAnswer::class, function (Faker\Generator $faker) {
+$factory->define(RatingGuideAnswer::class, function(Faker\Generator $faker) {
     return [
-        'rating_guide_question_id' => function () {
+        'rating_guide_question_id' => function() {
             return factory(RatingGuideQuestion::class)->create()->id;
         },
         'criterion_id' => null,

database/factories/TeamCultureFactory.php

Spacing +2 added lines, -2 removed lines

@@ -5,7 +5,7 @@ 
 use App\Models\TeamCulture;
 use App\Models\Manager;
 
-$factory->define(TeamCulture::class, function (Faker $faker) {
+$factory->define(TeamCulture::class, function(Faker $faker) {
     $faker_fr = FakerFactory::create('fr');
     return [
         'team_size' => $faker->numberBetween(5, 15),
@@ -18,7 +18,7 @@ 
         'operating_context:fr' => $faker_fr->paragraph(),
         'what_we_value:fr' => $faker_fr->paragraph(),
         'how_we_work:fr' => $faker_fr->paragraph(),
-        'manager_id' => function () {
+        'manager_id' => function() {
             return factory(Manager::class)->create()->id;
         }
     ];

app/Services/WhichPortal.php

Spacing +2 added lines, -2 removed lines

@@ -56,9 +56,9 @@
         if (WhichPortal::isApplicantPortal()) {
             return $routeName;
         } elseif (WhichPortal::isManagerPortal()) {
-            return 'manager.' . $routeName;
+            return 'manager.'.$routeName;
         } elseif (WhichPortal::isAdminPortal()) {
-            return 'admin.' . $routeName;
+            return 'admin.'.$routeName;
         }
         return $routeName;
     }

app/Traits/RememberDeviceTrait.php

Spacing +3 added lines, -3 removed lines

@@ -20,8 +20,8 @@ 
      */
     public function getRememberDeviceToken()
     {
-        if (! empty($this->getRememberDeviceTokenName())) {
-            return (string) $this->{$this->getRememberDeviceTokenName()};
+        if (!empty($this->getRememberDeviceTokenName())) {
+            return (string)$this->{$this->getRememberDeviceTokenName()};
         }
     }
 
@@ -33,7 +33,7 @@ 
      */
     public function setRememberDeviceToken($value)
     {
-        if (! empty($this->getRememberDeviceTokenName())) {
+        if (!empty($this->getRememberDeviceTokenName())) {
             // Make sure new value is immediately saved to db, as well as to model object.
             $this->where('id', $this->id)->update([$this->getRememberDeviceTokenName() => $value]);
             $this->{$this->getRememberDeviceTokenName()} = $value;

app/Http/Controllers/Auth/RecoveryCodeController.php

Spacing +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
         if ($valid_codes->contains($recovery_code)) {
             $authenticator = app(Authenticator::class)->boot($request);
             $authenticator->login();
-            $still_valid_codes = $valid_codes->filter(function ($value) use ($recovery_code) {
+            $still_valid_codes = $valid_codes->filter(function($value) use ($recovery_code) {
                 return $value != $recovery_code;
             });
             $user->recovery_codes = $still_valid_codes->toArray();

routes/web.php

Spacing +18 added lines, -18 removed lines

@@ -19,10 +19,10 @@ 
         'prefix' => LaravelLocalization::setLocale(),
         'middleware' => ['localeSessionRedirect', 'localizationRedirect', 'localeViewPath']
     ],
-    function (): void {
+    function(): void {
         /* Routes used for local demos */
         /* If creating public demos, make sure to add a meta robots noindex, nofollow tag */
-        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,13 +51,13 @@ 
             Route::view('builder-08', 'manager/builder-08')->middleware('localOnly')->name('jpb8');
         });
 
-        Route::group(['prefix' => config('app.applicant_prefix')], function (): void {
+        Route::group(['prefix' => config('app.applicant_prefix')], function(): void {
 
             Route::get('two-factor/use-recovery-code', 'Auth\RecoveryCodeController@use')->name('recovery_codes.use');
             Route::post('two-factor/use-recovery-code', 'Auth\RecoveryCodeController@authenticate')->name('recovery_codes.authenticate');
 
             /* Require being logged in */
-            Route::middleware(['auth'])->group(function () : void {
+            Route::middleware(['auth'])->group(function() : void {
 
                 /* Managers */
                 Route::get('managers/{manager}', 'ManagerProfileController@show')
@@ -69,7 +69,7 @@ 
              * IF user is logged in AND has activated 2fa, require one-time password.
              * This should include all routes except those related to authentication, to avoid loops.
              */
-            Route::middleware(['2fa'])->group(function (): void {
+            Route::middleware(['2fa'])->group(function(): void {
                 Route::post('/2fa', 'Auth\TwoFactorController@redirectToExpected')->name('2fa');
 
                 /* Home */
@@ -83,7 +83,7 @@ 
                     ->name('jobs.show');
 
                 /* 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 */
@@ -253,9 +253,9 @@ 
         });
 
         /* Manager Portal =========================================================== */
-        Route::group(['prefix' => config('app.manager_prefix')], function (): void {
+        Route::group(['prefix' => config('app.manager_prefix')], function(): void {
 
-            Route::middleware(['finishManagerRegistration'])->group(function (): void {
+            Route::middleware(['finishManagerRegistration'])->group(function(): void {
 
                 Route::get('two-factor/use-recovery-code', 'Auth\RecoveryCodeController@use')->name('manager.recovery_codes.use');
                 Route::post('two-factor/use-recovery-code', 'Auth\RecoveryCodeController@authenticate')->name('manager.recovery_codes.authenticate');
@@ -264,7 +264,7 @@ 
                  * IF user is logged in AND has activated 2fa, require one-time password.
                  * This should include all routes except those related to authentication, to avoid loops.
                  */
-                Route::middleware(['2fa'])->group(function (): void {
+                Route::middleware(['2fa'])->group(function(): void {
 
                     Route::post('/2fa', 'Auth\TwoFactorController@redirectToExpected')->name('manager.2fa');
 
@@ -282,7 +282,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');
 
@@ -297,7 +297,7 @@ 
                             ->name('manager.profile.update');
 
                         /* Reviewing applications/applicants requires two-factor authentication */
-                        Route::middleware(['2fa.required'])->group(function (): void {
+                        Route::middleware(['2fa.required'])->group(function(): void {
                             Route::get('jobs/{jobPoster}/applications', 'ApplicationByJobController@index')
                                 ->where('jobPoster', '[0-9]+')
                                 ->middleware('can:reviewApplicationsFor,jobPoster')
@@ -430,7 +430,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')
@@ -456,7 +456,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')
@@ -521,7 +521,7 @@ 
                 'prefix' => 'admin',
                 'middleware' => ['auth', 'role:admin']
             ],
-            function (): void {
+            function(): void {
                 /* Edit Job */
                 Route::get('jobs/{jobPoster}/edit', 'JobController@edit')
                     ->where('jobPoster', '[0-9]+')
@@ -542,7 +542,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')
@@ -564,7 +564,7 @@ 
 /* ALL NON-LOCALIZED ROUTES **/
 
 /* 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.
@@ -607,7 +607,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'
@@ -615,7 +615,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'
     ]);