@@ -29,9 +29,9 @@ |
||
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', |
@@ -58,7 +58,7 @@ discard block |
||
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 |
||
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', |
@@ -18,11 +18,11 @@ discard block |
||
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 |
||
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 | } |
@@ -19,8 +19,8 @@ discard block |
||
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 |
||
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 | } |
@@ -14,7 +14,7 @@ discard block |
||
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 |
||
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 | ]; |
@@ -24,7 +24,7 @@ discard block |
||
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 |
||
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 |
||
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', |
@@ -14,6 +14,6 @@ |
||
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 | } |
@@ -28,9 +28,9 @@ |
||
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', |
@@ -14,6 +14,6 @@ |
||
14 | 14 | */ |
15 | 15 | public function toArray($request) |
16 | 16 | { |
17 | - return parent::toArray($request); |
|
17 | + return parent::toArray ($request); |
|
18 | 18 | } |
19 | 19 | } |