GCTC-NTGC / TalentCloud

app/Providers/AuthServiceProvider.php

Spacing +5 added lines, -5 removed lines

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

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;
     }
 }