@@ -10,7 +10,7 @@ discard block |
||
10 | 10 | { |
11 | 11 | public function get() |
12 | 12 | { |
13 | - if (! backpack_user()->hasPermissionTo('courses.edit')) { |
|
13 | + if (!backpack_user()->hasPermissionTo('courses.edit')) { |
|
14 | 14 | abort(403); |
15 | 15 | } |
16 | 16 | |
@@ -23,7 +23,7 @@ discard block |
||
23 | 23 | |
24 | 24 | public function update(UpdateConfigRequest $request) |
25 | 25 | { |
26 | - if (! backpack_user()->hasPermissionTo('courses.edit')) { |
|
26 | + if (!backpack_user()->hasPermissionTo('courses.edit')) { |
|
27 | 27 | abort(403); |
28 | 28 | } |
29 | 29 |
@@ -19,7 +19,7 @@ |
||
19 | 19 | // if the sync with external mailing system is enabled |
20 | 20 | if (config('mailing-system.external_mailing_enabled') == true) { |
21 | 21 | |
22 | - match ($request->input('status')) { |
|
22 | + match($request->input('status')) { |
|
23 | 23 | 1 => $listId = config('mailing-system.mailerlite.activeStudentsListId'), |
24 | 24 | 2, 3 => $listId = config('mailing-system.mailerlite.inactiveStudentsListId'), |
25 | 25 | default => abort(422, 'List ID not found'), |
@@ -32,7 +32,7 @@ |
||
32 | 32 | $report_end_date = $request->report_end_date ? Carbon::parse($request->report_end_date) : Carbon::parse($period->end); |
33 | 33 | |
34 | 34 | // if we are dealing with a complete period, add theoretical volumes |
35 | - if (! $request->report_start_date && ! $request->report_end_date) { |
|
35 | + if (!$request->report_start_date && !$request->report_end_date) { |
|
36 | 36 | // ensure the report end date is not before the end date to avoid inconsistent results. |
37 | 37 | $report_end_date = $report_start_date->max($report_end_date); |
38 | 38 |
@@ -32,7 +32,7 @@ |
||
32 | 32 | // foreach partner with an alert set for this day of month |
33 | 33 | foreach (Partner::where('send_report_on', Carbon::now()->day)->get() as $partner) { |
34 | 34 | // get all courses |
35 | - $courses = $partner->courses()->whereHas('events', function (Builder $query) use ($period_start, $period_end) { |
|
35 | + $courses = $partner->courses()->whereHas('events', function(Builder $query) use ($period_start, $period_end) { |
|
36 | 36 | $query->where('start', '>=', $period_start->toDateString())->where('end', '<=', $period_end->toDateString()); |
37 | 37 | }) |
38 | 38 | ->with('events')->withCount('events') |
@@ -34,12 +34,12 @@ discard block |
||
34 | 34 | */ |
35 | 35 | protected function schedule(Schedule $schedule) |
36 | 36 | { |
37 | - $schedule->call(function () { |
|
37 | + $schedule->call(function() { |
|
38 | 38 | Log::info('Sending attendance reminders'); |
39 | 39 | $this->remindPendingAttendance(); |
40 | 40 | })->dailyAt('08:15'); |
41 | 41 | |
42 | - $schedule->call(function () { |
|
42 | + $schedule->call(function() { |
|
43 | 43 | Log::info('Checking default periods'); |
44 | 44 | |
45 | 45 | // when we finish the current period; remove the manual override to automatically fallbackto the next one |
@@ -57,14 +57,14 @@ discard block |
||
57 | 57 | })->dailyAt('00:00'); |
58 | 58 | |
59 | 59 | if (config('settings.partnership_alerts')) { |
60 | - $schedule->call(function () { |
|
60 | + $schedule->call(function() { |
|
61 | 61 | Log::info('Checking expired partnerships'); |
62 | 62 | |
63 | 63 | // if one of the partnerships is expiring soon, send an email |
64 | - $partners = Partner::where(function ($query) { |
|
64 | + $partners = Partner::where(function($query) { |
|
65 | 65 | $query->whereNotNull('expired_on')->where('expired_on', '<', Carbon::now()->addDays(28)); |
66 | 66 | }) |
67 | - ->where(function ($query) { |
|
67 | + ->where(function($query) { |
|
68 | 68 | $query->whereNull('last_alert_sent_at')->orWhere('last_alert_sent_at', '>', Carbon::now()->subDays(28))->get(); |
69 | 69 | }); |
70 | 70 | |
@@ -75,14 +75,14 @@ discard block |
||
75 | 75 | } |
76 | 76 | |
77 | 77 | if (config('settings.external_courses_report')) { |
78 | - $schedule->call(function () { |
|
78 | + $schedule->call(function() { |
|
79 | 79 | Log::info('Sending external courses reports'); |
80 | 80 | event(new ExternalCoursesReportEvent()); |
81 | 81 | })->dailyAt('02:10'); |
82 | 82 | } |
83 | 83 | |
84 | 84 | if (config('settings.monthly_report')) { |
85 | - $schedule->call(function () { |
|
85 | + $schedule->call(function() { |
|
86 | 86 | Log::info('Sending monthly hours report'); |
87 | 87 | event(new MonthlyReportEvent()); |
88 | 88 | })->monthlyOn(20); |
@@ -32,7 +32,7 @@ |
||
32 | 32 | $this->crud->allowAccess(['showstudentphotoroster']); |
33 | 33 | $this->crud->setOperationSetting('setFromDb', true); |
34 | 34 | |
35 | - $this->crud->operation('showstudentphotoroster', function () { |
|
35 | + $this->crud->operation('showstudentphotoroster', function() { |
|
36 | 36 | $this->crud->loadDefaultOperationSettingsFromConfig(); |
37 | 37 | }); |
38 | 38 |
@@ -20,7 +20,7 @@ discard block |
||
20 | 20 | |
21 | 21 | $this->hideSensitiveRequestDetails(); |
22 | 22 | |
23 | - Telescope::filter(function (IncomingEntry $entry) { |
|
23 | + Telescope::filter(function(IncomingEntry $entry) { |
|
24 | 24 | if ($this->app->isLocal()) { |
25 | 25 | return true; |
26 | 26 | } |
@@ -61,6 +61,6 @@ discard block |
||
61 | 61 | */ |
62 | 62 | protected function gate() |
63 | 63 | { |
64 | - Gate::define('viewTelescope', fn ($user = null) => backpack_user() && backpack_user()->email === config('app.admin_email')); |
|
64 | + Gate::define('viewTelescope', fn($user = null) => backpack_user() && backpack_user()->email === config('app.admin_email')); |
|
65 | 65 | } |
66 | 66 | } |
@@ -132,7 +132,7 @@ discard block |
||
132 | 132 | |
133 | 133 | public function updateCourse(Course $course) : void |
134 | 134 | { |
135 | - if (! $course->lms_id) { |
|
135 | + if (!$course->lms_id) { |
|
136 | 136 | $this->createCourse($course); |
137 | 137 | } |
138 | 138 | |
@@ -161,7 +161,7 @@ discard block |
||
161 | 161 | Log::info('found these teachers IDs on LMS:'.implode(', ', $teachers->pluck('id')->toArray())); |
162 | 162 | |
163 | 163 | // if the course has no teacher, stop |
164 | - if (! $course->teacher) { |
|
164 | + if (!$course->teacher) { |
|
165 | 165 | Log::alert('The course has no teacher on local system, removing all teachers from remote'); |
166 | 166 | foreach ($teachers as $teacher) { |
167 | 167 | $this->removeTeacher($course->lms_id, $teacher['id']); |
@@ -176,7 +176,7 @@ discard block |
||
176 | 176 | } |
177 | 177 | } |
178 | 178 | |
179 | - if (! $course->teacher->user->lms_id || ! $teachers->contains('id', $course->teacher->user->lms_id)) { |
|
179 | + if (!$course->teacher->user->lms_id || !$teachers->contains('id', $course->teacher->user->lms_id)) { |
|
180 | 180 | Log::info('the course teacher has changed, need to update it'); |
181 | 181 | $this->addTeacher($course); |
182 | 182 | } |
@@ -186,14 +186,14 @@ discard block |
||
186 | 186 | |
187 | 187 | public function enrollStudent(Course $course, Student $student): void |
188 | 188 | { |
189 | - if (! $course->lms_id) { |
|
189 | + if (!$course->lms_id) { |
|
190 | 190 | abort(404, 'This course is not synced with external LMS'); |
191 | 191 | } |
192 | 192 | |
193 | 193 | $courseId = $course->lms_id; |
194 | 194 | |
195 | 195 | // if the student is not synced with the LMS, create them |
196 | - if (! $student->user->lms_id) { |
|
196 | + if (!$student->user->lms_id) { |
|
197 | 197 | $this->createUser($student->user); |
198 | 198 | } |
199 | 199 | |
@@ -216,7 +216,7 @@ discard block |
||
216 | 216 | // only process if the course has a teacher |
217 | 217 | if ($course->teacher_id) { |
218 | 218 | // if the teacher doesn't exist on LMS, create them |
219 | - if (! $course->teacher->user->lms_id) { |
|
219 | + if (!$course->teacher->user->lms_id) { |
|
220 | 220 | Log::info('creating user now'); |
221 | 221 | $this->createUser($course->teacher->user); |
222 | 222 | } |
@@ -112,7 +112,7 @@ discard block |
||
112 | 112 | } |
113 | 113 | |
114 | 114 | // get past events for the course |
115 | - $events = $course->events->filter(fn ($value, $key) => Carbon::parse($value->start) < Carbon::now())->sortByDesc('start'); |
|
115 | + $events = $course->events->filter(fn($value, $key) => Carbon::parse($value->start) < Carbon::now())->sortByDesc('start'); |
|
116 | 116 | |
117 | 117 | // if the course has any past events |
118 | 118 | if (($events->count() == 0) || ($course->enrollments()->count() == 0)) { |
@@ -196,7 +196,7 @@ discard block |
||
196 | 196 | |
197 | 197 | public function toggleEventAttendanceStatus(Event $event, Request $request) |
198 | 198 | { |
199 | - if (! backpack_user()->hasPermissionTo('courses.edit')) { |
|
199 | + if (!backpack_user()->hasPermissionTo('courses.edit')) { |
|
200 | 200 | abort(403); |
201 | 201 | } |
202 | 202 | $event->exempt_attendance = (int) $request->status; |
@@ -207,7 +207,7 @@ discard block |
||
207 | 207 | |
208 | 208 | public function toggleCourseAttendanceStatus(Course $course, Request $request) |
209 | 209 | { |
210 | - if (! backpack_user()->hasPermissionTo('courses.edit')) { |
|
210 | + if (!backpack_user()->hasPermissionTo('courses.edit')) { |
|
211 | 211 | abort(403); |
212 | 212 | } |
213 | 213 | $course->exempt_attendance = (int) $request->status; |