GCTC-NTGC / TalentCloud

app/Models/WorkEnvironment.php

Spacing +5 added lines, -5 removed lines

@@ -49,26 +49,26 @@
 
     public function manager()
     {
-        return $this->belongsTo(\App\Models\Manager::class);
+        return $this->belongsTo (\App\Models\Manager::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 workplace_photo_captions() //phpcs:ignore
     {
-        return $this->hasMany(\App\Models\WorkplacePhotoCaption::class);
+        return $this->hasMany (\App\Models\WorkplacePhotoCaption::class);
     }
 
     public function work_environment_translations() //phpcs:ignore
     {
-        return $this->hasMany(\App\Models\WorkEnvironmentTranslation::class);
+        return $this->hasMany (\App\Models\WorkEnvironmentTranslation::class);
     }
 }

app/Models/WorkSample.php

Spacing +9 added lines, -9 removed lines

@@ -25,7 +25,7 @@ 
  * @property \Illuminate\Database\Eloquent\Collection $skill_declarations
  * @property \App\Models\Applicant $applicant
  */
-class WorkSample extends BaseModel {
+class WorkSample extends BaseModel{
 
     protected $casts = [
         'name' => 'string',
@@ -43,20 +43,20 @@ 
         'description'
     ];
 
-    public function file_type() {
-        return $this->belongsTo(\App\Models\Lookup\FileType::class);
+    public function file_type(){
+        return $this->belongsTo (\App\Models\Lookup\FileType::class);
     }
 
-    public function application_work_samples() {
-        return $this->hasMany(\App\Models\ApplicationWorkSample::class);
+    public function application_work_samples(){
+        return $this->hasMany (\App\Models\ApplicationWorkSample::class);
     }
 
-    public function skill_declarations() {
-        return $this->belongsToMany(\App\Models\SkillDeclaration::class);
+    public function skill_declarations(){
+        return $this->belongsToMany (\App\Models\SkillDeclaration::class);
     }
 
-    public function applicant() {
-        return $this->belongsTo(\App\Models\Applicant::class);
+    public function applicant(){
+        return $this->belongsTo (\App\Models\Applicant::class);
     }
 
 }

app/Http/Controllers/AssessmentPlanController.php

Spacing +16 added lines, -16 removed lines

@@ -22,39 +22,39 @@
      */
     public function getForJob(JobPoster $jobPoster)
     {
-        if (Gate::denies('view-assessment-plan', $jobPoster)) {
-            abort(403);
+        if (Gate::denies ('view-assessment-plan', $jobPoster)) {
+            abort (403);
         }
 
-        $criteria = Criteria::where('job_poster_id', $jobPoster->id)->get();
+        $criteria = Criteria::where ('job_poster_id', $jobPoster->id)->get ();
         $criteriaTranslated = [];
         foreach ($criteria as $criterion) {
             // TODO: getTranslationsArray probably makes DB calls every loop. Find a way to profile & optimize.
-            $criteriaTranslated[] = array_merge($criterion->toArray(), $criterion->getTranslationsArray());
+            $criteriaTranslated[] = array_merge ($criterion->toArray (), $criterion->getTranslationsArray ());
         }
-        $criteriaIds = $criteria->pluck('id');
-        $assessments = Assessment::whereIn('criterion_id', $criteriaIds)->get();
+        $criteriaIds = $criteria->pluck ('id');
+        $assessments = Assessment::whereIn ('criterion_id', $criteriaIds)->get ();
         // Check for newly created assessment plan, and initialize any empty criteria to have the
         // "Narrative Review" option set.
-        $assessmentCriteriaIds = $assessments->pluck('criterion_id');
-        $emptyAssessments = array_diff($criteriaIds->toArray(), $assessmentCriteriaIds->toArray());
+        $assessmentCriteriaIds = $assessments->pluck ('criterion_id');
+        $emptyAssessments = array_diff ($criteriaIds->toArray (), $assessmentCriteriaIds->toArray ());
         if (!empty($emptyAssessments)) {
-            $narrativeReview = AssessmentType::where('key', 'narrative_assessment')->first();
+            $narrativeReview = AssessmentType::where ('key', 'narrative_assessment')->first ();
             foreach ($emptyAssessments as $criterionId) {
-                Assessment::create([
+                Assessment::create ([
                     'criterion_id' => $criterionId,
                     'assessment_type_id' => $narrativeReview->id
                 ]);
             }
-            $assessments = Assessment::whereIn('criterion_id', $criteriaIds)->get();
+            $assessments = Assessment::whereIn ('criterion_id', $criteriaIds)->get ();
         }
-        $questions = RatingGuideQuestion::where('job_poster_id', $jobPoster->id)->get();
-        $answers = RatingGuideAnswer::whereIn('rating_guide_question_id', $questions->pluck('id'))->get();
+        $questions = RatingGuideQuestion::where ('job_poster_id', $jobPoster->id)->get ();
+        $answers = RatingGuideAnswer::whereIn ('rating_guide_question_id', $questions->pluck ('id'))->get ();
         return [
             'criteria' => $criteriaTranslated,
-            'assessments' => $assessments->toArray(),
-            'rating_guide_questions' => $questions->toArray(),
-            'rating_guide_answers' => $answers->toArray()
+            'assessments' => $assessments->toArray (),
+            'rating_guide_questions' => $questions->toArray (),
+            'rating_guide_answers' => $answers->toArray ()
         ];
     }
 }

app/Http/Controllers/Admin/JobPosterCrudController.php

Spacing +50 added lines, -50 removed lines

@@ -19,60 +19,60 @@ 
      */
     public function setup() : void
     {
-        $this->crud->setModel('App\Models\JobPoster');
-        $this->crud->setRoute('admin/job-poster');
-        $this->crud->setEntityNameStrings('Job Poster', 'Job Posters');
+        $this->crud->setModel ('App\Models\JobPoster');
+        $this->crud->setRoute ('admin/job-poster');
+        $this->crud->setEntityNameStrings ('Job Poster', 'Job Posters');
 
-        if (!$this->request->has('order')) {
-            $this->crud->orderBy('close_date_time', 'desc');
+        if (!$this->request->has ('order')) {
+            $this->crud->orderBy ('close_date_time', 'desc');
         }
     }
 
     public function setupListOperation()
     {
         // Add the custom blade buttons found in resources/views/vendor/backpack/crud/buttons/.
-        $this->crud->addButtonFromView('line', 'job_admin_edit', 'job_admin_edit', 'end');
-        $this->crud->addButtonFromView('line', 'spb_link', 'spb_link', 'end');
-        $this->crud->addButtonFromView('line', 'jpb_link', 'jpb_link', 'end');
-        $this->crud->addButtonFromView('line', 'job_poster_link', 'job_poster_link', 'end');
+        $this->crud->addButtonFromView ('line', 'job_admin_edit', 'job_admin_edit', 'end');
+        $this->crud->addButtonFromView ('line', 'spb_link', 'spb_link', 'end');
+        $this->crud->addButtonFromView ('line', 'jpb_link', 'jpb_link', 'end');
+        $this->crud->addButtonFromView ('line', 'job_poster_link', 'job_poster_link', 'end');
 
 
-        $this->crud->addColumn([
+        $this->crud->addColumn ([
             'name' => 'id',
             'type' => 'number',
             'label' => 'ID'
         ]);
-        $this->crud->addColumn([
+        $this->crud->addColumn ([
             'name' => 'title',
             'type' => 'text',
             'label' => 'Title'
         ]);
-        $this->crud->addColumn([
+        $this->crud->addColumn ([
             'name' => 'status',
             'label' => 'Status',
             'type' => 'model_function',
             'function_name' => 'status'
         ]);
-        $this->crud->addColumn([
+        $this->crud->addColumn ([
             'name' => 'published',
             'label' => 'Published',
             'type' => 'check',
         ]);
-        $this->crud->addColumn([
+        $this->crud->addColumn ([
             'name' => 'manager_user_name',
             'type' => 'closure',
             'label' => 'Manager',
             'orderable' => false,
-            'function' => function ($entry) {
-                return '<a href="' . route('manager.profile.edit', $entry->manager->user->id) . '" target="_blank">' . $entry->manager->user->full_name . '</a>';
+            'function' => function ($entry){
+                return '<a href="'.route ('manager.profile.edit', $entry->manager->user->id).'" target="_blank">'.$entry->manager->user->full_name.'</a>';
             }
         ]);
-        $this->crud->addColumn([
+        $this->crud->addColumn ([
             'name' => 'department.name',
             'label' => 'Department',
             'type' => 'text'
         ]);
-        $this->crud->addColumn([
+        $this->crud->addColumn ([
             'name' => 'submitted_applications_count',
             'label' => 'Total Applications',
             'type' => 'model_function',
@@ -80,19 +80,19 @@ 
         ]);
 
         // Filters.
-        $this->crud->addFilter([
+        $this->crud->addFilter ([
             'name' => 'departments',
             'type' => 'select2_multiple',
             'label' => 'Departments'
-        ], function () {
-            return Department::all()->pluck('name', 'id')->toArray();
-        }, function ($values) {
-            $this->crud->addClause('WhereHas', 'department', function ($query) use ($values) {
-                foreach (json_decode($values) as $key => $value) {
+        ], function (){
+            return Department::all ()->pluck ('name', 'id')->toArray ();
+        }, function ($values){
+            $this->crud->addClause ('WhereHas', 'department', function ($query) use ($values) {
+                foreach (json_decode ($values) as $key => $value) {
                     if ($key === 0) {
-                        $query->where('id', $value);
+                        $query->where ('id', $value);
                     } else {
-                        $query->orWhere('id', $value);
+                        $query->orWhere ('id', $value);
                     }
                 }
             });
@@ -101,7 +101,7 @@ 
 
     public function setupUpdateOperation()
     {
-        $this->crud->addField([
+        $this->crud->addField ([
             'name' => 'title',
             'label' => 'Title',
             'type' => 'text',
@@ -109,22 +109,22 @@ 
                 'readonly' => 'readonly'
             ]
         ]);
-        $this->crud->addField([
+        $this->crud->addField ([
             'name' => 'salary_min',
             'type' => 'number',
             'label' => 'Minimum Salary',
         ]);
-        $this->crud->addField([
+        $this->crud->addField ([
             'name' => 'salary_max',
             'type' => 'number',
             'label' => 'Maximum Salary',
         ]);
-        $this->crud->addField([
+        $this->crud->addField ([
             'name' => 'noc',
             'type' => 'number',
             'label' => 'NOC Code',
         ]);
-        $this->crud->addField([
+        $this->crud->addField ([
             'name' => 'open_date_time',
             'label' => 'Open Date',
             'type' => 'date_picker',
@@ -133,7 +133,7 @@ 
                 'format' => 'yyyy-mm-dd',
             ],
         ]);
-        $this->crud->addField([
+        $this->crud->addField ([
             'name' => 'close_date_time',
             'label' => 'Close Date',
             'type' => 'date_picker',
@@ -142,7 +142,7 @@ 
                 'format' => 'yyyy-mm-dd',
             ],
         ]);
-        $this->crud->addField([
+        $this->crud->addField ([
             'name' => 'start_date_time',
             'label' => 'Start Date',
             'type' => 'date_picker',
@@ -151,17 +151,17 @@ 
                 'format' => 'yyyy-mm-dd',
             ],
         ]);
-        $this->crud->addField([
+        $this->crud->addField ([
             'name' => 'process_number',
             'type' => 'number',
             'label' => 'Process #',
         ]);
-        $this->crud->addField([
+        $this->crud->addField ([
             'name' => 'priority_clearance_number',
             'type' => 'number',
             'label' => 'Priority Clearance #',
         ]);
-        $this->crud->addField([
+        $this->crud->addField ([
             'name' => 'loo_issuance_date',
             'type' => 'date_picker',
             'label' => 'Letter of Offer Issuance Date',
@@ -170,10 +170,10 @@ 
                'format' => 'yyyy-mm-dd',
             ],
         ]);
-        if ($this->crud->getCurrentEntry() &&
-            !$this->crud->getCurrentEntry()->published
+        if ($this->crud->getCurrentEntry () &&
+            !$this->crud->getCurrentEntry ()->published
         ) {
-            $this->crud->addField([
+            $this->crud->addField ([
                 'name' => 'published',
                 'label' => 'Publish',
                 'type' => 'checkbox'
@@ -183,20 +183,20 @@ 
 
     public function update()
     {
-        $open_date = $this->crud->request->request->get('open_date_time');
-        $close_date = $this->crud->request->request->get('close_date_time');
-        $start_date = $this->crud->request->request->get('start_date_time');
-        $this->crud->request->request->remove('open_date_time');
-        $this->crud->request->request->remove('close_date_time');
-        $this->crud->request->request->remove('start_date_time');
+        $open_date = $this->crud->request->request->get ('open_date_time');
+        $close_date = $this->crud->request->request->get ('close_date_time');
+        $start_date = $this->crud->request->request->get ('start_date_time');
+        $this->crud->request->request->remove ('open_date_time');
+        $this->crud->request->request->remove ('close_date_time');
+        $this->crud->request->request->remove ('start_date_time');
         // Manipulates the input fields to save the "end of day" timestamp for
         // open/close/start dates.
-        $this->crud->request->request->add([
-            'open_date_time' => ptDayStartToUtcTime($open_date),
-            'close_date_time' => ptDayEndToUtcTime($close_date),
-            'start_date_time' => ptDayStartToUtcTime($start_date),
+        $this->crud->request->request->add ([
+            'open_date_time' => ptDayStartToUtcTime ($open_date),
+            'close_date_time' => ptDayEndToUtcTime ($close_date),
+            'start_date_time' => ptDayStartToUtcTime ($start_date),
         ]);
-        $response = $this->traitUpdate();
+        $response = $this->traitUpdate ();
 
         return $response;
     }

app/Http/Controllers/Admin/ManagerCrudController.php

Spacing +13 added lines, -13 removed lines

@@ -17,49 +17,49 @@ 
      */
     public function setup() : void
     {
-        $this->crud->setModel('App\Models\User');
-        $this->crud->setRoute('admin/manager');
-        $this->crud->setEntityNameStrings('manager', 'managers');
+        $this->crud->setModel ('App\Models\User');
+        $this->crud->setRoute ('admin/manager');
+        $this->crud->setEntityNameStrings ('manager', 'managers');
 
         // Don't show 'basic' users.
-        $this->crud->addClause('whereIn', 'user_role_id', [2, 3]);
+        $this->crud->addClause ('whereIn', 'user_role_id', [2, 3]);
     }
 
     public function setupListOperation()
     {
-        $this->crud->removeButton('update');
+        $this->crud->removeButton ('update');
 
-        $this->crud->addColumn([
+        $this->crud->addColumn ([
             'name' => 'manager.id',
             'key' => 'manager_id',
             'type' => 'number',
             'label' => 'ID'
         ]);
-        $this->crud->addColumn([
+        $this->crud->addColumn ([
             'name' => 'manager.name',
             'key' => 'manager_name',
             'type' => 'text',
             'label' => 'Name'
         ]);
-        $this->crud->addColumn([
+        $this->crud->addColumn ([
             'name' => 'user_role.name',
             'type' => 'text',
             'key' => 'user_role_name',
             'label' => 'Role'
         ]);
-        $this->crud->addColumn([
+        $this->crud->addColumn ([
             'name' => 'email',
             'key' => 'manager_email',
             'type' => 'text',
             'label' => 'Email'
         ]);
-        $this->crud->addColumn([
+        $this->crud->addColumn ([
             'name' => 'gov_email',
             'key' => 'government_email',
             'type' => 'text',
             'label' => 'Government Email'
         ]);
-        $this->crud->addColumn([
+        $this->crud->addColumn ([
             'name' => 'manager.department.name',
             'key' => 'manager_department',
             'type' => 'text',
@@ -68,7 +68,7 @@ 
         ]);
 
         // Add the custom blade button found in resources/views/vendor/backpack/crud/buttons/profile_edit.blade.php.
-        $this->crud->addButtonFromView('line', 'create_job_poster', 'create_job_poster', 'beginning');
-        $this->crud->addButtonFromView('line', 'profile_edit', 'profile_edit', 'end');
+        $this->crud->addButtonFromView ('line', 'create_job_poster', 'create_job_poster', 'beginning');
+        $this->crud->addButtonFromView ('line', 'profile_edit', 'profile_edit', 'end');
     }
 }

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/ManagerProfileController.php

Spacing +28 added lines, -28 removed lines

@@ -26,7 +26,7 @@ 
      */
     public function show(Request $request, Manager $manager)
     {
-        $manager_profile = Lang::get('applicant/manager_profile');
+        $manager_profile = Lang::get ('applicant/manager_profile');
 
         $manager_profile_sections = [
             [
@@ -65,9 +65,9 @@ 
             ]
         ];
 
-        return view('applicant/manager', [
+        return view ('applicant/manager', [
             'manager_profile' => $manager_profile,
-            'urls' => Lang::get('common/urls'),
+            'urls' => Lang::get ('common/urls'),
             'manager' => $manager,
             'manager_profile_photo_url' => '/images/user.png', // TODO get real photo.
             'manager_profile_sections' => $manager_profile_sections,
@@ -83,8 +83,8 @@ 
      */
     public function editAuthenticated(Request $request)
     {
-        $manager = $request->user()->manager;
-        return redirect(route('manager.profile.edit', $manager));
+        $manager = $request->user ()->manager;
+        return redirect (route ('manager.profile.edit', $manager));
     }
 
     /**
@@ -99,25 +99,25 @@ 
         // TODO: Improve workplace photos, and reference them in template direction from WorkEnvironment model.
         $workplacePhotos = [];
 
-        $frequencies = Frequency::all();
+        $frequencies = Frequency::all ();
         $linkedInUrlPattern = LinkedInUrlRule::PATTERN;
         $twitterHandlePattern = TwitterHandleRule::PATTERN;
 
-        return view('manager/profile', [
+        return view ('manager/profile', [
             // Localization.
-            'profile_l10n' => Lang::get('manager/profile'),
+            'profile_l10n' => Lang::get ('manager/profile'),
             // Data.
-            'urls' => Lang::get('common/urls'),
+            'urls' => Lang::get ('common/urls'),
             'user' => $manager->user,
             'manager' => $manager,
             'manager_profile_photo_url' => '/images/user.png', // TODO get real photo.
             'workplace_photos' => $workplacePhotos,
-            'departments' => Department::all(),
+            'departments' => Department::all (),
             'telework_options' => $frequencies,
             'flex_hour_options' => $frequencies,
             'radio_options' => $frequencies,
-            'managerEN' => $manager->translate('en'),
-            'managerFR' => $manager->translate('fr'),
+            'managerEN' => $manager->translate ('en'),
+            'managerFR' => $manager->translate ('fr'),
             'linkedInUrlPattern' => $linkedInUrlPattern,
             'twitterHandlePattern' => $twitterHandlePattern,
         ]);
@@ -134,27 +134,27 @@ 
     {
         // TODO: save workplace Photos.
         // TODO: remove control of name in production.
-        $input = $request->input();
+        $input = $request->input ();
 
         // redirect to error messages element if validation fails
-        if (isset($request->validator) && $request->validator->fails()) {
+        if (isset($request->validator) && $request->validator->fails ()) {
             $hash = '#managerProfileFormErrors';
-            return redirect(route('manager.profile.edit', $manager).$hash)
-                        ->withErrors($request->validator)
-                        ->withInput();
+            return redirect (route ('manager.profile.edit', $manager).$hash)
+                        ->withErrors ($request->validator)
+                        ->withInput ();
         }
 
-        $validated = $request->validated();
+        $validated = $request->validated ();
 
         $user = $manager->user;
-        $user->fill($validated);
+        $user->fill ($validated);
         if (!empty($input['new_password'])) {
-            $user->password = Hash::make($input['new_password']);
+            $user->password = Hash::make ($input['new_password']);
         }
-        $user->save();
+        $user->save ();
 
-        $manager->fill($validated);
-        $manager->save();
+        $manager->fill ($validated);
+        $manager->save ();
 
         // Use the button that was clicked to decide which element to redirect to.
         switch ($input['submit']) {
@@ -172,18 +172,18 @@ 
                 break;
         }
 
-        return redirect(route('manager.profile.edit', $manager).$hash);
+        return redirect (route ('manager.profile.edit', $manager).$hash);
     }
 
     public function faq(Request $request)
     {
-        $show_demo_notification = $request->user() && $request->user()->isDemoManager();
+        $show_demo_notification = $request->user () && $request->user ()->isDemoManager ();
 
-        return view(
+        return view (
             'applicant/static_faq',
             [
-                'breadcrumb_home' => route('manager.home'),
-                'faq' => Lang::get('applicant/faq'),
+                'breadcrumb_home' => route ('manager.home'),
+                'faq' => Lang::get ('applicant/faq'),
                 'manager_sidebar_active' => 'active',
                 'show_demo_notification' => $show_demo_notification,
             ]

app/Http/Controllers/Api/JobApiController.php

Spacing +25 added lines, -25 removed lines

@@ -22,7 +22,7 @@ 
     public function __construct()
     {
         // This applies the appropriate policy to each resource route.
-        $this->authorizeResource(JobPoster::class, 'job');
+        $this->authorizeResource (JobPoster::class, 'job');
     }
 
     /**
@@ -35,14 +35,14 @@ 
      */
     private function jobToArray(JobPoster $job)
     {
-        $criteria = Criteria::where('job_poster_id', $job->id)->get();
+        $criteria = Criteria::where ('job_poster_id', $job->id)->get ();
 
-        $toApiArray = function ($model) {
-            return array_merge($model->toArray(), $model->getTranslationsArray());
+        $toApiArray = function ($model){
+            return array_merge ($model->toArray (), $model->getTranslationsArray ());
         };
-        $criteriaTranslated = $criteria->map($toApiArray);
+        $criteriaTranslated = $criteria->map ($toApiArray);
 
-        $jobArray = array_merge($job->toApiArray(), ['criteria' => $criteriaTranslated]);
+        $jobArray = array_merge ($job->toApiArray (), ['criteria' => $criteriaTranslated]);
         return $jobArray;
     }
     /**
@@ -63,14 +63,14 @@ 
      */
     public function store(StoreJobPoster $request)
     {
-        $data = $request->validated();
-        $job = new JobPoster();
-        $job->manager_id = $request->user()->manager->id;
+        $data = $request->validated ();
+        $job = new JobPoster ();
+        $job->manager_id = $request->user ()->manager->id;
         // Defaulting JPB created Jobs to monthly terms for now.
-        $job->job_term_id = JobTerm::where('name', 'month')->value('id');
-        $job->fill($data);
-        $job->save();
-        return response()->json($this->jobToArray($job));
+        $job->job_term_id = JobTerm::where ('name', 'month')->value ('id');
+        $job->fill ($data);
+        $job->save ();
+        return response ()->json ($this->jobToArray ($job));
     }
 
     /**
@@ -81,7 +81,7 @@ 
      */
     public function show(JobPoster $job)
     {
-        return response()->json($this->jobToArray($job));
+        return response ()->json ($this->jobToArray ($job));
     }
 
     /**
@@ -93,14 +93,14 @@ 
      */
     public function update(UpdateJobPoster $request, JobPoster $job)
     {
-        $data = $request->validated();
+        $data = $request->validated ();
         // Only values both in the JobPoster->fillable array,
         // and returned by UpdateJobPoster->validatedData(), will be set.
-        $job->fill($data);
+        $job->fill ($data);
         // Defaulting JPB updated jobs to monthly for now.
-        $job->job_term_id = JobTerm::where('name', 'month')->value('id');
-        $job->save();
-        return response()->json($this->jobToArray($job->fresh()));
+        $job->job_term_id = JobTerm::where ('name', 'month')->value ('id');
+        $job->save ();
+        return response ()->json ($this->jobToArray ($job->fresh ()));
     }
 
     /**
@@ -125,18 +125,18 @@ 
         // Check to avoid submitting for review multiple times.
         if ($job->review_requested_at === null) {
             // Update review request timestamp.
-            $job->review_requested_at = new Date();
-            $job->save();
+            $job->review_requested_at = new Date ();
+            $job->save ();
 
             // Send email.
-            $reviewer_email = config('mail.reviewer_email');
+            $reviewer_email = config ('mail.reviewer_email');
             if (isset($reviewer_email)) {
-                Mail::to($reviewer_email)->send(new JobPosterReviewRequested($job, Auth::user()));
+                Mail::to ($reviewer_email)->send (new JobPosterReviewRequested ($job, Auth::user ()));
             } else {
-                Log::error('The reviewer email environment variable is not set.');
+                Log::error ('The reviewer email environment variable is not set.');
             }
         }
 
-        return response()->json($this->jobToArray($job));
+        return response ()->json ($this->jobToArray ($job));
     }
 }

app/Http/Controllers/Api/JobTaskController.php

Spacing +15 added lines, -15 removed lines

@@ -17,14 +17,14 @@ 
      */
     public function toApiArray(JobPosterKeyTask $model)
     {
-        return array_merge($model->toArray(), $model->getTranslationsArray());
+        return array_merge ($model->toArray (), $model->getTranslationsArray ());
     }
 
     public function indexByJob(JobPoster $jobPoster)
     {
         $toApiArray = array($this, 'toApiArray');
-        $taskArray = JobPosterKeyTask::where('job_poster_id', $jobPoster->id)->get()->map($toApiArray);
-        return response()->json($taskArray);
+        $taskArray = JobPosterKeyTask::where ('job_poster_id', $jobPoster->id)->get ()->map ($toApiArray);
+        return response ()->json ($taskArray);
     }
 
     /**
@@ -38,35 +38,35 @@ 
     {
         $toApiArray = array($this, 'toApiArray');
 
-        $newTasks = collect($request->validated()); // Collection of JobPosterKeyTasks.
+        $newTasks = collect ($request->validated ()); // Collection of JobPosterKeyTasks.
         $oldTasks = $jobPoster->job_poster_key_tasks;
 
         $savedNewTaskIds = [];
 
         // First, delete old tasks that weren't resubmitted, and update those that were.
         foreach ($oldTasks as $task) {
-            $newTask = $newTasks->firstWhere('id', $task['id']);
+            $newTask = $newTasks->firstWhere ('id', $task['id']);
             if ($newTask) {
                 $savedNewTaskIds[] = $newTask['id'];
-                $task->fill(collect($newTask)->only(['en', 'fr'])->toArray());
-                $task->save();
+                $task->fill (collect ($newTask)->only (['en', 'fr'])->toArray ());
+                $task->save ();
             } else {
-                $task->delete();
+                $task->delete ();
             }
         }
 
         // Now, save any new tasks that remain.
         foreach ($newTasks as $task) {
-            if ($this->isUnsaved($task, $savedNewTaskIds)) {
-                $jobPosterTask = new JobPosterKeyTask();
+            if ($this->isUnsaved ($task, $savedNewTaskIds)) {
+                $jobPosterTask = new JobPosterKeyTask ();
                 $jobPosterTask->job_poster_id = $jobPoster->id;
-                $jobPosterTask->fill(collect($task)->only(['en', 'fr'])->toArray());
-                $jobPosterTask->save();
+                $jobPosterTask->fill (collect ($task)->only (['en', 'fr'])->toArray ());
+                $jobPosterTask->save ();
             }
         }
 
-        $taskArray = $jobPoster->fresh()->job_poster_key_tasks->map($toApiArray);
-        return response()->json($taskArray);
+        $taskArray = $jobPoster->fresh ()->job_poster_key_tasks->map ($toApiArray);
+        return response ()->json ($taskArray);
     }
 
     /**
@@ -78,6 +78,6 @@ 
      */
     private function isUnsaved($task, array $savedTaskIds): bool
     {
-        return !array_key_exists('id', $task) || !in_array($task['id'], $savedTaskIds);
+        return !array_key_exists ('id', $task) || !in_array ($task['id'], $savedTaskIds);
     }
 }