GCTC-NTGC / TalentCloud

app/Http/Controllers/Auth/AuthController.php

Spacing +12 added lines, -12 removed lines

@@ -12,27 +12,27 @@
      *
      * @return mixed[]
      */
-    protected function auth_routes()
+    protected function auth_routes ()
     {
-        if (WhichPortal::isManagerPortal()) {
+        if (WhichPortal::isManagerPortal ()) {
             $routes = [
-                'home' => route('manager.home'),
-                'login' => route('manager.login'),
-                'register' => route('manager.register'),
+                'home' => route ('manager.home'),
+                'login' => route ('manager.login'),
+                'register' => route ('manager.register'),
                 'password' => [
-                    'email' => route('manager.password.email'),
-                    'request' => route('manager.password.request'),
+                    'email' => route ('manager.password.email'),
+                    'request' => route ('manager.password.request'),
                 ],
                 // 'passwords.reset' => route('manager.password.reset'),
             ];
         } else {
             $routes = [
-                'home' => route('home'),
-                'login' => route('login'),
-                'register' => route('register'),
+                'home' => route ('home'),
+                'login' => route ('login'),
+                'register' => route ('register'),
                 'password' => [
-                    'email' => route('password.email'),
-                    'request' => route('password.request'),
+                    'email' => route ('password.email'),
+                    'request' => route ('password.request'),
                 ],
                 // 'passwords.reset' => route('password.reset'),
             ];

app/Http/Controllers/Auth/FirstVisitController.php

Spacing +22 added lines, -22 removed lines

@@ -17,18 +17,18 @@ 
      * @param Request $request
      * @return \Illuminate\Http\Response
      */
-    public function showFirstVisitManagerForm()
+    public function showFirstVisitManagerForm ()
     {
         $routes = [
-            'return' => route('home'),
-            'continue' => route('manager.finish_registration'),
+            'return' => route ('home'),
+            'continue' => route ('manager.finish_registration'),
         ];
 
-        return view('auth.first_visit_manager', [
+        return view ('auth.first_visit_manager', [
             'routes' => $routes,
-            'first_visit' => Lang::get('common/auth/first_manager_visit'),
-            'departments' => Department::all(),
-            'not_in_gov_option' => ['value' => 0, 'name' => Lang::get('common/auth/register.not_in_gov')],
+            'first_visit' => Lang::get ('common/auth/first_manager_visit'),
+            'departments' => Department::all (),
+            'not_in_gov_option' => ['value' => 0, 'name' => Lang::get ('common/auth/register.not_in_gov')],
         ]);
     }
 
@@ -38,36 +38,36 @@ 
      * @param Request $request
      * @return void
      */
-    public function finishManagerRegistration(Request $request)
+    public function finishManagerRegistration (Request $request)
     {
-        $data = $request->all();
-        $validator = RegistrationValidator::finalizeManagerValidator($data);
-        $validator->validate();
+        $data = $request->all ();
+        $validator = RegistrationValidator::finalizeManagerValidator ($data);
+        $validator->validate ();
 
-        $user = $request->user();
+        $user = $request->user ();
 
         // Save manager specific fields to user
-        $managerDepartment = Department::find($data['department']);
+        $managerDepartment = Department::find ($data['department']);
         $inGovernment = ($managerDepartment !== null);
         $user->not_in_gov = !$inGovernment;
         $user->gov_email = $inGovernment ? $data['gov_email'] : null;
-        $user->save();
-        $user->refresh();
+        $user->save ();
+        $user->refresh ();
 
         // Add (or update) manager profile
         // NOTE: modifying a field in $user, and saving it, appears to create Manager object. I don't know how. -- Tristan
         // That means that after setting not_in_gov or gov_email, a manager already exists here. Adding a new one will throw an exception.
         $department_id = $inGovernment ? $managerDepartment->id : null;
         if ($user->manager === null) {
-            $user->applicant()->save(new Manager());
-            $user->refresh();
+            $user->applicant ()->save (new Manager ());
+            $user->refresh ();
         }
         $user->manager->department_id = $department_id;
-        $user->manager->save();
+        $user->manager->save ();
 
-        $user->refresh();
-        $expectedUrl = session()->remove('url.expected');
-        session()->remove('url.expected');
-        return redirect($expectedUrl);
+        $user->refresh ();
+        $expectedUrl = session ()->remove ('url.expected');
+        session ()->remove ('url.expected');
+        return redirect ($expectedUrl);
     }
 }

app/Http/Controllers/Auth/RegisterController.php

Spacing +44 added lines, -44 removed lines

@@ -35,9 +35,9 @@ 
      *
      * @return string
      */
-    protected function redirectTo()
+    protected function redirectTo ()
     {
-        $redirectTo = WhichPortal::isManagerPortal() ? route('manager.home') : route('home');
+        $redirectTo = WhichPortal::isManagerPortal () ? route('manager.home') : route ('home');
         return $redirectTo;
     }
 
@@ -46,9 +46,9 @@ 
      *
      * @return void
      */
-    public function __construct()
+    public function __construct ()
     {
-        $this->middleware('guest');
+        $this->middleware ('guest');
     }
 
     /**
@@ -56,12 +56,12 @@ 
      *
      * @return \Illuminate\Http\Response
      */
-    public function showRegistrationForm()
+    public function showRegistrationForm ()
     {
-        return view('auth.register', [
-            'routes' => $this->auth_routes(),
-            'register' => Lang::get('common/auth/register'),
-            'home_url' => route('home'),
+        return view ('auth.register', [
+            'routes' => $this->auth_routes (),
+            'register' => Lang::get ('common/auth/register'),
+            'home_url' => route ('home'),
         ]);
     }
 
@@ -70,14 +70,14 @@ 
      *
      * @return \Illuminate\Http\Response
      */
-    public function showManagerRegistrationForm()
+    public function showManagerRegistrationForm ()
     {
-        return view('auth.register_manager', [
-            'routes' => $this->auth_routes(),
-            'register' => Lang::get('common/auth/register'),
-            'not_in_gov_option' => ['value' => 0, 'name' => Lang::get('common/auth/register.not_in_gov')],
-            'departments' => Department::all(),
-            'home_url' => route('manager.home'),
+        return view ('auth.register_manager', [
+            'routes' => $this->auth_routes (),
+            'register' => Lang::get ('common/auth/register'),
+            'not_in_gov_option' => ['value' => 0, 'name' => Lang::get ('common/auth/register.not_in_gov')],
+            'departments' => Department::all (),
+            'home_url' => route ('manager.home'),
         ]);
     }
 
@@ -88,9 +88,9 @@ 
      * @param  array $data Incoming registration data.
      * @return \Illuminate\Contracts\Validation\Validator
      */
-    protected function validator(array $data)
+    protected function validator (array $data)
     {
-        return RegistrationValidator::userValidator($data);
+        return RegistrationValidator::userValidator ($data);
     }
 
     /**
@@ -99,9 +99,9 @@ 
      * @param  array $data Incoming registration data.
      * @return \Illuminate\Contracts\Validation\Validator
      */
-    protected function managerValidator(array $data)
+    protected function managerValidator (array $data)
     {
-        return RegistrationValidator::managerValidator($data);
+        return RegistrationValidator::managerValidator ($data);
     }
 
     /**
@@ -110,22 +110,22 @@ 
      * @param  array $data Incoming User data.
      * @return \App\Models\User
      */
-    protected function create(array $data)
+    protected function create (array $data)
     {
-        $user = new User();
+        $user = new User ();
         $user->first_name = $data['first_name'];
         $user->last_name = $data['last_name'];
         $user->email = $data['email'];
-        $user->password = Hash::make($data['password']);
+        $user->password = Hash::make ($data['password']);
 
         // Default to basic user.
-        $user->setRole('basic');
+        $user->setRole ('basic');
 
-        $user->save();
+        $user->save ();
 
-        $user->applicant()->save(new Applicant());
+        $user->applicant ()->save (new Applicant ());
 
-        return $user->fresh();
+        return $user->fresh ();
     }
 
     /**
@@ -134,31 +134,31 @@ 
      * @param  array $data Incoming User data.
      * @return \App\Models\User
      */
-    protected function createManager(array $data)
+    protected function createManager (array $data)
     {
         // Create basic user.
-        $user = $this->create($data);
+        $user = $this->create ($data);
 
         // Save manager specific fields.
-        $managerDepartment = Department::find($data['department']);
+        $managerDepartment = Department::find ($data['department']);
         $inGovernment = ($managerDepartment !== null);
         $user->not_in_gov = !$inGovernment;
         $user->gov_email = $inGovernment ? $data['gov_email'] : null;
-        $user->save();
-        $user->refresh();
+        $user->save ();
+        $user->refresh ();
 
         // Add (or update) manager profile.
         // NOTE: modifying a field in $user, and saving it, appears to create Manager object. I don't know how. -- Tristan
         // That means that after setting not_in_gov or gov_email, a manager already exists here. Adding a new one will throw an exception.
         $department_id = $inGovernment ? $managerDepartment->id : null;
         if ($user->manager === null) {
-            $user->applicant()->save(new Manager());
-            $user->refresh();
+            $user->applicant ()->save (new Manager ());
+            $user->refresh ();
         }
         $user->manager->department_id = $department_id;
-        $user->manager->save();
+        $user->manager->save ();
 
-        return $user->fresh();
+        return $user->fresh ();
     }
 
     /**
@@ -169,9 +169,9 @@ 
      * @param  mixed                    $user    Incoming User data.
      * @return \Illuminate\Http\Response
      */
-    protected function registered(Request $request, $user)
+    protected function registered (Request $request, $user)
     {
-        return redirect()->intended($this->redirectTo());
+        return redirect ()->intended ($this->redirectTo ());
     }
 
     /**
@@ -181,15 +181,15 @@ 
      * @param  \Illuminate\Http\Request  $request
      * @return \Illuminate\Http\Response
      */
-    public function registerManager(Request $request)
+    public function registerManager (Request $request)
     {
-        $this->managerValidator($request->all())->validate();
+        $this->managerValidator ($request->all ())->validate ();
 
-        event(new Registered($user = $this->createManager($request->all())));
+        event (new Registered ($user = $this->createManager ($request->all ())));
 
-        $this->guard()->login($user);
+        $this->guard ()->login ($user);
 
-        return $this->registered($request, $user)
-            ?: redirect($this->redirectPath());
+        return $this->registered ($request, $user)
+            ?: redirect ($this->redirectPath ());
     }
 }

app/Http/Controllers/Auth/LoginController.php

Spacing +13 added lines, -13 removed lines

@@ -42,9 +42,9 @@ 
      *
      * @return string
      */
-    protected function redirectTo()
+    protected function redirectTo ()
     {
-        $redirectTo = WhichPortal::isManagerPortal() ? route('manager.home') : route('home');
+        $redirectTo = WhichPortal::isManagerPortal () ? route('manager.home') : route ('home');
         return $redirectTo;
     }
 
@@ -53,9 +53,9 @@ 
      *
      * @return void
      */
-    public function __construct()
+    public function __construct ()
     {
-        $this->middleware('guest')->except('logout');
+        $this->middleware ('guest')->except ('logout');
     }
 
     /**
@@ -64,13 +64,13 @@ 
      *
      * @return \Illuminate\Http\Response
      */
-    public function showLoginForm()
+    public function showLoginForm ()
     {
-        $home_url = WhichPortal::isManagerPortal() ? route('manager.home') : route('home');
+        $home_url = WhichPortal::isManagerPortal () ? route('manager.home') : route ('home');
 
-        return view('auth/login', [
-            'routes' => $this->auth_routes(),
-            'login' => Lang::get('common/auth/login'),
+        return view ('auth/login', [
+            'routes' => $this->auth_routes (),
+            'login' => Lang::get ('common/auth/login'),
             'home_url' => $home_url,
         ]);
     }
@@ -82,13 +82,13 @@ 
      * @param  \Illuminate\Http\Request $request Incoming Request object.
      * @return \Illuminate\Http\Response
      */
-    public function logout(Request $request)
+    public function logout (Request $request)
     {
-        $this->guard()->logout();
+        $this->guard ()->logout ();
 
-        $request->session()->invalidate();
+        $request->session ()->invalidate ();
 
         // This causes logout to redirect to the same page as login.
-        return redirect($this->redirectPath());
+        return redirect ($this->redirectPath ());
     }
 }

app/Http/Controllers/Auth/ResetPasswordController.php

Spacing +10 added lines, -10 removed lines

@@ -29,9 +29,9 @@ 
      *
      * @return string
      */
-    protected function redirectTo()
+    protected function redirectTo ()
     {
-        $redirectTo = WhichPortal::isManagerPortal() ? route('manager.home') : route('home');
+        $redirectTo = WhichPortal::isManagerPortal () ? route('manager.home') : route ('home');
         return $redirectTo;
     }
 
@@ -40,9 +40,9 @@ 
      *
      * @return void
      */
-    public function __construct()
+    public function __construct ()
     {
-        $this->middleware('guest');
+        $this->middleware ('guest');
     }
 
     /**
@@ -54,13 +54,13 @@ 
      * @param  string|null              $token   Validation token.
      * @return \Illuminate\Http\Response
      */
-    public function showResetForm(Request $request, $token = null)
+    public function showResetForm (Request $request, $token = null)
     {
-        return view('auth.passwords.reset')->with([
+        return view ('auth.passwords.reset')->with ([
             'token' => $token,
             'email' => $request->email,
-            'routes' => $this->auth_routes(),
-            'reset_password' => Lang::get('common/auth/reset_password'),
+            'routes' => $this->auth_routes (),
+            'reset_password' => Lang::get ('common/auth/reset_password'),
         ]);
     }
 
@@ -70,7 +70,7 @@ 
      *
      * @return array
      */
-    protected function rules()
+    protected function rules ()
     {
         return [
             'token' => 'required',
@@ -90,7 +90,7 @@ 
      *
      * @return array
      */
-    protected function validationErrorMessages()
+    protected function validationErrorMessages ()
     {
         return [];
     }

app/Http/Controllers/RatingGuideAnswerController.php

Spacing +17 added lines, -17 removed lines

@@ -15,17 +15,17 @@ 
      * @throws \InvalidArgumentException For missing $expected_answer.
      * @return mixed
      */
-    public function store(StoreRatingGuideAnswer $request)
+    public function store (StoreRatingGuideAnswer $request)
     {
         // Authorization handled by the FormRequest.
         // Data validation handled by this line.
-        $data = $request->validated();
-        $ratingGuideAnswer = new RatingGuideAnswer($data);
-        $ratingGuideAnswer->save();
+        $data = $request->validated ();
+        $ratingGuideAnswer = new RatingGuideAnswer ($data);
+        $ratingGuideAnswer->save ();
 
         return [
             'success' => "Successfully created rating guide answer $ratingGuideAnswer->id",
-            'rating_guide_answer' => $ratingGuideAnswer->toArray(),
+            'rating_guide_answer' => $ratingGuideAnswer->toArray (),
         ];
     }
 
@@ -35,10 +35,10 @@ 
      * @param  \App\Models\RatingGuideAnswer $ratingGuideAnswer Incoming object.
      * @return mixed
      */
-    public function show(RatingGuideAnswer $ratingGuideAnswer)
+    public function show (RatingGuideAnswer $ratingGuideAnswer)
     {
-        $this->authorize('view', $ratingGuideAnswer);
-        return $ratingGuideAnswer->toArray();
+        $this->authorize ('view', $ratingGuideAnswer);
+        return $ratingGuideAnswer->toArray ();
     }
 
     /**
@@ -49,18 +49,18 @@ 
      * @throws \InvalidArgumentException For empty $expected_answer.
      * @return mixed
      */
-    public function update(UpdateRatingGuideAnswer $request, RatingGuideAnswer $ratingGuideAnswer)
+    public function update (UpdateRatingGuideAnswer $request, RatingGuideAnswer $ratingGuideAnswer)
     {
         // Authorization handled by the FormRequest.
         // Data validation handled by this line.
-        $data = $request->validated();
+        $data = $request->validated ();
 
-        $ratingGuideAnswer->fill($data);
-        $ratingGuideAnswer->save();
-        $ratingGuideAnswer->refresh();
+        $ratingGuideAnswer->fill ($data);
+        $ratingGuideAnswer->save ();
+        $ratingGuideAnswer->refresh ();
         return [
             'success' => "Successfully updated rating guide answer $ratingGuideAnswer->id",
-            'rating_guide_answer' => $ratingGuideAnswer->toArray(),
+            'rating_guide_answer' => $ratingGuideAnswer->toArray (),
         ];
     }
 
@@ -70,10 +70,10 @@ 
      * @param  \App\Models\RatingGuideAnswer $ratingGuideAnswer Incoming object.
      * @return mixed
      */
-    public function destroy(RatingGuideAnswer $ratingGuideAnswer)
+    public function destroy (RatingGuideAnswer $ratingGuideAnswer)
     {
-        $this->authorize('delete', $ratingGuideAnswer);
-        $ratingGuideAnswer->delete();
+        $this->authorize ('delete', $ratingGuideAnswer);
+        $ratingGuideAnswer->delete ();
 
         return [
             'success' => "Successfully deleted rating guide answer $ratingGuideAnswer->id"

app/Http/Controllers/JobController.php

Spacing +78 added lines, -78 removed lines

@@ -21,28 +21,28 @@ 
      *
      * @return \Illuminate\Http\Response
      */
-    public function index()
+    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
         ]);
     }
@@ -52,33 +52,33 @@ 
      *
      * @return \Illuminate\Http\Response
      */
-    public function managerIndex()
+    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,
         ]);
@@ -91,9 +91,9 @@ 
      * @param  \App\Models\JobPoster    $jobPoster Job Poster object.
      * @return \Illuminate\Http\Response
      */
-    public function destroy(Request $request, JobPoster $jobPoster)
+    public function destroy (Request $request, JobPoster $jobPoster)
     {
-        $jobPoster->delete();
+        $jobPoster->delete ();
     }
 
     /**
@@ -103,16 +103,16 @@ 
      * @param  \App\Models\JobPoster    $jobPoster Job Poster object.
      * @return \Illuminate\Http\Response
      */
-    public function show(Request $request, JobPoster $jobPoster)
+    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(
+            'essential' => $jobPoster->criteria->filter (
                 function ($value, $key) {
                     return $value->criteria_type->name == 'essential';
                 }
             ),
-            'asset' => $jobPoster->criteria->filter(
+            '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'),
                 ]
             );
         }
@@ -213,24 +213,24 @@ 
      * @param  \App\Models\JobPoster    $jobPoster Job Poster object.
      * @return \Illuminate\Http\Response
      */
-    public function edit(Request $request, JobPoster $jobPoster)
+    public function edit (Request $request, JobPoster $jobPoster)
     {
         $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),
             ]
         );
     }
@@ -241,23 +241,23 @@ 
      * @param  \App\Models\Manager $manager Incoming Manager object.
      * @return \Illuminate\Http\Response Job Create view
      */
-    public function createAsManager(Manager $manager)
+    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);
     }
 
     /**
@@ -268,23 +268,23 @@ 
      * @param  \App\Models\JobPoster    $jobPoster Optional Job Poster object.
      * @return \Illuminate\Http\Response
      */
-    public function store(Request $request, JobPoster $jobPoster)
+    public function store (Request $request, JobPoster $jobPoster)
     {
         // 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));
     }
 
     /**
@@ -295,20 +295,20 @@ 
      * @param  boolean               $replace   Remove existing relationships.
      * @return void
      */
-    protected function fillAndSaveJobPosterQuestions(array $input, JobPoster $jobPoster, bool $replace) : void
+    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 ();
         }
     }
 
@@ -330,23 +330,23 @@ 
      *
      * @return mixed[]|void
      */
-    protected function populateDefaultQuestions()
+    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/ApplicantProfileController.php

Spacing +32 added lines, -32 removed lines

@@ -27,7 +27,7 @@ 
      * @param  \App\Models\Applicant    $applicant Incoming Applicant object.
      * @return \Illuminate\Http\Response
      */
-    public function show(Request $request, Applicant $applicant)
+    public function show (Request $request, Applicant $applicant)
     {
         /*
          * TODO:
@@ -41,11 +41,11 @@ 
          *             description // Question description text
          */
 
-        return view(
+        return view (
             'manager/applicant_profile',
             [
                 // Localization Strings.
-                'profile' => Lang::get('manager/applicant_profile'), // Change text
+                'profile' => Lang::get ('manager/applicant_profile'), // Change text
                 // User Data.
                 'user' => $applicant->user,
                 'applicant' => $applicant,
@@ -60,10 +60,10 @@ 
      * @param  \Illuminate\Http\Request $request Incoming request.
      * @return \Illuminate\Http\Response
      */
-    public function editAuthenticated(Request $request)
+    public function editAuthenticated (Request $request)
     {
-        $applicant = $request->user()->applicant;
-        return redirect(route('profile.about.edit', $applicant));
+        $applicant = $request->user ()->applicant;
+        return redirect (route ('profile.about.edit', $applicant));
     }
 
     /**
@@ -73,16 +73,16 @@ 
      * @param  \App\Models\Applicant    $applicant Applicant to view and edit.
      * @return \Illuminate\Http\Response
      */
-    public function edit(Request $request, Applicant $applicant)
+    public function edit (Request $request, Applicant $applicant)
     {
-        $profileQuestions = ApplicantProfileQuestion::all();
+        $profileQuestions = ApplicantProfileQuestion::all ();
 
-        $profileText = Lang::get('applicant/applicant_profile');
+        $profileText = Lang::get ('applicant/applicant_profile');
 
         $profileQuestionForms = [];
         foreach ($profileQuestions as $question) {
             $answerObj = $applicant->applicant_profile_answers
-                ->where('applicant_profile_question_id', $question->id)->first();
+                ->where ('applicant_profile_question_id', $question->id)->first ();
             $answer = $answerObj ? $answerObj->answer : null;
 
             $formValues = [
@@ -93,13 +93,13 @@ 
                 'answer_label' => $profileText['about_section']['answer_label'],
                 'input_name' => $this->answerFormInputName.'['.$question->id.']'
             ];
-            array_push($profileQuestionForms, $formValues);
+            array_push ($profileQuestionForms, $formValues);
         }
 
         $linkedInUrlPattern = LinkedInUrlRule::PATTERN;
         $twitterHandlePattern = TwitterHandleRule::PATTERN;
 
-        return view(
+        return view (
             'applicant/profile_01_about',
             [
                 // Localized strings.
@@ -110,7 +110,7 @@ 
                 'user' => $applicant->user,
                 'applicant' => $applicant,
                 'profile_photo_url' => '/images/user.png', // TODO: get real photos.
-                'form_submit_action' => route('profile.about.update', $applicant),
+                'form_submit_action' => route ('profile.about.update', $applicant),
                 'linkedInUrlPattern' => $linkedInUrlPattern,
                 'twitterHandlePattern' => $twitterHandlePattern,
             ]
@@ -124,44 +124,44 @@ 
      * @param  \App\Models\Applicant    $applicant Applicant object to update.
      * @return \Illuminate\Http\Response
      */
-    public function update(Request $request, Applicant $applicant)
+    public function update (Request $request, Applicant $applicant)
     {
-        $questions = ApplicantProfileQuestion::all();
+        $questions = ApplicantProfileQuestion::all ();
 
-        $validator = new UpdateApplicationProfileValidator($applicant);
-        $validator->validate($request->all());
+        $validator = new UpdateApplicationProfileValidator ($applicant);
+        $validator->validate ($request->all ());
 
         foreach ($questions as $question) {
-            $answerName = $this->answerFormInputName . '.' . $question->id;
-            if ($request->has($answerName)) {
-                $answer = ApplicantProfileAnswer::where(
+            $answerName = $this->answerFormInputName.'.'.$question->id;
+            if ($request->has ($answerName)) {
+                $answer = ApplicantProfileAnswer::where (
                     [
                         'applicant_id' => $applicant->id,
                         'applicant_profile_question_id' => $question->id
                     ]
-                )->first();
+                )->first ();
                 if ($answer == null) {
-                    $answer = new ApplicantProfileAnswer();
-                    $answer->applicant_id =$applicant->id;
+                    $answer = new ApplicantProfileAnswer ();
+                    $answer->applicant_id = $applicant->id;
                     $answer->applicant_profile_question_id = $question->id;
                 }
-                $answer->answer = $request->input($answerName);
-                $answer->save();
+                $answer->answer = $request->input ($answerName);
+                $answer->save ();
             }
         }
 
-        $input = $request->input();
-        $applicant->fill(
+        $input = $request->input ();
+        $applicant->fill (
             [
                 'tagline' => $input['tagline'],
                 'twitter_username' => $input['twitter_username'],
                 'linkedin_url' => $input['linkedin_url'],
             ]
         );
-        $applicant->save();
+        $applicant->save ();
 
         $user = $applicant->user;
-        $user->fill(
+        $user->fill (
             [
                 'first_name' => $input['profile_first_name'],
                 'last_name' => $input['profile_last_name'],
@@ -169,10 +169,10 @@ 
             ]
         );
         if ($input['new_password']) {
-            $user->password = Hash::make($input['new_password']); // TODO: change password in seperate form!
+            $user->password = Hash::make ($input['new_password']); // TODO: change password in seperate form!
         }
-        $user->save();
+        $user->save ();
 
-        return redirect()->route('profile.about.edit', $applicant);
+        return redirect ()->route ('profile.about.edit', $applicant);
     }
 }

app/Http/Controllers/JobBuilderController.php

Spacing +4 added lines, -4 removed lines

@@ -12,12 +12,12 @@
      * Show the Job Builder mini SPA
      * @return \Illuminate\Http\Response
      */
-    public function show()
+    public function show ()
     {
-        return view(
+        return view (
             'manager/job-builder-root'
-        )->with([
-            'title' => Lang::get('manager/job_builder.title'),
+        )->with ([
+            'title' => Lang::get ('manager/job_builder.title'),
         ]);
     }
 }