GCTC-NTGC / TalentCloud

app/Http/ViewComposers/MenuComposer.php

Indentation +2 added lines, -2 removed lines

@@ -209,13 +209,13 @@
                 'settings_link' => route('manager.settings.edit'),
             ];
         } elseif (WhichPortal::isAdminPortal()) {
-             $loginModals = [
+                $loginModals = [
                 'modals' => Lang::get('common/login_modals'),
                 'register_link' => route('register'),
                 'login_link' => backpack_url('login'),
                 'logout_link' => backpack_url('logout'),
                 'settings_link' => route('settings.edit'),
-             ];
+                ];
         } else {
             $loginModals = [
                 'modals' => Lang::get('common/login_modals'),

app/Http/Controllers/Auth/TwoFactorController.php

Indentation +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
             $user->email,
             $secret
         );
-         $profile_url = '';
+            $profile_url = '';
         if (WhichPortal::isApplicantPortal()) {
             $profile_url = route('settings.edit');
         } elseif (WhichPortal::isManagerPortal()) {

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'
     ]);

app/Listeners/LogSuccessfulLogin.php

Spacing +1 added lines, -1 removed lines

@@ -28,6 +28,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.'}');
     }
 }

app/Listeners/LogUserCreated.php

Spacing +1 added lines, -1 removed lines

@@ -26,6 +26,6 @@
      */
     public function handle(UserCreated $event)
     {
-        Log::notice('User created: ' . $event->user);
+        Log::notice('User created: '.$event->user);
     }
 }

app/Listeners/LogUserUpdated.php

Spacing +1 added lines, -1 removed lines

@@ -26,6 +26,6 @@
      */
     public function handle(UserUpdated $event)
     {
-        Log::notice('User updated: ' . $event->user);
+        Log::notice('User updated: '.$event->user);
     }
 }

app/Listeners/ApplicationStatusChanged.php

Spacing +6 added lines, -6 removed lines

@@ -31,24 +31,24 @@
 
         if (Auth::check()) {
             $user = Auth::user();
-            $userText = '{id=' . $user->id . ', email=' . $user->email . '}';
+            $userText = '{id='.$user->id.', email='.$user->email.'}';
         } else {
             $userText = '{null}';
         }
 
         // Log when application is first created
         if ($application->wasRecentlyCreated) {
-            $applicationText = '{id=' . $application->id . ', status=' . $application->application_status->name . '}';
+            $applicationText = '{id='.$application->id.', status='.$application->application_status->name.'}';
 
-            Log::notice('Application created: application ' . $applicationText . ' has been created by user ' . $userText);
+            Log::notice('Application created: application '.$applicationText.' has been created by user '.$userText);
         }
         // Log if application status has been changed
         elseif ($application->application_status_id != $application->getOriginal('application_status_id')) {
             $freshApplication = $application->fresh();
-            $applicationText = '{id=' . $freshApplication->id . '}';
-            $statusText = '{' . $freshApplication->application_status->name . '}';
+            $applicationText = '{id='.$freshApplication->id.'}';
+            $statusText = '{'.$freshApplication->application_status->name.'}';
 
-            Log::notice('Application status changed: application ' . $applicationText . ' has been changed to ' . $statusText . ' by user ' . $userText);
+            Log::notice('Application status changed: application '.$applicationText.' has been changed to '.$statusText.' by user '.$userText);
         }
     }
 }

app/Console/Kernel.php

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
      */
     protected function commands(): void
     {
-        $this->load(__DIR__ . '/Commands');
+        $this->load(__DIR__.'/Commands');
 
         require base_path('routes/console.php');
     }
@@ -58,7 +58,7 @@ 
         // search $needle in process status
         $result = array_filter(
             $process_status,
-            function ($var) use ($needle) {
+            function($var) use ($needle) {
                 return strpos($var, $needle);
             }
         );