Passed
Push — task/common-translation-packag... ( 1125af...852212 )
by Grant
08:02
created
app/Http/Requests/BatchUpdateCriteria.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -29,9 +29,9 @@
 block discarded – undo
29 29
     {
30 30
         return [
31 31
             '*.id' => 'present',
32
-            '*.criteria_type_id' => ['required', new ValidIdRule(CriteriaType::class)],
33
-            '*.skill_id' => ['required', new ValidIdRule(Skill::class)],
34
-            '*.skill_level_id' => ['required', new ValidIdRule(SkillLevel::class)],
32
+            '*.criteria_type_id' => ['required', new ValidIdRule (CriteriaType::class)],
33
+            '*.skill_id' => ['required', new ValidIdRule (Skill::class)],
34
+            '*.skill_level_id' => ['required', new ValidIdRule (SkillLevel::class)],
35 35
             '*.description.en' => 'nullable|string',
36 36
             '*.description.fr' => 'nullable|string',
37 37
             '*.specificity.en' => 'nullable|string',
Please login to merge, or discard this patch.
app/Http/Requests/UpdateManagerProfileRequest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
                 'email',
59 59
                 // Email may match existing email for this user,
60 60
                 // but must be unique if changed.
61
-                Rule::unique('users', 'email')->ignore($this->manager->user->id)
61
+                Rule::unique ('users', 'email')->ignore ($this->manager->user->id)
62 62
             ],
63 63
             'gov_email' => [
64 64
                 'nullable',
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
                 'confirmed'
80 80
             ],
81 81
 
82
-            'department_id' => ['required', new ValidIdRule(Department::class)],
82
+            'department_id' => ['required', new ValidIdRule (Department::class)],
83 83
 
84 84
             'years_experience' => 'nullable|numeric|min:0',
85 85
             'gc_directory_url' => 'nullable|url',
Please login to merge, or discard this patch.
app/Http/Requests/StoreRatingGuideAnswer.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -18,11 +18,11 @@  discard block
 block discarded – undo
18 18
     public function authorize()
19 19
     {
20 20
         // Ensure the user can make answers, question exists, and user is the owner of the question it answers.
21
-        if ($this->user()->can('create', RatingGuideAnswer::class)) {
22
-            $questionId = (int) $this->input('rating_guide_question_id');
21
+        if ($this->user ()->can ('create', RatingGuideAnswer::class)) {
22
+            $questionId = (int) $this->input ('rating_guide_question_id');
23 23
             if ($questionId) {
24
-                $question = RatingGuideQuestion::find($questionId);
25
-                return $question && $this->user()->can('update', $question);
24
+                $question = RatingGuideQuestion::find ($questionId);
25
+                return $question && $this->user ()->can ('update', $question);
26 26
             }
27 27
         }
28 28
 
@@ -37,8 +37,8 @@  discard block
 block discarded – undo
37 37
     public function rules()
38 38
     {
39 39
         return [
40
-            'rating_guide_question_id' => ['required', new ValidIdRule(RatingGuideQuestion::class)],
41
-            'criterion_id' => ['nullable', new ValidIdRule(Criteria::class)],
40
+            'rating_guide_question_id' => ['required', new ValidIdRule (RatingGuideQuestion::class)],
41
+            'criterion_id' => ['nullable', new ValidIdRule (Criteria::class)],
42 42
             'expected_answer' => 'nullable|string',
43 43
         ];
44 44
     }
Please login to merge, or discard this patch.
app/Http/Requests/UpdateRatingGuideAnswer.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -19,8 +19,8 @@  discard block
 block discarded – undo
19 19
         //$answer = RatingGuideAnswer::find($this->route()->originalParameter('ratingGuideAnswer'));
20 20
 
21 21
         // The id parameter in the route is typehinted to magically become a RatingGuideAnswer object.
22
-        $answer = $this->route('ratingGuideAnswer');
23
-        return $answer && $this->user()->can('update', $answer);
22
+        $answer = $this->route ('ratingGuideAnswer');
23
+        return $answer && $this->user ()->can ('update', $answer);
24 24
     }
25 25
 
26 26
     /**
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
         return [
34 34
             // RatingGuideQUestionId shouldn't be updated after creation
35 35
             //'rating_guide_question_id' => ['required', new RatingGuideQuestionRule()],
36
-            'criterion_id' => ['nullable', new ValidIdRule(Criteria::class)],
36
+            'criterion_id' => ['nullable', new ValidIdRule (Criteria::class)],
37 37
             'expected_answer' => 'nullable|string',
38 38
         ];
39 39
     }
Please login to merge, or discard this patch.
app/Http/Requests/SkillCrudRequest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
     public function authorize() : bool
15 15
     {
16 16
         // Only allow updates if the user is a logged in Admin.
17
-        return backpack_auth()->check();
17
+        return backpack_auth ()->check ();
18 18
     }
19 19
 
20 20
     /**
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
     public function rules() : array
26 26
     {
27 27
         return [
28
-            'name' => 'required|unique_translation:skills,name' . (isset($this->id) ? ",{$this->id}" : ''),
28
+            'name' => 'required|unique_translation:skills,name'.(isset($this->id) ? ",{$this->id}" : ''),
29 29
             'description' => 'required',
30 30
             'skill_type_id' => 'exists:skill_types,id'
31 31
         ];
Please login to merge, or discard this patch.
app/Http/Requests/UpdateJobPoster.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
      */
25 25
     public function authorize(): bool
26 26
     {
27
-        $job = $this->route('job');
27
+        $job = $this->route ('job');
28 28
         // Published jobs cannot be updated.
29 29
         return $job && $job->published_at === null;
30 30
     }
@@ -36,23 +36,23 @@  discard block
 block discarded – undo
36 36
      */
37 37
     public function rules(): array
38 38
     {
39
-        $dateFormat = Config::get('app.api_datetime_format');
39
+        $dateFormat = Config::get ('app.api_datetime_format');
40 40
         $dateFormatRule = "date_format:$dateFormat";
41 41
         $sliderRule = 'between:1,4';
42 42
         return [
43
-            'chosen_lang' => ['nullable', Rule::in(['en', 'fr'])],
43
+            'chosen_lang' => ['nullable', Rule::in (['en', 'fr'])],
44 44
             'term_qty' => 'nullable|numeric',
45 45
             'open_date_time' =>['nullable', $dateFormatRule],
46 46
             'close_date_time' => ['nullable', $dateFormatRule],
47 47
             'start_date_time' =>['nullable', $dateFormatRule],
48
-            'department_id' => ['nullable', new ValidIdRule(Department::class)],
49
-            'province_id' => ['nullable', new ValidIdRule(Province::class)],
50
-            'security_clearance_id' => ['nullable', new ValidIdRule(SecurityClearance::class)],
51
-            'language_requirement_id' => ['nullable', new ValidIdRule(LanguageRequirement::class)],
48
+            'department_id' => ['nullable', new ValidIdRule (Department::class)],
49
+            'province_id' => ['nullable', new ValidIdRule (Province::class)],
50
+            'security_clearance_id' => ['nullable', new ValidIdRule (SecurityClearance::class)],
51
+            'language_requirement_id' => ['nullable', new ValidIdRule (LanguageRequirement::class)],
52 52
             'salary_min' => 'nullable|numeric',
53 53
             'salary_max' => 'nullable|numeric',
54 54
             'noc' => 'nullable|numeric',
55
-            'classification_id' => ['nullable', new ValidIdRule(Classification::class)],
55
+            'classification_id' => ['nullable', new ValidIdRule (Classification::class)],
56 56
             'classification_level' => 'nullable|numeric',
57 57
             'remote_work_allowed' => 'nullable|boolean',
58 58
             'team_size' => 'nullable|numeric',
@@ -63,10 +63,10 @@  discard block
 block discarded – undo
63 63
             'experimental_vs_ongoing' => ['nullable', $sliderRule],
64 64
             'citizen_facing_vs_back_office' => ['nullable', $sliderRule],
65 65
             'collaborative_vs_independent' => ['nullable', $sliderRule],
66
-            'telework_allowed_frequency_id' => ['nullable', new ValidIdRule(Frequency::class)],
67
-            'flexible_hours_frequency_id' => ['nullable', new ValidIdRule(Frequency::class)],
68
-            'travel_requirement_id' => ['nullable', new ValidIdRule(TravelRequirement::class)],
69
-            'overtime_requirement_id' => ['nullable', new ValidIdRule(OvertimeRequirement::class)],
66
+            'telework_allowed_frequency_id' => ['nullable', new ValidIdRule (Frequency::class)],
67
+            'flexible_hours_frequency_id' => ['nullable', new ValidIdRule (Frequency::class)],
68
+            'travel_requirement_id' => ['nullable', new ValidIdRule (TravelRequirement::class)],
69
+            'overtime_requirement_id' => ['nullable', new ValidIdRule (OvertimeRequirement::class)],
70 70
             'city.en' => 'nullable|string',
71 71
             'city.fr' => 'nullable|string',
72 72
             'title.en' => 'nullable|string',
Please login to merge, or discard this patch.
app/Http/Requests/StoreJobPoster.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -14,6 +14,6 @@
 block discarded – undo
14 14
     public function authorize(): bool
15 15
     {
16 16
         // The STORE job poster method requires the user's manager id.
17
-        return $this->user()->manager !== null;
17
+        return $this->user ()->manager !== null;
18 18
     }
19 19
 }
Please login to merge, or discard this patch.
app/Http/Requests/UpdateManagerApi.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -28,9 +28,9 @@
 block discarded – undo
28 28
      */
29 29
     public function rules()
30 30
     {
31
-        $frequencyRule = new ValidIdRule(Frequency::class);
31
+        $frequencyRule = new ValidIdRule (Frequency::class);
32 32
         return [
33
-            'department_id' => ['nullable', new ValidIdRule(Department::class)],
33
+            'department_id' => ['nullable', new ValidIdRule (Department::class)],
34 34
             'gov_email' => [
35 35
                 'nullable',
36 36
                 'string',
Please login to merge, or discard this patch.
app/Http/Resources/JobPosterKeyTask.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -14,6 +14,6 @@
 block discarded – undo
14 14
      */
15 15
     public function toArray($request)
16 16
     {
17
-        return parent::toArray($request);
17
+        return parent::toArray ($request);
18 18
     }
19 19
 }
Please login to merge, or discard this patch.