@@ -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 | |
@@ -52,11 +52,11 @@ |
||
| 52 | 52 | } |
| 53 | 53 | |
| 54 | 54 | View::composer( |
| 55 | - ['partials.create_new_contact', 'students.edit-contact'], function ($view) { |
|
| 55 | + ['partials.create_new_contact', 'students.edit-contact'], function($view) { |
|
| 56 | 56 | $view->with('contact_types', ContactRelationship::all()); |
| 57 | 57 | }); |
| 58 | 58 | |
| 59 | - View::composer('partials.add_book_to_student', function ($view) { |
|
| 59 | + View::composer('partials.add_book_to_student', function($view) { |
|
| 60 | 60 | $view->with('books', Book::all()); |
| 61 | 61 | $view->with('statuses', EnrollmentStatusType::all()); |
| 62 | 62 | }); |
@@ -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 | } |