GCTC-NTGC / TalentCloud

app/Models/Lookup/CriteriaType.php

Spacing +2 added lines, -2 removed lines

@@ -35,11 +35,11 @@
 
     public function criteria()
     {
-        return $this->hasMany(\App\Models\Criteria::class);
+        return $this->hasMany (\App\Models\Criteria::class);
     }
 
     public function criteria_type_translations() //phpcs:ignore
     {
-        return $this->hasMany(\App\Models\CriteriaTypeTranslation::class);
+        return $this->hasMany (\App\Models\CriteriaTypeTranslation::class);
     }
 }

app/Models/Lookup/ExperienceLevelTranslation.php

Spacing +3 added lines, -3 removed lines

@@ -21,15 +21,15 @@
  * 
  * @property \App\Models\Lookup\ExperienceLevel $experience_level
  */
-class ExperienceLevelTranslation extends BaseModel {
+class ExperienceLevelTranslation extends BaseModel{
 
     protected $casts = [
         'experience_level_id' => 'int'
     ];
     protected $fillable = [];
 
-    public function experience_level() {
-        return $this->belongsTo(\App\Models\Lookup\ExperienceLevel::class);
+    public function experience_level(){
+        return $this->belongsTo (\App\Models\Lookup\ExperienceLevel::class);
     }
 
 }

app/Models/JobPosterTranslation.php

Spacing +1 added lines, -1 removed lines

@@ -61,6 +61,6 @@
 
     public function job_poster()
     {
-        return $this->belongsTo(\App\Models\JobPoster::class);
+        return $this->belongsTo (\App\Models\JobPoster::class);
     }
 }

app/Models/ManagerTranslation.php

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
  *
  * @property \App\Models\Manager $manager
  */
-class ManagerTranslation extends BaseModel {
+class ManagerTranslation extends BaseModel{
 
     protected $casts = [
         'manager_id' => 'int'
@@ -46,8 +46,8 @@ 
         'learning_path'
     ];
 
-    public function manager() {
-        return $this->belongsTo(\App\Models\Manager::class);
+    public function manager(){
+        return $this->belongsTo (\App\Models\Manager::class);
     }
 
 }

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/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],