@@ -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 | } |
@@ -20,8 +20,8 @@ discard block |
||
| 20 | 20 | */ |
| 21 | 21 | public function indexByJob(JobPoster $jobPoster) |
| 22 | 22 | { |
| 23 | - $criteriaByJob = Criteria::where('job_poster_id', $jobPoster->id)->get(); |
|
| 24 | - return CriteriaResource::collection($criteriaByJob); |
|
| 23 | + $criteriaByJob = Criteria::where ('job_poster_id', $jobPoster->id)->get (); |
|
| 24 | + return CriteriaResource::collection ($criteriaByJob); |
|
| 25 | 25 | } |
| 26 | 26 | |
| 27 | 27 | /** |
@@ -33,38 +33,38 @@ discard block |
||
| 33 | 33 | */ |
| 34 | 34 | public function batchUpdate(Request $request, JobPoster $jobPoster) |
| 35 | 35 | { |
| 36 | - $newCriteria = collect($request->input()); // TODO: switch to validated |
|
| 36 | + $newCriteria = collect ($request->input ()); // TODO: switch to validated |
|
| 37 | 37 | $oldCriteria = $jobPoster->criteria; |
| 38 | 38 | |
| 39 | 39 | $updatedIds = []; |
| 40 | 40 | |
| 41 | 41 | // First, delete old criteria that weren't resubmitted, and update those that were. |
| 42 | 42 | foreach ($oldCriteria as $criteria) { |
| 43 | - $newData = $newCriteria->firstWhere('id', $criteria['id']); |
|
| 43 | + $newData = $newCriteria->firstWhere ('id', $criteria['id']); |
|
| 44 | 44 | if ($newData) { |
| 45 | 45 | $updatedIds[] = $criteria->id; |
| 46 | - $this->updateCriteria($criteria, $newData); |
|
| 46 | + $this->updateCriteria ($criteria, $newData); |
|
| 47 | 47 | } else { |
| 48 | - $this->deleteCriteria($criteria); |
|
| 48 | + $this->deleteCriteria ($criteria); |
|
| 49 | 49 | } |
| 50 | 50 | } |
| 51 | 51 | |
| 52 | 52 | $isUnsaved = function ($criteria, $savedIds): bool { |
| 53 | - return !array_key_exists('id', $criteria) || !in_array($criteria['id'], $savedIds); |
|
| 53 | + return !array_key_exists ('id', $criteria) || !in_array ($criteria['id'], $savedIds); |
|
| 54 | 54 | }; |
| 55 | 55 | |
| 56 | 56 | // Now, save any new criteria that remain. |
| 57 | 57 | foreach ($newCriteria as $criteriaData) { |
| 58 | - if ($isUnsaved($criteriaData, $updatedIds)) { |
|
| 59 | - $criteria = new Criteria(); |
|
| 58 | + if ($isUnsaved ($criteriaData, $updatedIds)) { |
|
| 59 | + $criteria = new Criteria (); |
|
| 60 | 60 | $criteria->job_poster_id = $jobPoster->id; |
| 61 | - $fillableData = collect($criteriaData)->except(['id', 'job_poster_id'])->toArray(); |
|
| 62 | - $criteria->fill($fillableData); |
|
| 63 | - $this->createCriteria($criteria); |
|
| 61 | + $fillableData = collect ($criteriaData)->except (['id', 'job_poster_id'])->toArray (); |
|
| 62 | + $criteria->fill ($fillableData); |
|
| 63 | + $this->createCriteria ($criteria); |
|
| 64 | 64 | } |
| 65 | 65 | } |
| 66 | 66 | |
| 67 | - return CriteriaResource::collection($jobPoster->fresh()->criteria); |
|
| 67 | + return CriteriaResource::collection ($jobPoster->fresh ()->criteria); |
|
| 68 | 68 | } |
| 69 | 69 | |
| 70 | 70 | /** |
@@ -75,12 +75,12 @@ discard block |
||
| 75 | 75 | */ |
| 76 | 76 | protected function createCriteria(Criteria $criteria) |
| 77 | 77 | { |
| 78 | - $criteria->save(); |
|
| 79 | - $notification = $this->makeAssessmentPlanNotification( |
|
| 78 | + $criteria->save (); |
|
| 79 | + $notification = $this->makeAssessmentPlanNotification ( |
|
| 80 | 80 | 'CREATE', |
| 81 | 81 | $criteria |
| 82 | 82 | ); |
| 83 | - $notification->save(); |
|
| 83 | + $notification->save (); |
|
| 84 | 84 | |
| 85 | 85 | return $criteria; |
| 86 | 86 | } |
@@ -98,19 +98,19 @@ discard block |
||
| 98 | 98 | if ($oldCriteria->skill_level_id != $newData['skill_level_id'] |
| 99 | 99 | || $oldCriteria->skill_id != $newData['skill_id'] |
| 100 | 100 | ) { |
| 101 | - $notification = $this->makeAssessmentPlanNotification( |
|
| 101 | + $notification = $this->makeAssessmentPlanNotification ( |
|
| 102 | 102 | 'UPDATE', |
| 103 | 103 | $oldCriteria, |
| 104 | 104 | $newData['skill_id'], |
| 105 | 105 | $newData['skill_level_id'], |
| 106 | 106 | $newData['criteria_type_id'] |
| 107 | 107 | ); |
| 108 | - $notification->save(); |
|
| 108 | + $notification->save (); |
|
| 109 | 109 | } |
| 110 | 110 | // Get just the data that can be changed. |
| 111 | - $fillableData = collect($newData)->except(['id', 'job_poster_id'])->toArray(); |
|
| 112 | - $oldCriteria->fill($fillableData); |
|
| 113 | - $oldCriteria->save(); |
|
| 111 | + $fillableData = collect ($newData)->except (['id', 'job_poster_id'])->toArray (); |
|
| 112 | + $oldCriteria->fill ($fillableData); |
|
| 113 | + $oldCriteria->save (); |
|
| 114 | 114 | } |
| 115 | 115 | |
| 116 | 116 | /** |
@@ -121,15 +121,15 @@ discard block |
||
| 121 | 121 | */ |
| 122 | 122 | protected function deleteCriteria(Criteria $criteria): void |
| 123 | 123 | { |
| 124 | - $notification = $notification = $this->makeAssessmentPlanNotification( |
|
| 124 | + $notification = $notification = $this->makeAssessmentPlanNotification ( |
|
| 125 | 125 | 'DELETE', |
| 126 | 126 | $criteria |
| 127 | 127 | ); |
| 128 | - $notification->save(); |
|
| 128 | + $notification->save (); |
|
| 129 | 129 | |
| 130 | 130 | // Delete assessments related to this criteria. |
| 131 | - Assessment::where('criterion_id', $criteria->id)->delete(); |
|
| 132 | - $criteria->delete(); |
|
| 131 | + Assessment::where ('criterion_id', $criteria->id)->delete (); |
|
| 132 | + $criteria->delete (); |
|
| 133 | 133 | } |
| 134 | 134 | |
| 135 | 135 | /** |
@@ -148,8 +148,8 @@ discard block |
||
| 148 | 148 | $newSkillId = null, |
| 149 | 149 | $newSkillLevelId = null, |
| 150 | 150 | $newCriteriaTypeId = null |
| 151 | - ) { |
|
| 152 | - $notification = new AssessmentPlanNotification(); |
|
| 151 | + ){ |
|
| 152 | + $notification = new AssessmentPlanNotification (); |
|
| 153 | 153 | $notification->job_poster_id = $criteria->job_poster_id; |
| 154 | 154 | $notification->type = $type; |
| 155 | 155 | $notification->criteria_id = $criteria->id; |