GCTC-NTGC / TalentCloud

app/Http/Requests/SkillCrudRequest.php

Spacing +5 added lines, -5 removed lines

@@ -11,10 +11,10 @@ 
      *
      * @return boolean
      */
-    public function authorize() : bool
+    public function authorize () : bool
     {
         // Only allow updates if the user is a logged in Admin.
-        return backpack_auth()->check();
+        return backpack_auth ()->check ();
     }
 
     /**
@@ -22,10 +22,10 @@ 
      *
      * @return string[]
      */
-    public function rules() : array
+    public function rules () : array
     {
         return [
-            'name' => 'required|unique_translation:skills,name' . (isset($this->id) ? ",{$this->id}" : ''),
+            'name' => 'required|unique_translation:skills,name'.(isset($this->id) ? ",{$this->id}" : ''),
             'description' => 'required',
             'skill_type_id' => 'exists:skill_types,id'
         ];
@@ -36,7 +36,7 @@ 
      *
      * @return string[]
      */
-    public function messages() : array
+    public function messages () : array
     {
         return [
             'name.required' => 'Please enter a Skill name.',

app/Http/Requests/UpdateJobPoster.php

Spacing +14 added lines, -14 removed lines

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

app/Http/Requests/UpdateManagerProfileRequest.php

Spacing +5 added lines, -5 removed lines

@@ -26,7 +26,7 @@ 
      *
      * @return boolean
      */
-    public function authorize()
+    public function authorize ()
     {
         return $this->manager !== null;
     }
@@ -36,7 +36,7 @@ 
      *
      * @return void
      */
-    protected function failedValidation(\Illuminate\Contracts\Validation\Validator $validator)
+    protected function failedValidation (\Illuminate\Contracts\Validation\Validator $validator)
     {
         $this->validator = $validator;
     }
@@ -46,7 +46,7 @@ 
      *
      * @return array
      */
-    public function rules()
+    public function rules ()
     {
         return [
             'first_name' => 'required|string|max:191',
@@ -58,7 +58,7 @@ 
                 'email',
                 // Email may match existing email for this user,
                 // but must be unique if changed.
-                Rule::unique('users', 'email')->ignore($this->manager->user->id)
+                Rule::unique ('users', 'email')->ignore ($this->manager->user->id)
             ],
             'gov_email' => [
                 'nullable',
@@ -79,7 +79,7 @@ 
                 'confirmed'
             ],
 
-            'department_id' => ['required', new ValidIdRule(Department::class)],
+            'department_id' => ['required', new ValidIdRule (Department::class)],
 
             'years_experience' => 'nullable|numeric|min:0',
             'gc_directory_url' => 'nullable|url',

app/Http/Middleware/DateLocale.php

Spacing +3 added lines, -3 removed lines

@@ -14,10 +14,10 @@
      * @param  \Closure  $next
      * @return mixed
      */
-    public function handle($request, Closure $next)
+    public function handle ($request, Closure $next)
     {
-        Date::setLocale(app()->getLocale());
+        Date::setLocale (app ()->getLocale ());
         
-        return $next($request);
+        return $next ($request);
     }
 }

app/Http/Middleware/VerifyCsrfToken.php

Spacing +6 added lines, -6 removed lines

@@ -32,15 +32,15 @@
      * @param  \Symfony\Component\HttpFoundation\Response $response Outgoing response.
      * @return \Symfony\Component\HttpFoundation\Response
      */
-    protected function addCookieToResponse($request, $response)
+    protected function addCookieToResponse ($request, $response)
     {
         if ($this->addHttpCookie) {
-            $config = config('session');
-            $response->headers->setCookie(
-                new Cookie(
+            $config = config ('session');
+            $response->headers->setCookie (
+                new Cookie (
                     'XSRF-TOKEN',
-                    $request->session()->token(),
-                    $this->availableAt(60 * $config['lifetime']),
+                    $request->session ()->token (),
+                    $this->availableAt (60 * $config['lifetime']),
                     $config['path'],
                     $config['domain'],
                     $config['secure'],

app/Http/Middleware/FinishManagerRegistration.php

Spacing +6 added lines, -6 removed lines

@@ -12,15 +12,15 @@
      * @param  \Closure  $next
      * @return mixed
      */
-    public function handle($request, Closure $next)
+    public function handle ($request, Closure $next)
     {
-        $user = $request->user();
+        $user = $request->user ();
 
-        if ($user !== null && $user->isManager() && !$user->isGovIdentityConfirmed()) {
-            $request->session()->put('url.expected', $request->url());
-            return redirect(route('manager.first_visit'));
+        if ($user !== null && $user->isManager () && !$user->isGovIdentityConfirmed ()) {
+            $request->session ()->put ('url.expected', $request->url ());
+            return redirect (route ('manager.first_visit'));
         }
 
-        return $next($request);
+        return $next ($request);
     }
 }

app/Http/Middleware/CheckIfAdmin.php

Spacing +12 added lines, -12 removed lines

@@ -20,9 +20,9 @@ 
      *
      * @return bool [description]
      */
-    private function checkIfUserIsAdmin($user)
+    private function checkIfUserIsAdmin ($user)
     {
-        return ($user->isAdmin());
+        return ($user->isAdmin ());
     }
 
     /**
@@ -32,12 +32,12 @@ 
      *
      * @return [type] [description]
      */
-    private function respondToUnauthorizedRequest($request)
+    private function respondToUnauthorizedRequest ($request)
     {
-        if ($request->ajax() || $request->wantsJson()) {
-            return response(trans('backpack::base.unauthorized'), 401);
+        if ($request->ajax () || $request->wantsJson ()) {
+            return response (trans ('backpack::base.unauthorized'), 401);
         } else {
-            return redirect()->guest(backpack_url('login'));
+            return redirect ()->guest (backpack_url ('login'));
         }
     }
 
@@ -49,16 +49,16 @@ 
      *
      * @return mixed
      */
-    public function handle($request, Closure $next)
+    public function handle ($request, Closure $next)
     {
-        if (backpack_auth()->guest()) {
-            return $this->respondToUnauthorizedRequest($request);
+        if (backpack_auth ()->guest ()) {
+            return $this->respondToUnauthorizedRequest ($request);
         }
 
-        if (!$this->checkIfUserIsAdmin(backpack_user())) {
-            return $this->respondToUnauthorizedRequest($request);
+        if (!$this->checkIfUserIsAdmin (backpack_user ())) {
+            return $this->respondToUnauthorizedRequest ($request);
         }
 
-        return $next($request);
+        return $next ($request);
     }
 }

app/Http/Middleware/CheckRole.php

Spacing +10 added lines, -10 removed lines

@@ -18,24 +18,24 @@
      * @param  string  $role
      * @return mixed
      */
-    public function handle($request, Closure $next, $role)
+    public function handle ($request, Closure $next, $role)
     {
         // If user logged in as admin, always pass, regardless of $role
-        if (Auth::check() && Auth::user()->isAdmin()) {
-            Log::info('CheckRole Bypassed as Admin');
-            return $next($request);
+        if (Auth::check () && Auth::user ()->isAdmin ()) {
+            Log::info ('CheckRole Bypassed as Admin');
+            return $next ($request);
         }
 
         // Redirect if not logged in, or if not the correct role
-        if (Auth::guest() || !Auth::user()->hasRole($role)) {
-            Log::info('CheckRole Failed');
+        if (Auth::guest () || !Auth::user ()->hasRole ($role)) {
+            Log::info ('CheckRole Failed');
             // TODO: redirect to some sort of error messag
-            if (WhichPortal::isManagerPortal()) {
-                return redirect(route('manager.home'));
+            if (WhichPortal::isManagerPortal ()) {
+                return redirect (route ('manager.home'));
             } else {
-                return redirect(route('home'));
+                return redirect (route ('home'));
             }
         }
-        return $next($request);
+        return $next ($request);
     }
 }

app/Http/Middleware/LocalOnly.php

Spacing +4 added lines, -4 removed lines

@@ -13,11 +13,11 @@
      * @param  \Closure  $next
      * @return mixed
      */
-    public function handle($request, Closure $next)
+    public function handle ($request, Closure $next)
     {
-        if (\App::environment('local')) {
-            return $next($request);
+        if (\App::environment ('local')) {
+            return $next ($request);
         }
-        abort(404);
+        abort (404);
     }
 }