GCTC-NTGC / TalentCloud

app/Models/Classification.php

Spacing +2 added lines, -2 removed lines

@@ -27,11 +27,11 @@
 
     public function skills()
     {
-        return $this->belongsToMany(\App\Models\Skill::class)->withTimestamps();
+        return $this->belongsToMany (\App\Models\Skill::class)->withTimestamps ();
     }
 
     public function job_posters() //phpcs:ignore
     {
-        return $this->belongsToMany(\App\Models\JobPoster::class);
+        return $this->belongsToMany (\App\Models\JobPoster::class);
     }
 }

routes/web.php

Spacing +287 added lines, -287 removed lines

@@ -13,467 +13,467 @@ 
 |
  */
 
-Route::group(
+Route::group (
     [
-        'prefix' => LaravelLocalization::setLocale(),
+        'prefix' => LaravelLocalization::setLocale (),
         'middleware' => ['localeSessionRedirect', 'localizationRedirect', 'localeViewPath']
     ],
     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');
+            Route::view ('blog', 'common/blog-index')->middleware ('localOnly')->name ('blog');
 
             /* Temporary Blog Post */
-            Route::view('post', 'common/blog-post')->middleware('localOnly')->name('post');
+            Route::view ('post', 'common/blog-post')->middleware ('localOnly')->name ('post');
 
             /* Static - Reliability Form Demo */
-            Route::view('reliability', 'demos/reliability/index', ['reliability' => Lang::get('common/reliability')])->name('reliability');
+            Route::view ('reliability', 'demos/reliability/index', ['reliability' => Lang::get ('common/reliability')])->name ('reliability');
 
             // /* Temp Builder 01 (Intro) */
-            Route::view('builder-01', 'manager/builder-01')->middleware('localOnly')->name('jpb1');
+            Route::view ('builder-01', 'manager/builder-01')->middleware ('localOnly')->name ('jpb1');
             // /* Temp Builder 02 (Job info) */
-            Route::view('builder-02', 'manager/builder-02')->middleware('localOnly')->name('jpb2');
+            Route::view ('builder-02', 'manager/builder-02')->middleware ('localOnly')->name ('jpb2');
             // /* Temp Builder 03 (Work Environment) */
-            Route::view('builder-03', 'manager/builder-03')->middleware('localOnly')->name('jpb3');
+            Route::view ('builder-03', 'manager/builder-03')->middleware ('localOnly')->name ('jpb3');
             // /* Temp Builder 04 (Impact) */
-            Route::view('builder-04', 'manager/builder-04')->middleware('localOnly')->name('jpb4');
+            Route::view ('builder-04', 'manager/builder-04')->middleware ('localOnly')->name ('jpb4');
             // /* Temp Builder 05 (Tasks) */
-            Route::view('builder-05', 'manager/builder-05')->middleware('localOnly')->name('jpb5');
+            Route::view ('builder-05', 'manager/builder-05')->middleware ('localOnly')->name ('jpb5');
             // /* Temp Builder 06 (Skills) */
-            Route::view('builder-06', 'manager/builder-06')->middleware('localOnly')->name('jpb6');
+            Route::view ('builder-06', 'manager/builder-06')->middleware ('localOnly')->name ('jpb6');
             // /* Temp Builder 07 (Education) */
-            Route::view('builder-07', 'manager/builder-07')->middleware('localOnly')->name('jpb7');
+            Route::view ('builder-07', 'manager/builder-07')->middleware ('localOnly')->name ('jpb7');
             // /* Temp Builder 08 (Review) */
-            Route::view('builder-08', 'manager/builder-08')->middleware('localOnly')->name('jpb8');
+            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 {
 
             /* Home */
-            Route::get('/', 'HomepageController@applicant')->name('home');
+            Route::get ('/', 'HomepageController@applicant')->name ('home');
 
             /* Jobs */
-            Route::get('jobs', 'JobController@index')->name('jobs.index');
+            Route::get ('jobs', 'JobController@index')->name ('jobs.index');
 
-            Route::get('jobs/{jobPoster}', 'JobController@show')
-                ->middleware('can:view,jobPoster')
-                ->name('jobs.show');
+            Route::get ('jobs/{jobPoster}', 'JobController@show')
+                ->middleware ('can:view,jobPoster')
+                ->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')
-                    ->name('managers.show');
+                Route::get ('managers/{manager}', 'ManagerProfileController@show')
+                    ->middleware ('can:view,manager')
+                    ->name ('managers.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 */
-                Route::get('applications', 'ApplicationController@index')->name('applications.index');
+                Route::get ('applications', 'ApplicationController@index')->name ('applications.index');
 
                 /* View Application */
-                Route::get('applications/{application}', 'ApplicationController@show')
-                    ->middleware('can:view,application')
-                    ->name('applications.show');
+                Route::get ('applications/{application}', 'ApplicationController@show')
+                    ->middleware ('can:view,application')
+                    ->name ('applications.show');
 
                 /* Step 01 */
-                Route::get('jobs/{jobPoster}/application/step-01', 'ApplicationByJobController@editBasics')->name('job.application.edit.1');
+                Route::get ('jobs/{jobPoster}/application/step-01', 'ApplicationByJobController@editBasics')->name ('job.application.edit.1');
 
                 /* Step 02 */
-                Route::get('jobs/{jobPoster}/application/step-02', 'ApplicationByJobController@editExperience')->name('job.application.edit.2');
+                Route::get ('jobs/{jobPoster}/application/step-02', 'ApplicationByJobController@editExperience')->name ('job.application.edit.2');
 
                 /* Step 03 */
-                Route::get('jobs/{jobPoster}/application/step-03', 'ApplicationByJobController@editEssentialSkills')->name('job.application.edit.3');
+                Route::get ('jobs/{jobPoster}/application/step-03', 'ApplicationByJobController@editEssentialSkills')->name ('job.application.edit.3');
 
                 /* Step 04 */
-                Route::get('jobs/{jobPoster}/application/step-04', 'ApplicationByJobController@editAssetSkills')->name('job.application.edit.4');
+                Route::get ('jobs/{jobPoster}/application/step-04', 'ApplicationByJobController@editAssetSkills')->name ('job.application.edit.4');
 
                 /* Step 05 */
-                Route::get('jobs/{jobPoster}/application/step-05', 'ApplicationByJobController@preview')->name('job.application.edit.5');
+                Route::get ('jobs/{jobPoster}/application/step-05', 'ApplicationByJobController@preview')->name ('job.application.edit.5');
 
                 /* Step 06 */
-                Route::get('jobs/{jobPoster}/application/step-06', 'ApplicationByJobController@confirm')->name('job.application.edit.6');
+                Route::get ('jobs/{jobPoster}/application/step-06', 'ApplicationByJobController@confirm')->name ('job.application.edit.6');
 
                 /* Step 06: Complete */
-                Route::get('jobs/{jobPoster}/application/complete', 'ApplicationByJobController@complete')->name('job.application.complete');
+                Route::get ('jobs/{jobPoster}/application/complete', 'ApplicationByJobController@complete')->name ('job.application.complete');
 
                 /* Application Update routes */
 
                 /* Step 01 */
-                Route::post('jobs/{jobPoster}/application/step-01/update', 'ApplicationByJobController@updateBasics')->name('job.application.update.1');
+                Route::post ('jobs/{jobPoster}/application/step-01/update', 'ApplicationByJobController@updateBasics')->name ('job.application.update.1');
 
                 /* Step 02 */
-                Route::post('jobs/{jobPoster}/application/step-02/update', 'ApplicationByJobController@updateExperience')->name('job.application.update.2');
+                Route::post ('jobs/{jobPoster}/application/step-02/update', 'ApplicationByJobController@updateExperience')->name ('job.application.update.2');
 
                 /* Step 03 */
-                Route::post('jobs/{jobPoster}/application/step-03/update', 'ApplicationByJobController@updateEssentialSkills')->name('job.application.update.3');
+                Route::post ('jobs/{jobPoster}/application/step-03/update', 'ApplicationByJobController@updateEssentialSkills')->name ('job.application.update.3');
 
                 /* Step 04 */
-                Route::post('jobs/{jobPoster}/application/step-04/update', 'ApplicationByJobController@updateAssetSkills')->name('job.application.update.4');
+                Route::post ('jobs/{jobPoster}/application/step-04/update', 'ApplicationByJobController@updateAssetSkills')->name ('job.application.update.4');
 
                 /* Step 05 */
-                Route::post('jobs/{jobPoster}/application/submit', 'ApplicationByJobController@submit')->name('job.application.submit');
+                Route::post ('jobs/{jobPoster}/application/submit', 'ApplicationByJobController@submit')->name ('job.application.submit');
 
-                Route::get('profile', 'ApplicantProfileController@editAuthenticated')->name('profile');
-                Route::get('profile/about', 'ApplicantProfileController@editAuthenticated');
+                Route::get ('profile', 'ApplicantProfileController@editAuthenticated')->name ('profile');
+                Route::get ('profile/about', 'ApplicantProfileController@editAuthenticated');
 
                 /* Profile - About Me */
-                Route::get('profile/{applicant}/about', 'ApplicantProfileController@edit')
-                    ->middleware('can:view,applicant')
-                    ->middleware('can:update,applicant')
-                    ->name('profile.about.edit');
+                Route::get ('profile/{applicant}/about', 'ApplicantProfileController@edit')
+                    ->middleware ('can:view,applicant')
+                    ->middleware ('can:update,applicant')
+                    ->name ('profile.about.edit');
 
-                Route::post('profile/{applicant}/about/update', 'ApplicantProfileController@update')
-                    ->middleware('can:update,applicant')
-                    ->name('profile.about.update');
+                Route::post ('profile/{applicant}/about/update', 'ApplicantProfileController@update')
+                    ->middleware ('can:update,applicant')
+                    ->name ('profile.about.update');
 
                 /* Profile - My Experience */
-                Route::get('profile/experience', 'ExperienceController@editAuthenticated');
+                Route::get ('profile/experience', 'ExperienceController@editAuthenticated');
 
-                Route::get('profile/{applicant}/experience', 'ExperienceController@edit')
-                    ->middleware('can:view,applicant')
-                    ->middleware('can:update,applicant')
-                    ->name('profile.experience.edit');
+                Route::get ('profile/{applicant}/experience', 'ExperienceController@edit')
+                    ->middleware ('can:view,applicant')
+                    ->middleware ('can:update,applicant')
+                    ->name ('profile.experience.edit');
 
-                Route::post('profile/{applicant}/experience/update', 'ExperienceController@update')
-                    ->middleware('can:update,applicant')
-                    ->name('profile.experience.update');
+                Route::post ('profile/{applicant}/experience/update', 'ExperienceController@update')
+                    ->middleware ('can:update,applicant')
+                    ->name ('profile.experience.update');
 
                 /* Profile - My Skills */
-                Route::get('profile/skills', 'SkillDeclarationController@editAuthenticated');
+                Route::get ('profile/skills', 'SkillDeclarationController@editAuthenticated');
 
-                Route::get('profile/{applicant}/skills', 'SkillDeclarationController@edit')
-                    ->middleware('can:view,applicant')
-                    ->middleware('can:update,applicant')
-                    ->name('profile.skills.edit');
+                Route::get ('profile/{applicant}/skills', 'SkillDeclarationController@edit')
+                    ->middleware ('can:view,applicant')
+                    ->middleware ('can:update,applicant')
+                    ->name ('profile.skills.edit');
 
                 /* Profile - My References */
-                Route::get('profile/references', 'ReferencesController@editAuthenticated');
+                Route::get ('profile/references', 'ReferencesController@editAuthenticated');
 
-                Route::get('profile/{applicant}/references', 'ReferencesController@edit')
-                    ->middleware('can:view,applicant')
-                    ->middleware('can:update,applicant')
-                    ->name('profile.references.edit');
+                Route::get ('profile/{applicant}/references', 'ReferencesController@edit')
+                    ->middleware ('can:view,applicant')
+                    ->middleware ('can:update,applicant')
+                    ->name ('profile.references.edit');
 
                 /* Profile - My Portfolio */
-                Route::get('profile/portfolio', 'WorkSamplesController@editAuthenticated');
+                Route::get ('profile/portfolio', 'WorkSamplesController@editAuthenticated');
 
-                Route::get('profile/{applicant}/portfolio', 'WorkSamplesController@edit')
-                    ->middleware('can:view,applicant')
-                    ->middleware('can:update,applicant')
-                    ->name('profile.work_samples.edit');
+                Route::get ('profile/{applicant}/portfolio', 'WorkSamplesController@edit')
+                    ->middleware ('can:view,applicant')
+                    ->middleware ('can:update,applicant')
+                    ->name ('profile.work_samples.edit');
             });
 
             /* Static - FAQ */
-            Route::get('faq', 'FaqController')->name('faq');
+            Route::get ('faq', 'FaqController')->name ('faq');
 
             /* Static - Privacy Policy */
-            Route::view('privacy', 'common/static_privacy', ['privacy' => Lang::get('common/privacy')])
-                ->name('privacy');
+            Route::view ('privacy', 'common/static_privacy', ['privacy' => Lang::get ('common/privacy')])
+                ->name ('privacy');
 
             /* Static - Terms of Service */
-            Route::view('tos', 'common/static_tos', ['tos' => Lang::get('common/tos')])->name('tos');
+            Route::view ('tos', 'common/static_tos', ['tos' => Lang::get ('common/tos')])->name ('tos');
 
             /* Static - ITP */
-            Route::view('indigenous', 'common/static-itp', ['itp' => Lang::get('common/itp')])->name('itp');
+            Route::view ('indigenous', 'common/static-itp', ['itp' => Lang::get ('common/itp')])->name ('itp');
 
             /* Authentication =========================================================== */
 
             // Laravel default login, logout, register, and reset routes
-            Route::get('login', 'Auth\LoginController@showLoginForm')->name('login');
-            Route::post('login', 'Auth\LoginController@login')->name('login.post');
-            Route::post('logout', 'Auth\LoginController@logout')->name('logout');
+            Route::get ('login', 'Auth\LoginController@showLoginForm')->name ('login');
+            Route::post ('login', 'Auth\LoginController@login')->name ('login.post');
+            Route::post ('logout', 'Auth\LoginController@logout')->name ('logout');
 
             // Registration Routes...
-            Route::get('register', 'Auth\RegisterController@showRegistrationForm')->name('register');
-            Route::post('register', 'Auth\RegisterController@register')->name('register.post');
+            Route::get ('register', 'Auth\RegisterController@showRegistrationForm')->name ('register');
+            Route::post ('register', 'Auth\RegisterController@register')->name ('register.post');
 
             // Password Reset Routes...
-            Route::get('password/reset', 'Auth\ForgotPasswordController@showLinkRequestForm')->name('password.request');
-            Route::post('password/email', 'Auth\ForgotPasswordController@sendResetLinkEmail')->name('password.email');
-            Route::get('password/reset/{token}', 'Auth\ResetPasswordController@showResetForm')->name('password.reset');
-            Route::post('password/reset', 'Auth\ResetPasswordController@reset')->name('password.reset.post');
+            Route::get ('password/reset', 'Auth\ForgotPasswordController@showLinkRequestForm')->name ('password.request');
+            Route::post ('password/email', 'Auth\ForgotPasswordController@sendResetLinkEmail')->name ('password.email');
+            Route::get ('password/reset/{token}', 'Auth\ResetPasswordController@showResetForm')->name ('password.reset');
+            Route::post ('password/reset', 'Auth\ResetPasswordController@reset')->name ('password.reset.post');
         });
 
 
         /* Manager Portal =========================================================== */
 
-        Route::group([
-            'prefix' => config('app.manager_prefix'),
+        Route::group ([
+            'prefix' => config ('app.manager_prefix'),
         ], function (): void {
 
-            Route::middleware(['finishManagerRegistration'])->group(function (): void {
+            Route::middleware (['finishManagerRegistration'])->group (function (): void {
 
                 /* Home */
-                Route::get('/', 'HomepageController@manager')->name('manager.home');
+                Route::get ('/', 'HomepageController@manager')->name ('manager.home');
 
                 /* Static - FAQ */
-                Route::get(
+                Route::get (
                     'faq',
                     'ManagerProfileController@faq'
-                )->name('manager.faq');
+                )->name ('manager.faq');
 
-                Route::get(
+                Route::get (
                     'faq#manager-who',
                     'ManagerProfileController@faq'
-                )->name('manager.faq.section');
+                )->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');
+                    Route::get ('profile', 'ManagerProfileController@editAuthenticated')->name ('manager.profile');
 
                     /* Profile */
-                    Route::get('profile/{manager}/edit', 'ManagerProfileController@edit')
-                    ->middleware('can:view,manager')
-                    ->middleware('can:update,manager')
-                    ->name('manager.profile.edit');
+                    Route::get ('profile/{manager}/edit', 'ManagerProfileController@edit')
+                    ->middleware ('can:view,manager')
+                    ->middleware ('can:update,manager')
+                    ->name ('manager.profile.edit');
 
-                    Route::post('profile/{manager}/update', 'ManagerProfileController@update')
-                    ->middleware('can:update,manager')
-                    ->name('manager.profile.update');
+                    Route::post ('profile/{manager}/update', 'ManagerProfileController@update')
+                    ->middleware ('can:update,manager')
+                    ->name ('manager.profile.update');
 
                     /* View Application */
-                    Route::get('applications/{application}', 'ApplicationController@show')
-                    ->middleware('can:view,application')
-                    ->name('manager.applications.show');
+                    Route::get ('applications/{application}', 'ApplicationController@show')
+                    ->middleware ('can:view,application')
+                    ->name ('manager.applications.show');
 
                     /* View Applicant Profile */
-                    Route::get('applicants/{applicant}', 'ApplicantProfileController@show')
-                    ->middleware('can:view,applicant')
-                    ->name('manager.applicants.show');
+                    Route::get ('applicants/{applicant}', 'ApplicantProfileController@show')
+                    ->middleware ('can:view,applicant')
+                    ->name ('manager.applicants.show');
 
                     /* Job Index */
-                    Route::get('jobs', 'JobController@managerIndex')->name('manager.jobs.index');
+                    Route::get ('jobs', 'JobController@managerIndex')->name ('manager.jobs.index');
 
                     /* View Job Poster */
-                    Route::get('jobs/{jobPoster}', 'JobController@show')
-                    ->where('jobPoster', '[0-9]+')
-                    ->middleware('can:view,jobPoster')
-                    ->name('manager.jobs.show');
+                    Route::get ('jobs/{jobPoster}', 'JobController@show')
+                    ->where ('jobPoster', '[0-9]+')
+                    ->middleware ('can:view,jobPoster')
+                    ->name ('manager.jobs.show');
 
-                    Route::get('jobs/{jobPoster}/applications', 'ApplicationByJobController@index')
-                    ->where('jobPoster', '[0-9]+')
-                    ->middleware('can:reviewApplicationsFor,jobPoster')
-                    ->name('manager.jobs.applications');
+                    Route::get ('jobs/{jobPoster}/applications', 'ApplicationByJobController@index')
+                    ->where ('jobPoster', '[0-9]+')
+                    ->middleware ('can:reviewApplicationsFor,jobPoster')
+                    ->name ('manager.jobs.applications');
 
                     /* Job Builder */
-                    Route::get(
+                    Route::get (
                         'jobs/builder',
                         'JobBuilderController@show'
-                    )->name('manager.jobs.create');
+                    )->name ('manager.jobs.create');
 
-                    Route::get(
+                    Route::get (
                         'jobs/{jobId}/builder',
                         'JobBuilderController@show'
                     )
-                        ->where('jobPoster', '[0-9]+')
-                        ->name('manager.jobs.edit');
-                    Route::get(
+                        ->where ('jobPoster', '[0-9]+')
+                        ->name ('manager.jobs.edit');
+                    Route::get (
                         'jobs/{jobId}/builder/intro',
                         'JobBuilderController@show'
-                    )->where('jobPoster', '[0-9]+');
-                    Route::get(
+                    )->where ('jobPoster', '[0-9]+');
+                    Route::get (
                         'jobs/{jobId}/builder/details',
                         'JobBuilderController@show'
-                    )->where('jobPoster', '[0-9]+');
-                    Route::get(
+                    )->where ('jobPoster', '[0-9]+');
+                    Route::get (
                         'jobs/{jobId}/builder/environment',
                         'JobBuilderController@show'
-                    )->where('jobPoster', '[0-9]+');
-                    Route::get(
+                    )->where ('jobPoster', '[0-9]+');
+                    Route::get (
                         'jobs/{jobId}/builder/impact',
                         'JobBuilderController@show'
-                    )->where('jobPoster', '[0-9]+');
-                    Route::get(
+                    )->where ('jobPoster', '[0-9]+');
+                    Route::get (
                         'jobs/{jobId}/builder/tasks',
                         'JobBuilderController@show'
-                    )->where('jobPoster', '[0-9]+');
-                    Route::get(
+                    )->where ('jobPoster', '[0-9]+');
+                    Route::get (
                         'jobs/{jobId}/builder/skills',
                         'JobBuilderController@show'
-                    )->where('jobPoster', '[0-9]+');
-                    Route::get(
+                    )->where ('jobPoster', '[0-9]+');
+                    Route::get (
                         'jobs/{jobId}/builder/review',
                         'JobBuilderController@show'
                     )
-                        ->where('jobPoster', '[0-9]+')
-                        ->name('manager.jobs.review');
+                        ->where ('jobPoster', '[0-9]+')
+                        ->name ('manager.jobs.review');
 
                     /* Delete Job */
-                    Route::delete('jobs/{jobPoster}', 'JobController@destroy')
-                        ->where('jobPoster', '[0-9]+')
-                        ->middleware('can:delete,jobPoster')
-                        ->name('manager.jobs.destroy');
+                    Route::delete ('jobs/{jobPoster}', 'JobController@destroy')
+                        ->where ('jobPoster', '[0-9]+')
+                        ->middleware ('can:delete,jobPoster')
+                        ->name ('manager.jobs.destroy');
 
-                    Route::view(
+                    Route::view (
                         'jobs/{jobPoster}/assessment-plan',
                         'manager/assessment_plan'
                     )
-                        ->where('jobPoster', '[0-9]+')
-                        ->name('manager.jobs.screening_plan');
+                        ->where ('jobPoster', '[0-9]+')
+                        ->name ('manager.jobs.screening_plan');
                 });
             });
 
             // 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::get('first-visit', 'Auth\FirstVisitController@showFirstVisitManagerForm')
-                    ->name('manager.first_visit');
-                Route::post('finish_registration', 'Auth\FirstVisitController@finishManagerRegistration')
-                    ->name('manager.finish_registration');
+            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')
+                    ->name ('manager.finish_registration');
             });
 
             // Laravel default login, logout, register, and reset routes
-            Route::get('login', 'Auth\LoginController@showLoginForm')->name('manager.login');
-            Route::post('login', 'Auth\LoginController@login')->name('manager.login.post');
-            Route::post('logout', 'Auth\LoginController@logout')->name('manager.logout');
+            Route::get ('login', 'Auth\LoginController@showLoginForm')->name ('manager.login');
+            Route::post ('login', 'Auth\LoginController@login')->name ('manager.login.post');
+            Route::post ('logout', 'Auth\LoginController@logout')->name ('manager.logout');
 
             // Registration Routes...
-            Route::get('register', 'Auth\RegisterController@showManagerRegistrationForm')->name('manager.register');
-            Route::post('register', 'Auth\RegisterController@registerManager')->name('manager.register.post');
+            Route::get ('register', 'Auth\RegisterController@showManagerRegistrationForm')->name ('manager.register');
+            Route::post ('register', 'Auth\RegisterController@registerManager')->name ('manager.register.post');
 
             // Password Reset Routes...
-            Route::get('password/reset', 'Auth\ForgotPasswordController@showLinkRequestForm')->name('manager.password.request');
-            Route::post('password/email', 'Auth\ForgotPasswordController@sendResetLinkEmail')->name('manager.password.email');
-            Route::get('password/reset/{token}', 'Auth\ResetPasswordController@showResetForm')->name('manager.password.reset');
-            Route::post('password/reset', 'Auth\ResetPasswordController@reset')->name('manager.password.reset.post');
+            Route::get ('password/reset', 'Auth\ForgotPasswordController@showLinkRequestForm')->name ('manager.password.request');
+            Route::post ('password/email', 'Auth\ForgotPasswordController@sendResetLinkEmail')->name ('manager.password.email');
+            Route::get ('password/reset/{token}', 'Auth\ResetPasswordController@showResetForm')->name ('manager.password.reset');
+            Route::post ('password/reset', 'Auth\ResetPasswordController@reset')->name ('manager.password.reset.post');
         });
 
         /* 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')
-                ->name('courses.destroy');
+            Route::delete ('courses/{course}', 'CourseController@destroy')
+                ->middleware ('can:delete,course')
+                ->name ('courses.destroy');
 
-            Route::delete('degrees/{degree}', 'DegreeController@destroy')
-                ->middleware('can:delete,degree')
-                ->name('degrees.destroy');
+            Route::delete ('degrees/{degree}', 'DegreeController@destroy')
+                ->middleware ('can:delete,degree')
+                ->name ('degrees.destroy');
 
-            Route::delete('work-experiences/{workExperience}', 'WorkExperienceController@destroy')
-                ->middleware('can:delete,workExperience')
-                ->name('work_experiences.destroy');
+            Route::delete ('work-experiences/{workExperience}', 'WorkExperienceController@destroy')
+                ->middleware ('can:delete,workExperience')
+                ->name ('work_experiences.destroy');
 
-            Route::post('skill-declarations', 'SkillDeclarationController@create')
-                ->middleware('can:create,App\Models\SkillDeclaration')
-                ->name('skill_declarations.create');
+            Route::post ('skill-declarations', 'SkillDeclarationController@create')
+                ->middleware ('can:create,App\Models\SkillDeclaration')
+                ->name ('skill_declarations.create');
 
-            Route::put('skill-declarations/{skillDeclaration}', 'SkillDeclarationController@update')
-                ->middleware('can:update,skillDeclaration')
-                ->name('skill_declarations.update');
+            Route::put ('skill-declarations/{skillDeclaration}', 'SkillDeclarationController@update')
+                ->middleware ('can:update,skillDeclaration')
+                ->name ('skill_declarations.update');
 
-            Route::delete('skill-declarations/{skillDeclaration}', 'SkillDeclarationController@destroy')
-                ->middleware('can:delete,skillDeclaration')
-                ->name('skill_declarations.destroy');
+            Route::delete ('skill-declarations/{skillDeclaration}', 'SkillDeclarationController@destroy')
+                ->middleware ('can:delete,skillDeclaration')
+                ->name ('skill_declarations.destroy');
 
-            Route::post('references', 'ReferencesController@update')
-                ->middleware('can:create,App\Models\Reference')
-                ->name('references.create');
+            Route::post ('references', 'ReferencesController@update')
+                ->middleware ('can:create,App\Models\Reference')
+                ->name ('references.create');
 
-            Route::put('references/{reference}', 'ReferencesController@update')
-                ->middleware('can:update,reference')
-                ->name('references.update');
+            Route::put ('references/{reference}', 'ReferencesController@update')
+                ->middleware ('can:update,reference')
+                ->name ('references.update');
 
-            Route::delete('references/{reference}', 'ReferencesController@destroy')
-                ->middleware('can:delete,reference')
-                ->name('references.destroy');
+            Route::delete ('references/{reference}', 'ReferencesController@destroy')
+                ->middleware ('can:delete,reference')
+                ->name ('references.destroy');
 
-            Route::post('work-samples', 'WorkSamplesController@update')
-                ->middleware('can:create,App\Models\WorkSample')
-                ->name('work_samples.create');
+            Route::post ('work-samples', 'WorkSamplesController@update')
+                ->middleware ('can:create,App\Models\WorkSample')
+                ->name ('work_samples.create');
 
-            Route::put('work-samples/{workSample}', 'WorkSamplesController@update')
-                ->middleware('can:update,workSample')
-                ->name('work_samples.update');
+            Route::put ('work-samples/{workSample}', 'WorkSamplesController@update')
+                ->middleware ('can:update,workSample')
+                ->name ('work_samples.update');
 
-            Route::delete('work-samples/{workSample}', 'WorkSamplesController@destroy')
-                ->middleware('can:delete,workSample')
-                ->name('work_samples.destroy');
+            Route::delete ('work-samples/{workSample}', 'WorkSamplesController@destroy')
+                ->middleware ('can:delete,workSample')
+                ->name ('work_samples.destroy');
 
-            Route::delete('applications/{application}', 'ApplicationController@destroy')
-                ->middleware('can:delete,application')
-                ->name('applications.destroy');
+            Route::delete ('applications/{application}', 'ApplicationController@destroy')
+                ->middleware ('can:delete,application')
+                ->name ('applications.destroy');
 
-            Route::put('applications/{application}/review', 'ApplicationReviewController@updateForApplication')
-                ->middleware('can:review,application')
-                ->name('application_reviews.update');
+            Route::put ('applications/{application}/review', 'ApplicationReviewController@updateForApplication')
+                ->middleware ('can:review,application')
+                ->name ('application_reviews.update');
         });
 
         /* Non-Backpack Admin Portal (localized pages) =========================================================== */
-        Route::group(
+        Route::group (
             [
                 'prefix' => 'admin',
                 'middleware' => ['auth', 'role:admin']
             ],
             function (): void {
                 /* Edit Job */
-                Route::get('jobs/{jobPoster}/edit', 'JobController@edit')
-                    ->where('jobPoster', '[0-9]+')
-                    ->middleware('can:update,jobPoster')
-                    ->name('admin.jobs.edit');
-                Route::post('jobs/{jobPoster}', 'JobController@store')
-                    ->where('jobPoster', '[0-9]+')
-                    ->middleware('can:update,jobPoster')
-                    ->name('admin.jobs.update');
+                Route::get ('jobs/{jobPoster}/edit', 'JobController@edit')
+                    ->where ('jobPoster', '[0-9]+')
+                    ->middleware ('can:update,jobPoster')
+                    ->name ('admin.jobs.edit');
+                Route::post ('jobs/{jobPoster}', 'JobController@store')
+                    ->where ('jobPoster', '[0-9]+')
+                    ->middleware ('can:update,jobPoster')
+                    ->name ('admin.jobs.update');
             }
         );
 
         /* HR Advisor Portal =========================================================== */
 
-        Route::group([
-            'prefix' => config('app.hr_prefix'),
+        Route::group ([
+            'prefix' => config ('app.hr_prefix'),
         ], function (): void {
 
-            Route::middleware(['finishHrRegistration'])->group(function (): void {
+            Route::middleware (['finishHrRegistration'])->group (function (): void {
 
-                Route::get('/', 'HomepageController@hr_advisor')->name('hr_advisor.home');
+                Route::get ('/', 'HomepageController@hr_advisor')->name ('hr_advisor.home');
             });
 
             // These routes must be excluded from the finishHrAdvisorRegistration middleware to avoid an infinite loop of redirects
-            Route::middleware(['auth', 'role:hr_advisor'])->group(function (): void {
-                Route::get('first-visit', 'Auth\FirstVisitController@showFirstVisitHrForm')
-                    ->name('hr_advisor.first_visit');
-                Route::post('finish_registration', 'Auth\FirstVisitController@finishHrRegistration')
-                    ->name('hr_advisor.finish_registration');
+            Route::middleware (['auth', 'role:hr_advisor'])->group (function (): void {
+                Route::get ('first-visit', 'Auth\FirstVisitController@showFirstVisitHrForm')
+                    ->name ('hr_advisor.first_visit');
+                Route::post ('finish_registration', 'Auth\FirstVisitController@finishHrRegistration')
+                    ->name ('hr_advisor.finish_registration');
             });
 
             // Laravel default login, logout, register, and reset routes
-            Route::get('login', 'Auth\LoginController@showLoginForm')->name('hr_advisor.login');
-            Route::post('login', 'Auth\LoginController@login')->name('hr_advisor.login.post');
-            Route::post('logout', 'Auth\LoginController@logout')->name('hr_advisor.logout');
+            Route::get ('login', 'Auth\LoginController@showLoginForm')->name ('hr_advisor.login');
+            Route::post ('login', 'Auth\LoginController@login')->name ('hr_advisor.login.post');
+            Route::post ('logout', 'Auth\LoginController@logout')->name ('hr_advisor.logout');
 
             // Registration Routes...
-            Route::get('register', 'Auth\RegisterController@showHrRegistrationForm')->name('hr_advisor.register');
-            Route::post('register', 'Auth\RegisterController@registerHrAdvisor')->name('hr_advisor.register.post');
+            Route::get ('register', 'Auth\RegisterController@showHrRegistrationForm')->name ('hr_advisor.register');
+            Route::post ('register', 'Auth\RegisterController@registerHrAdvisor')->name ('hr_advisor.register.post');
 
             // Password Reset Routes...
-            Route::get('password/reset', 'Auth\ForgotPasswordController@showLinkRequestForm')->name('hr_advisor.password.request');
-            Route::post('password/email', 'Auth\ForgotPasswordController@sendResetLinkEmail')->name('hr_advisor.password.email');
-            Route::get('password/reset/{token}', 'Auth\ResetPasswordController@showResetForm')->name('hr_advisor.password.reset');
-            Route::post('password/reset', 'Auth\ResetPasswordController@reset')->name('hr_advisor.password.reset.post');
+            Route::get ('password/reset', 'Auth\ForgotPasswordController@showLinkRequestForm')->name ('hr_advisor.password.request');
+            Route::post ('password/email', 'Auth\ForgotPasswordController@sendResetLinkEmail')->name ('hr_advisor.password.email');
+            Route::get ('password/reset/{token}', 'Auth\ResetPasswordController@showResetForm')->name ('hr_advisor.password.reset');
+            Route::post ('password/reset', 'Auth\ResetPasswordController@reset')->name ('hr_advisor.password.reset.post');
         });
     }
 );
 
 /* Non-Backpack Admin Portal (non-localized pages) =========================================================== */
-Route::group(
+Route::group (
     [
         'prefix' => 'admin',
         'middleware' => ['auth', 'role:admin']
     ],
     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')
-            ->name('admin.jobs.create_as_manager');
+        Route::post ('jobs/create/as-manager/{manager}', 'JobController@createAsManager')
+            ->middleware ('can:create,App\Models\JobPoster')
+            ->name ('admin.jobs.create_as_manager');
     }
 );
 
@@ -483,85 +483,85 @@ 
 
 
 /** 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');
+    Route::get ('jobs/{jobPoster}/assessment-plan', 'AssessmentPlanController@getForJob');
     // Public, not protected by policy or gate
-    Route::get('skills', 'Api\SkillController@index');
-    Route::get('departments', 'Api\DepartmentController@index');
+    Route::get ('skills', 'Api\SkillController@index');
+    Route::get ('departments', 'Api\DepartmentController@index');
 
     // Resource Routes are protected by policies in controllers instead of middleware.
-    Route::resource('assessments', 'AssessmentController')->except([
+    Route::resource ('assessments', 'AssessmentController')->except ([
         'create', 'edit', 'index'
     ]);
-    Route::apiResource('rating-guide-answers', 'RatingGuideAnswerController')->except([
+    Route::apiResource ('rating-guide-answers', 'RatingGuideAnswerController')->except ([
         'index'
-    ])->parameters([
+    ])->parameters ([
         'rating-guide-answers' => 'ratingGuideAnswer'
     ]);
-    Route::resource('rating-guide-questions', 'RatingGuideQuestionController')->except([
+    Route::resource ('rating-guide-questions', 'RatingGuideQuestionController')->except ([
         'create', 'edit', 'index'
     ]);
-    Route::resource('assessment-plan-notifications', 'AssessmentPlanNotificationController')->except([
+    Route::resource ('assessment-plan-notifications', 'AssessmentPlanNotificationController')->except ([
         'store', 'create', 'edit'
     ]);
     // TODO: add policy middleware
-    Route::get('jobs/{jobPoster}/tasks', 'Api\JobTaskController@indexByJob')
-        ->where('jobPoster', '[0-9]+')
-        ->middleware('can:view,jobPoster');
-    Route::put('jobs/{jobPoster}/tasks', 'Api\JobTaskController@batchUpdate')
-        ->where('jobPoster', '[0-9]+')
-        ->middleware('can:update,jobPoster');
-
-
-    Route::get('jobs/{jobPoster}/criteria', 'Api\CriteriaController@indexByJob')
-        ->where('jobPoster', '[0-9]+')
-        ->middleware('can:view,jobPoster');
-    Route::put('jobs/{jobPoster}/criteria', 'Api\CriteriaController@batchUpdate')
-        ->where('jobPoster', '[0-9]+')
-        ->middleware('can:update,jobPoster');
-
-
-    Route::post('jobs/{job}/submit', 'Api\JobApiController@submitForReview')
-        ->where('job', '[0-9]+')
-        ->middleware('can:submitForReview,job')
-        ->name('api.jobs.submit');
-    Route::resource('jobs', 'Api\JobApiController')->only([
+    Route::get ('jobs/{jobPoster}/tasks', 'Api\JobTaskController@indexByJob')
+        ->where ('jobPoster', '[0-9]+')
+        ->middleware ('can:view,jobPoster');
+    Route::put ('jobs/{jobPoster}/tasks', 'Api\JobTaskController@batchUpdate')
+        ->where ('jobPoster', '[0-9]+')
+        ->middleware ('can:update,jobPoster');
+
+
+    Route::get ('jobs/{jobPoster}/criteria', 'Api\CriteriaController@indexByJob')
+        ->where ('jobPoster', '[0-9]+')
+        ->middleware ('can:view,jobPoster');
+    Route::put ('jobs/{jobPoster}/criteria', 'Api\CriteriaController@batchUpdate')
+        ->where ('jobPoster', '[0-9]+')
+        ->middleware ('can:update,jobPoster');
+
+
+    Route::post ('jobs/{job}/submit', 'Api\JobApiController@submitForReview')
+        ->where ('job', '[0-9]+')
+        ->middleware ('can:submitForReview,job')
+        ->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'
     ]);
 
-    Route::resource('managers', 'Api\ManagerApiController')->only([
+    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'
     ]);
 
     // User must be logged in to user currentuser routes
-    Route::get('currentuser/manager', 'Api\ManagerApiController@showAuthenticated')
-        ->middleware('auth');
+    Route::get ('currentuser/manager', 'Api\ManagerApiController@showAuthenticated')
+        ->middleware ('auth');
 
     // Claim / unclaim job routes, HR portal
-    Route::put('jobs/{job}/claim', 'Api\ClaimJobApiController@store')
-        ->middleware('can:claim,job')
-        ->where('job', '[0-9]+');
-    Route::delete('jobs/{job}/claim', 'Api\ClaimJobApiController@destroy')
-        ->middleware('can:unClaim,job')
-        ->where('job', '[0-9]+');
-
-    Route::get('hr-advisors/{hrAdvisor}', 'Api\HrAdvisorController@show')
-        ->middleware('can:view,hrAdvisor');
-
-    Route::put('hr-advisors/{hrAdvisor}/claims/{job}', 'Api\ClaimJobApiController@claimJob')
-        ->middleware('can:update,hrAdvisor')
-        ->where('hrAdvisor', '[0-9]+')
-        ->where('job', '[0-9]+');
-    Route::delete('hr-advisors/{hrAdvisor}/claims/{job}', 'Api\ClaimJobApiController@unclaimJob')
-        ->middleware('can:update,hrAdvisor')
-        ->where('hrAdvisor', '[0-9]+')
-        ->where('job', '[0-9]+');
+    Route::put ('jobs/{job}/claim', 'Api\ClaimJobApiController@store')
+        ->middleware ('can:claim,job')
+        ->where ('job', '[0-9]+');
+    Route::delete ('jobs/{job}/claim', 'Api\ClaimJobApiController@destroy')
+        ->middleware ('can:unClaim,job')
+        ->where ('job', '[0-9]+');
+
+    Route::get ('hr-advisors/{hrAdvisor}', 'Api\HrAdvisorController@show')
+        ->middleware ('can:view,hrAdvisor');
+
+    Route::put ('hr-advisors/{hrAdvisor}/claims/{job}', 'Api\ClaimJobApiController@claimJob')
+        ->middleware ('can:update,hrAdvisor')
+        ->where ('hrAdvisor', '[0-9]+')
+        ->where ('job', '[0-9]+');
+    Route::delete ('hr-advisors/{hrAdvisor}/claims/{job}', 'Api\ClaimJobApiController@unclaimJob')
+        ->middleware ('can:update,hrAdvisor')
+        ->where ('hrAdvisor', '[0-9]+')
+        ->where ('job', '[0-9]+');
 });

app/Policies/HrAdvisorPolicy.php

Spacing +2 added lines, -2 removed lines

@@ -16,11 +16,11 @@
      */
     public function view(User $user, HrAdvisor $hrAdvisor)
     {
-        return $user->isHrAdvisor() && $hrAdvisor->user_id === $user->id;
+        return $user->isHrAdvisor () && $hrAdvisor->user_id === $user->id;
     }
 
     public function update(User $user, HrAdvisor $hrAdvisor)
     {
-        return $user->isHrAdvisor() && $hrAdvisor->user_id === $user->id;
+        return $user->isHrAdvisor () && $hrAdvisor->user_id === $user->id;
     }
 }

app/Models/HrAdvisor.php

Spacing +5 added lines, -5 removed lines

@@ -52,17 +52,17 @@ 
 
     public function user()
     {
-        return $this->belongsTo(\App\Models\User::class);
+        return $this->belongsTo (\App\Models\User::class);
     }
 
     public function department()
     {
-        return $this->belongsTo(\App\Models\Lookup\Department::class);
+        return $this->belongsTo (\App\Models\Lookup\Department::class);
     }
 
     public function claimed_jobs()
     {
-        return $this->belongsToMany(
+        return $this->belongsToMany (
             \App\Models\JobPoster::class,
             'claimed_jobs'
         );
@@ -76,7 +76,7 @@ 
     public function getNameAttribute(): string
     {
         if ($this->user !== null) {
-            return $this->user->first_name . ' ' . $this->user->last_name;
+            return $this->user->first_name.' '.$this->user->last_name;
         }
         return '';
     }
@@ -88,6 +88,6 @@ 
      */
     public function getClaimedJobIdsAttribute()
     {
-        return $this->claimed_jobs()->allRelatedIds()->toArray();
+        return $this->claimed_jobs ()->allRelatedIds ()->toArray ();
     }
 }

app/Models/JobPoster.php

Spacing +39 added lines, -39 removed lines

@@ -273,42 +273,42 @@ 
     // @codeCoverageIgnoreStart
     public function department() // phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\Lookup\Department::class);
+        return $this->belongsTo (\App\Models\Lookup\Department::class);
     }
 
     public function job_term() // phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\Lookup\JobTerm::class);
+        return $this->belongsTo (\App\Models\Lookup\JobTerm::class);
     }
 
     public function language_requirement() // phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\Lookup\LanguageRequirement::class);
+        return $this->belongsTo (\App\Models\Lookup\LanguageRequirement::class);
     }
 
     public function manager() // phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\Manager::class);
+        return $this->belongsTo (\App\Models\Manager::class);
     }
 
     public function province() // phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\Lookup\Province::class);
+        return $this->belongsTo (\App\Models\Lookup\Province::class);
     }
 
     public function security_clearance() // phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\Lookup\SecurityClearance::class);
+        return $this->belongsTo (\App\Models\Lookup\SecurityClearance::class);
     }
 
     public function criteria() // phpcs:ignore
     {
-        return $this->hasMany(\App\Models\Criteria::class);
+        return $this->hasMany (\App\Models\Criteria::class);
     }
 
     public function hr_advisors() // phpcs:ignore
     {
-        return $this->belongsToMany(
+        return $this->belongsToMany (
             \App\Models\HrAdvisor::class,
             'claimed_jobs'
         );
@@ -316,47 +316,47 @@ 
 
     public function job_applications() // phpcs:ignore
     {
-        return $this->hasMany(\App\Models\JobApplication::class);
+        return $this->hasMany (\App\Models\JobApplication::class);
     }
 
     public function job_poster_key_tasks() // phpcs:ignore
     {
-        return $this->hasMany(\App\Models\JobPosterKeyTask::class);
+        return $this->hasMany (\App\Models\JobPosterKeyTask::class);
     }
 
     public function job_poster_questions() // phpcs:ignore
     {
-        return $this->hasMany(\App\Models\JobPosterQuestion::class);
+        return $this->hasMany (\App\Models\JobPosterQuestion::class);
     }
 
     public function job_poster_translations() // phpcs:ignore
     {
-        return $this->hasMany(\App\Models\JobPosterTranslation::class);
+        return $this->hasMany (\App\Models\JobPosterTranslation::class);
     }
 
     public function telework_allowed_frequency() // phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\Lookup\Frequency::class);
+        return $this->belongsTo (\App\Models\Lookup\Frequency::class);
     }
 
     public function flexible_hours_frequency() // phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\Lookup\Frequency::class);
+        return $this->belongsTo (\App\Models\Lookup\Frequency::class);
     }
 
     public function travel_requirement() // phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\Lookup\TravelRequirement::class);
+        return $this->belongsTo (\App\Models\Lookup\TravelRequirement::class);
     }
 
     public function overtime_requirement() // phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\Lookup\OvertimeRequirement::class);
+        return $this->belongsTo (\App\Models\Lookup\OvertimeRequirement::class);
     }
 
     public function classification() // phpcs:ignore
     {
-        return $this->belongsTo(\App\Models\Classification::class);
+        return $this->belongsTo (\App\Models\Classification::class);
     }
     // @codeCoverageIgnoreEnd
     /* Artificial Relations */
@@ -369,8 +369,8 @@ 
      */
     public function submitted_applications() // phpcs:ignore
     {
-        return $this->hasMany(\App\Models\JobApplication::class)->whereDoesntHave('application_status', function ($query): void {
-            $query->where('name', 'draft');
+        return $this->hasMany (\App\Models\JobApplication::class)->whereDoesntHave ('application_status', function ($query): void {
+            $query->where ('name', 'draft');
         });
     }
 
@@ -389,7 +389,7 @@ 
      */
     public function resolveRouteBinding($value) // phpcs:ignore
     {
-        return $this->withCount('submitted_applications')->where('id', $value)->first() ?? abort(404);
+        return $this->withCount ('submitted_applications')->where ('id', $value)->first () ?? abort (404);
     }
 
     /**
@@ -403,7 +403,7 @@ 
     public function setPublishedAttribute($value): void
     {
         if ($value) {
-            $this->attributes['published_at'] = new Date();
+            $this->attributes['published_at'] = new Date ();
         } else {
             $this->attributes['published_at'] = null;
         }
@@ -417,7 +417,7 @@ 
 
     public function submitted_applications_count() //phpcs:ignore
     {
-        return $this->submitted_applications()->count();
+        return $this->submitted_applications ()->count ();
     }
 
     /**
@@ -427,15 +427,15 @@ 
      */
     public function applyBy(): array
     {
-        $localCloseDate = new Date($this->close_date_time); // This initializes the date object in UTC time.
-        $localCloseDate->setTimezone(new \DateTimeZone(self::TIMEZONE)); // Then set the time zone for display.
+        $localCloseDate = new Date ($this->close_date_time); // This initializes the date object in UTC time.
+        $localCloseDate->setTimezone (new \DateTimeZone (self::TIMEZONE)); // Then set the time zone for display.
         $displayDate = [
-            'date' => $localCloseDate->format(self::DATE_FORMAT[App::getLocale()]),
-            'time' => $localCloseDate->format(self::TIME_FORMAT[App::getLocale()])
+            'date' => $localCloseDate->format (self::DATE_FORMAT[App::getLocale ()]),
+            'time' => $localCloseDate->format (self::TIME_FORMAT[App::getLocale ()])
         ];
 
-        if (App::isLocale('fr')) {
-            $displayDate['time'] = str_replace(['EST', 'EDT'], ['HNE', 'HAE'], $displayDate['time']);
+        if (App::isLocale ('fr')) {
+            $displayDate['time'] = str_replace (['EST', 'EDT'], ['HNE', 'HAE'], $displayDate['time']);
         }
 
         return $displayDate;
@@ -449,7 +449,7 @@ 
      */
     public function displayStatus(): string
     {
-        return $this->isOpen() ? 'Open' : 'Closed';
+        return $this->isOpen () ? 'Open' : 'Closed';
     }
 
     /**
@@ -462,8 +462,8 @@ 
         return $this->published
             && $this->open_date_time !== null
             && $this->close_date_time !== null
-            && $this->open_date_time->isPast()
-            && $this->close_date_time->isFuture();
+            && $this->open_date_time->isPast ()
+            && $this->close_date_time->isFuture ();
     }
 
     /**
@@ -476,8 +476,8 @@ 
         return $this->published
             && $this->open_date_time !== null
             && $this->close_date_time !== null
-            && $this->open_date_time->isPast()
-            && $this->close_date_time->isPast();
+            && $this->open_date_time->isPast ()
+            && $this->close_date_time->isPast ();
     }
 
     /**
@@ -487,7 +487,7 @@ 
      */
     public function timeRemaining(): string
     {
-        $interval = $this->close_date_time->diff(Date::now());
+        $interval = $this->close_date_time->diff (Date::now ());
 
         $d = $interval->d;
         $h = $interval->h;
@@ -510,7 +510,7 @@ 
 
         $key = "common/time.$unit";
 
-        return Lang::choice($key, $count);
+        return Lang::choice ($key, $count);
     }
 
     /**
@@ -522,9 +522,9 @@ 
     public function status(): string
     {
         $status = 'draft';
-        if ($this->isOpen()) {
+        if ($this->isOpen ()) {
             $status = 'published';
-        } elseif ($this->isClosed()) {
+        } elseif ($this->isClosed ()) {
             $status = 'closed';
         } elseif ($this->review_requested_at !== null) {
             $status = 'submitted';
@@ -558,7 +558,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;
     }
@@ -571,7 +571,7 @@ 
      */
     public function toApiArray(): array
     {
-        $jobWithTranslations = array_merge($this->toArray(), $this->getTranslationsArray());
+        $jobWithTranslations = array_merge ($this->toArray (), $this->getTranslationsArray ());
         return $jobWithTranslations;
     }
 }

app/Http/Controllers/JobController.php

Spacing +71 added lines, -71 removed lines

@@ -23,26 +23,26 @@ 
      */
     public function index()
     {
-        $now = Carbon::now();
+        $now = Carbon::now ();
 
         // Find published jobs that are currently open for applications.
         // Eager load required relationships: Department, Province, JobTerm.
         // Eager load the count of submitted applications, to prevent the relationship
         // from being actually loaded and firing off events.
-        $jobs = JobPoster::where('open_date_time', '<=', $now)
-            ->where('close_date_time', '>=', $now)
-            ->where('published', true)
-            ->with([
+        $jobs = JobPoster::where ('open_date_time', '<=', $now)
+            ->where ('close_date_time', '>=', $now)
+            ->where ('published', true)
+            ->with ([
                 'department',
                 'province',
                 'job_term',
             ])
-            ->withCount([
+            ->withCount ([
                 'submitted_applications',
             ])
-            ->get();
-        return view('applicant/job_index', [
-            'job_index' => Lang::get('applicant/job_index'),
+            ->get ();
+        return view ('applicant/job_index', [
+            'job_index' => Lang::get ('applicant/job_index'),
             'jobs' => $jobs
         ]);
     }
@@ -54,31 +54,31 @@ 
      */
     public function managerIndex()
     {
-        $manager = Auth::user()->manager;
+        $manager = Auth::user ()->manager;
 
-        $jobs = JobPoster::where('manager_id', $manager->id)
-            ->with('classification')
-            ->withCount('submitted_applications')
-            ->get();
+        $jobs = JobPoster::where ('manager_id', $manager->id)
+            ->with ('classification')
+            ->withCount ('submitted_applications')
+            ->get ();
 
         foreach ($jobs as &$job) {
             $chosen_lang = $job->chosen_lang;
 
             // Show chosen lang title if current title is empty.
             if (empty($job->title)) {
-                $job->title = $job->translate($chosen_lang)->title;
+                $job->title = $job->translate ($chosen_lang)->title;
                 $job->trans_required = true;
             }
 
             // Always preview and edit in the chosen language.
-            $job->preview_link = LaravelLocalization::getLocalizedURL($chosen_lang, route('manager.jobs.show', $job));
-            $job->edit_link = LaravelLocalization::getLocalizedURL($chosen_lang, route('manager.jobs.edit', $job));
+            $job->preview_link = LaravelLocalization::getLocalizedURL ($chosen_lang, route ('manager.jobs.show', $job));
+            $job->edit_link = LaravelLocalization::getLocalizedURL ($chosen_lang, route ('manager.jobs.edit', $job));
         }
 
 
-        return view('manager/job_index', [
+        return view ('manager/job_index', [
             // Localization Strings.
-            'jobs_l10n' => Lang::get('manager/job_index'),
+            'jobs_l10n' => Lang::get ('manager/job_index'),
             // Data.
             'jobs' => $jobs,
         ]);
@@ -93,7 +93,7 @@ 
      */
     public function destroy(Request $request, JobPoster $jobPoster)
     {
-        $jobPoster->delete();
+        $jobPoster->delete ();
     }
 
     /**
@@ -105,14 +105,14 @@ 
      */
     public function show(Request $request, JobPoster $jobPoster)
     {
-        $jobPoster->load([
+        $jobPoster->load ([
             'department',
             'criteria.skill.skill_type',
             'manager.team_culture',
             'manager.work_environment'
         ]);
 
-        $user = Auth::user();
+        $user = Auth::user ();
 
         // TODO: Improve workplace photos, and reference them in template direction from WorkEnvironment model.
         $workplacePhotos = [];
@@ -125,36 +125,36 @@ 
 
         // TODO: replace route('manager.show',manager.id) in templates with link using slug.
         $criteria = [
-            'essential' => $jobPoster->criteria->filter(
-                function ($value, $key) {
+            'essential' => $jobPoster->criteria->filter (
+                function ($value, $key){
                     return $value->criteria_type->name == 'essential';
                 }
             ),
-            'asset' => $jobPoster->criteria->filter(
-                function ($value, $key) {
+            'asset' => $jobPoster->criteria->filter (
+                function ($value, $key){
                     return $value->criteria_type->name == 'asset';
                 }
             ),
         ];
 
-        $jobLang = Lang::get('applicant/job_post');
+        $jobLang = Lang::get ('applicant/job_post');
 
         $applyButton = [];
-        if (!$jobPoster->published && $this->authorize('update', $jobPoster)) {
+        if (!$jobPoster->published && $this->authorize ('update', $jobPoster)) {
             $applyButton = [
-                'href' => route('manager.jobs.edit', $jobPoster->id),
+                'href' => route ('manager.jobs.edit', $jobPoster->id),
                 'title' => $jobLang['apply']['edit_link_title'],
                 'text' => $jobLang['apply']['edit_link_label'],
             ];
-        } elseif (Auth::check() && $jobPoster->isOpen()) {
+        } elseif (Auth::check () && $jobPoster->isOpen ()) {
             $applyButton = [
-                'href' => route('job.application.edit.1', $jobPoster->id),
+                'href' => route ('job.application.edit.1', $jobPoster->id),
                 'title' => $jobLang['apply']['apply_link_title'],
                 'text' => $jobLang['apply']['apply_link_label'],
             ];
-        } elseif (Auth::guest() && $jobPoster->isOpen()) {
+        } elseif (Auth::guest () && $jobPoster->isOpen ()) {
             $applyButton = [
-                'href' => route('job.application.edit.1', $jobPoster->id),
+                'href' => route ('job.application.edit.1', $jobPoster->id),
                 'title' => $jobLang['apply']['login_link_title'],
                 'text' => $jobLang['apply']['login_link_label'],
             ];
@@ -166,19 +166,19 @@ 
             ];
         }
 
-        $jpb_release_date = strtotime('2019-08-21 16:18:17');
-        $job_created_at = strtotime($jobPoster->created_at);
+        $jpb_release_date = strtotime ('2019-08-21 16:18:17');
+        $job_created_at = strtotime ($jobPoster->created_at);
 
         // If the job poster is created after the release of the JPB.
         // Then, render with updated poster template.
         // Else, render with old poster template.
         if ($job_created_at > $jpb_release_date) {
             // Updated job poster (JPB).
-            return view(
+            return view (
                 'applicant/jpb_job_post',
                 [
                     'job_post' => $jobLang,
-                    'skill_template' => Lang::get('common/skills'),
+                    'skill_template' => Lang::get ('common/skills'),
                     'job' => $jobPoster,
                     'manager' => $jobPoster->manager,
                     'criteria' => $criteria,
@@ -187,7 +187,7 @@ 
             );
         } else {
             // Old job poster.
-            return view(
+            return view (
                 'applicant/job_post',
                 [
                     'job_post' => $jobLang,
@@ -199,7 +199,7 @@ 
                     'job' => $jobPoster,
                     'criteria' => $criteria,
                     'apply_button' => $applyButton,
-                    'skill_template' => Lang::get('common/skills'),
+                    'skill_template' => Lang::get ('common/skills'),
                 ]
             );
         }
@@ -217,20 +217,20 @@ 
     {
         $manager = $jobPoster->manager;
 
-        if ($jobPoster->job_poster_questions === null || $jobPoster->job_poster_questions->count() === 0) {
-            $jobPoster->job_poster_questions()->saveMany($this->populateDefaultQuestions());
-            $jobPoster->refresh();
+        if ($jobPoster->job_poster_questions === null || $jobPoster->job_poster_questions->count () === 0) {
+            $jobPoster->job_poster_questions ()->saveMany ($this->populateDefaultQuestions ());
+            $jobPoster->refresh ();
         }
 
-        return view(
+        return view (
             'manager/job_create',
             [
                 // Localization Strings.
-                'job_l10n' => Lang::get('manager/job_edit'),
+                'job_l10n' => Lang::get ('manager/job_edit'),
                 // Data.
                 'manager' => $manager,
                 'job' => $jobPoster,
-                'form_action_url' => route('admin.jobs.update', $jobPoster),
+                'form_action_url' => route ('admin.jobs.update', $jobPoster),
             ]
         );
     }
@@ -243,21 +243,21 @@ 
      */
     public function createAsManager(Manager $manager)
     {
-        $jobPoster = new JobPoster();
+        $jobPoster = new JobPoster ();
         $jobPoster->manager_id = $manager->id;
 
         // Save manager-specific info to the job poster - equivalent to the intro step of the JPB
-        $divisionEn = $manager->translate('en') !== null ? $manager->translate('en')->division : null;
-        $divisionFr = $manager->translate('fr') !== null ? $manager->translate('fr')->division : null;
-        $jobPoster->fill([
+        $divisionEn = $manager->translate ('en') !== null ? $manager->translate ('en')->division : null;
+        $divisionFr = $manager->translate ('fr') !== null ? $manager->translate ('fr')->division : null;
+        $jobPoster->fill ([
             'department_id' => $manager->department_id,
             'en' => ['division' => $divisionEn],
             'fr' => ['division' => $divisionFr],
         ]);
 
-        $jobPoster->save();
+        $jobPoster->save ();
 
-        return redirect()->route('manager.jobs.edit', $jobPoster->id);
+        return redirect ()->route ('manager.jobs.edit', $jobPoster->id);
     }
 
     /**
@@ -272,19 +272,19 @@ 
     {
         // Don't allow edits for published Job Posters
         // Also check auth while we're at it.
-        $this->authorize('update', $jobPoster);
-        JobPosterValidator::validateUnpublished($jobPoster);
+        $this->authorize ('update', $jobPoster);
+        JobPosterValidator::validateUnpublished ($jobPoster);
 
-        $input = $request->input();
+        $input = $request->input ();
 
         if ($jobPoster->manager_id == null) {
-            $jobPoster->manager_id = $request->user()->manager->id;
-            $jobPoster->save();
+            $jobPoster->manager_id = $request->user ()->manager->id;
+            $jobPoster->save ();
         }
 
-        $this->fillAndSaveJobPosterQuestions($input, $jobPoster, true);
+        $this->fillAndSaveJobPosterQuestions ($input, $jobPoster, true);
 
-        return redirect(route('manager.jobs.show', $jobPoster->id));
+        return redirect (route ('manager.jobs.show', $jobPoster->id));
     }
 
     /**
@@ -298,17 +298,17 @@ 
     protected function fillAndSaveJobPosterQuestions(array $input, JobPoster $jobPoster, bool $replace) : void
     {
         if ($replace) {
-            $jobPoster->job_poster_questions()->delete();
+            $jobPoster->job_poster_questions ()->delete ();
         }
 
-        if (!array_key_exists('question', $input) || !is_array($input['question'])) {
+        if (!array_key_exists ('question', $input) || !is_array ($input['question'])) {
             return;
         }
 
         foreach ($input['question'] as $question) {
-            $jobQuestion = new JobPosterQuestion();
+            $jobQuestion = new JobPosterQuestion ();
             $jobQuestion->job_poster_id = $jobPoster->id;
-            $jobQuestion->fill(
+            $jobQuestion->fill (
                 [
                     'en' => [
                         'question' => $question['question']['en'],
@@ -320,8 +320,8 @@ 
                     ]
                 ]
             );
-            $jobPoster->save();
-            $jobQuestion->save();
+            $jobPoster->save ();
+            $jobQuestion->save ();
         }
     }
 
@@ -333,20 +333,20 @@ 
     protected function populateDefaultQuestions()
     {
         $defaultQuestions = [
-            'en' => array_values(Lang::get('manager/job_create', [], 'en')['questions']),
-            'fr' => array_values(Lang::get('manager/job_create', [], 'fr')['questions']),
+            'en' => array_values (Lang::get ('manager/job_create', [], 'en')['questions']),
+            'fr' => array_values (Lang::get ('manager/job_create', [], 'fr')['questions']),
         ];
 
-        if (count($defaultQuestions['en']) !== count($defaultQuestions['fr'])) {
-            Log::warning('There must be the same number of French and English default questions for a Job Poster.');
+        if (count ($defaultQuestions['en']) !== count ($defaultQuestions['fr'])) {
+            Log::warning ('There must be the same number of French and English default questions for a Job Poster.');
             return;
         }
 
         $jobQuestions = [];
 
-        for ($i = 0; $i < count($defaultQuestions['en']); $i++) {
-            $jobQuestion = new JobPosterQuestion();
-            $jobQuestion->fill(
+        for ($i = 0; $i < count ($defaultQuestions['en']); $i++) {
+            $jobQuestion = new JobPosterQuestion ();
+            $jobQuestion->fill (
                 [
                     'en' => [
                         'question' => $defaultQuestions['en'][$i],

app/Http/Controllers/Api/HrAdvisorController.php

Spacing +1 added lines, -1 removed lines

@@ -15,6 +15,6 @@
      */
     public function show(HrAdvisor $hrAdvisor)
     {
-        return response()->json($hrAdvisor->toArray());
+        return response ()->json ($hrAdvisor->toArray ());
     }
 }

app/Http/Requests/UpdateJobPoster.php

Spacing +12 added lines, -12 removed lines

@@ -24,7 +24,7 @@ 
      */
     public function authorize(): bool
     {
-        $job = $this->route('job');
+        $job = $this->route ('job');
         // Published jobs cannot be updated.
         return $job && $job->published_at === null;
     }
@@ -36,23 +36,23 @@ 
      */
     public function rules(): array
     {
-        $dateFormat = Config::get('app.api_datetime_format');
+        $dateFormat = Config::get ('app.api_datetime_format');
         $dateFormatRule = "date_format:$dateFormat";
         $sliderRule = 'between:1,4';
         return [
-            'chosen_lang' => ['nullable', Rule::in(['en', 'fr'])],
+            'chosen_lang' => ['nullable', Rule::in (['en', 'fr'])],
             'term_qty' => 'nullable|numeric',
             'open_date_time' =>['nullable', $dateFormatRule],
             'close_date_time' => ['nullable', $dateFormatRule],
             'start_date_time' =>['nullable', $dateFormatRule],
-            'department_id' => ['nullable', new ValidIdRule(Department::class)],
-            'province_id' => ['nullable', new ValidIdRule(Province::class)],
-            'security_clearance_id' => ['nullable', new ValidIdRule(SecurityClearance::class)],
-            'language_requirement_id' => ['nullable', new ValidIdRule(LanguageRequirement::class)],
+            'department_id' => ['nullable', new ValidIdRule (Department::class)],
+            'province_id' => ['nullable', new ValidIdRule (Province::class)],
+            'security_clearance_id' => ['nullable', new ValidIdRule (SecurityClearance::class)],
+            'language_requirement_id' => ['nullable', new ValidIdRule (LanguageRequirement::class)],
             'salary_min' => 'nullable|numeric',
             'salary_max' => 'nullable|numeric',
             'noc' => 'nullable|numeric',
-            'classification_id' => ['nullable', new ValidIdRule(Classification::class)],
+            'classification_id' => ['nullable', new ValidIdRule (Classification::class)],
             'classification_level' => 'nullable|numeric',
             'remote_work_allowed' => 'nullable|boolean',
             'team_size' => 'nullable|numeric',
@@ -63,10 +63,10 @@ 
             'experimental_vs_ongoing' => ['nullable', $sliderRule],
             'citizen_facing_vs_back_office' => ['nullable', $sliderRule],
             'collaborative_vs_independent' => ['nullable', $sliderRule],
-            'telework_allowed_frequency_id' => ['nullable', new ValidIdRule(Frequency::class)],
-            'flexible_hours_frequency_id' => ['nullable', new ValidIdRule(Frequency::class)],
-            'travel_requirement_id' => ['nullable', new ValidIdRule(TravelRequirement::class)],
-            'overtime_requirement_id' => ['nullable', new ValidIdRule(OvertimeRequirement::class)],
+            'telework_allowed_frequency_id' => ['nullable', new ValidIdRule (Frequency::class)],
+            'flexible_hours_frequency_id' => ['nullable', new ValidIdRule (Frequency::class)],
+            'travel_requirement_id' => ['nullable', new ValidIdRule (TravelRequirement::class)],
+            'overtime_requirement_id' => ['nullable', new ValidIdRule (OvertimeRequirement::class)],
             'en.city' => 'nullable|string',
             'en.title' => 'nullable|string',
             'en.dept_impact' => 'nullable|string',

app/Http/Middleware/InitializeUser.php

Spacing +21 added lines, -21 removed lines

@@ -22,51 +22,51 @@
      */
     public function handle($request, Closure $next)
     {
-        if (Auth::check()) {
-            $user = Auth::user();
+        if (Auth::check ()) {
+            $user = Auth::user ();
 
             // If running in a local environment, and FORCE_ADMIN is true,
             // automatically set any logged in user to (temporarilly) be an admin
-            if (App::environment() == 'local' && Config::get('app.force_admin')) {
-                $user->setRole('admin');
-                $user->save();
+            if (App::environment () == 'local' && Config::get ('app.force_admin')) {
+                $user->setRole ('admin');
+                $user->save ();
             }
 
             // Ensure the user has a proper profile associated with it
             // If no profile exists yet create one.
             // Admins should be given an applicant and manager profile
-            if ($user->isApplicant() ||
-                    $user->isAdmin() ) {
+            if ($user->isApplicant () ||
+                    $user->isAdmin ()) {
                 $applicantProfile = $user->applicant;
                 if ($applicantProfile === null) {
-                    $applicantProfile = new Applicant();
+                    $applicantProfile = new Applicant ();
                     $applicantProfile->user_id = $user->id;
-                    $applicantProfile->save();
-                    $user->refresh();
+                    $applicantProfile->save ();
+                    $user->refresh ();
                 }
             }
-            if ($user->isManager() ||
-                    $user->isAdmin()) {
+            if ($user->isManager () ||
+                    $user->isAdmin ()) {
                 $managerProfile = $user->manager;
                 if ($managerProfile === null) {
-                    $managerProfile = new Manager();
+                    $managerProfile = new Manager ();
                     $managerProfile->user_id = $user->id;
-                    $managerProfile->save();
-                    $user->refresh();
+                    $managerProfile->save ();
+                    $user->refresh ();
                 }
             }
-            if ($user->isHrAdvisor() ||
-                    $user->isAdmin()) {
+            if ($user->isHrAdvisor () ||
+                    $user->isAdmin ()) {
                 $hrAdvisorProfile = $user->hr_advisor;
                 if ($hrAdvisorProfile === null) {
-                    $hrAdvisorProfile = new HrAdvisor();
+                    $hrAdvisorProfile = new HrAdvisor ();
                     $hrAdvisorProfile->user_id = $user->id;
-                    $hrAdvisorProfile->save();
-                    $user->refresh();
+                    $hrAdvisorProfile->save ();
+                    $user->refresh ();
                 }
             }
         }
 
-        return $next($request);
+        return $next ($request);
     }
 }