academico-sis / academico

app/Http/Controllers/Admin/EventCrudController.php

Spacing +9 added lines, -9 removed lines

@@ -116,13 +116,13 @@ 
         ]);
 
         CRUD::addFilter(
-            [ // daterange filter
+            [// daterange filter
                 'type' => 'date_range',
                 'name' => 'from_to',
                 'label' => __('Date range'),
             ],
             false,
-            function ($value) { // if the filter is active, apply these constraints
+            function($value) { // if the filter is active, apply these constraints
                 $dates = json_decode($value, null, 512, JSON_THROW_ON_ERROR);
 
                 if ($dates->from) {
@@ -141,10 +141,10 @@ 
                 'label' => __('Events with no course'),
             ],
             false,
-            function () { // if the filter is active, apply these constraints
+            function() { // if the filter is active, apply these constraints
                 $this->crud->query->where('course_id', null);
             },
-            function () { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
+            function() { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
             }
         );
 
@@ -155,22 +155,22 @@ 
                 'label' => __('Events with no teacher'),
             ],
             false,
-            function () { // if the filter is active, apply these constraints
+            function() { // if the filter is active, apply these constraints
                 CRUD::addClause('unassigned');
             }
         );
 
         CRUD::addFilter(
-            [ // select2 filter
+            [// select2 filter
                 'name' => 'teacher_id',
                 'type' => 'select2',
                 'label' => __('Teacher'),
             ],
-            fn () => Teacher::all()->pluck('name', 'id')->toArray(),
-            function ($value) {
+            fn() => Teacher::all()->pluck('name', 'id')->toArray(),
+            function($value) {
                 CRUD::addClause('where', 'teacher_id', $value);
             },
-            function () { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
+            function() { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
             }
         );
     }

app/Http/Controllers/Admin/TeacherCrudController.php

Spacing +11 added lines, -11 removed lines

@@ -46,10 +46,10 @@ 
                 'attribute' => 'lastname',
                 'model' => User::class,
                 'orderable' => true,
-                'orderLogic' => fn ($query, $column, $columnDirection) => $query->leftJoin('users', 'users.id', '=', 'students.id')
+                'orderLogic' => fn($query, $column, $columnDirection) => $query->leftJoin('users', 'users.id', '=', 'students.id')
                     ->orderBy('users.lastname', $columnDirection)->select('students.*'),
-                'searchLogic' => function ($query, $column, $searchTerm) {
-                    $query->orWhereHas('user', function ($q) use ($searchTerm) {
+                'searchLogic' => function($query, $column, $searchTerm) {
+                    $query->orWhereHas('user', function($q) use ($searchTerm) {
                         $q->where('lastname', 'like', '%'.$searchTerm.'%');
                     });
                 },
@@ -64,10 +64,10 @@ 
                 'attribute' => 'firstname',
                 'model' => User::class,
                 'orderable' => true,
-                'orderLogic' => fn ($query, $column, $columnDirection) => $query->leftJoin('users', 'users.id', '=', 'teachers.id')
+                'orderLogic' => fn($query, $column, $columnDirection) => $query->leftJoin('users', 'users.id', '=', 'teachers.id')
                     ->orderBy('users.firstname', $columnDirection)->select('teachers.*'),
-                'searchLogic' => function ($query, $column, $searchTerm) {
-                    $query->orWhereHas('user', function ($q) use ($searchTerm) {
+                'searchLogic' => function($query, $column, $searchTerm) {
+                    $query->orWhereHas('user', function($q) use ($searchTerm) {
                         $q->where('firstname', 'like', '%'.$searchTerm.'%');
                     });
                 },
@@ -81,10 +81,10 @@ 
                 'attribute' => 'email',
                 'model' => User::class,
                 'orderable' => true,
-                'orderLogic' => fn ($query, $column, $columnDirection) => $query->leftJoin('users', 'users.id', '=', 'teachers.id')
+                'orderLogic' => fn($query, $column, $columnDirection) => $query->leftJoin('users', 'users.id', '=', 'teachers.id')
                     ->orderBy('users.email', $columnDirection)->select('teachers.*'),
-                'searchLogic' => function ($query, $column, $searchTerm) {
-                    $query->orWhereHas('user', function ($q) use ($searchTerm) {
+                'searchLogic' => function($query, $column, $searchTerm) {
+                    $query->orWhereHas('user', function($q) use ($searchTerm) {
                         $q->where('email', 'like', '%'.$searchTerm.'%');
                     });
                 },
@@ -181,8 +181,8 @@ 
         $username_parts = array_filter(explode(' ', strtolower($fullName)));
         $username_parts = array_slice($username_parts, -2);
 
-        $part1 = (! empty($username_parts[0])) ? substr($username_parts[0], 0, 3) : '';
-        $part2 = (! empty($username_parts[1])) ? substr($username_parts[1], 0, 8) : '';
+        $part1 = (!empty($username_parts[0])) ? substr($username_parts[0], 0, 3) : '';
+        $part2 = (!empty($username_parts[1])) ? substr($username_parts[1], 0, 8) : '';
         $part3 = random_int(999, 9999);
 
         //str_shuffle to randomly shuffle all characters

app/Http/Controllers/Admin/InvoiceCrudController.php

Spacing +4 added lines, -4 removed lines

@@ -32,7 +32,7 @@ 
         CRUD::setModel(Invoice::class);
         CRUD::setRoute(config('backpack.base.route_prefix').'/invoice');
         CRUD::setEntityNameStrings(__('invoice'), __('invoices'));
-        if (! config('invoicing.price_categories_enabled')) {
+        if (!config('invoicing.price_categories_enabled')) {
             $this->crud->addButtonFromView('top', 'createInvoice', 'createInvoice', 'start');
         }
 
@@ -107,7 +107,7 @@ 
                 'label' => __('Date range'),
             ],
             false,
-            function ($value) { // if the filter is active, apply these constraints
+            function($value) { // if the filter is active, apply these constraints
                 $dates = json_decode($value, null, 512, JSON_THROW_ON_ERROR);
 
                 if ($dates->from) {
@@ -233,7 +233,7 @@ 
     {
         $invoice = Invoice::findOrFail($id)->load('payments');
 
-        if (! backpack_user()->can('enrollments.edit')) {
+        if (!backpack_user()->can('enrollments.edit')) {
             abort(403);
         }
 
@@ -241,7 +241,7 @@ 
             'invoice' => $invoice,
             'availablePaymentMethods' => Paymentmethod::all(),
             'editable' => true,
-            'enrollment' => $invoice->enrollments->first()?->product,
+            'enrollment' => $invoice->enrollments->first() ? ->product,
             'comments' => $invoice->comments,
             'afterSuccessUrl' => $invoice->enrollments->count() > 0 ? "/enrollment/{$invoice->enrollments->first()->product_id}/show" : '/invoice', // TODO fix this, an invoice can theoretically contain several enrollments
         ]);

app/Http/Controllers/Admin/CourseCrudController.php

Spacing +24 added lines, -24 removed lines

@@ -49,7 +49,7 @@ 
         CRUD::addClause('internal');
         $permissions = backpack_user()->getAllPermissions();
 
-        if (! $permissions->contains('name', 'courses.edit')) {
+        if (!$permissions->contains('name', 'courses.edit')) {
             CRUD::denyAccess(['update', 'create']);
         }
 
@@ -59,7 +59,7 @@ 
 
         CRUD::addButtonFromView('line', 'children_badge', 'children_badge', 'beginning');
 
-        if (! $permissions->contains('name', 'courses.delete')) {
+        if (!$permissions->contains('name', 'courses.delete')) {
             CRUD::denyAccess(['delete']);
         }
 
@@ -179,11 +179,11 @@ 
                 'type' => 'select2',
                 'label' => __('Rhythm'),
             ],
-            fn () => Rhythm::all()->pluck('name', 'id')->toArray(),
-            function ($value) {
+            fn() => Rhythm::all()->pluck('name', 'id')->toArray(),
+            function($value) {
                 CRUD::addClause('where', 'rhythm_id', $value);
             },
-            function () {
+            function() {
                 // if the filter is NOT active (the GET parameter "checkbox" does not exit)
             }
         );
@@ -194,11 +194,11 @@ 
                 'type' => 'select2',
                 'label' => __('Teacher'),
             ],
-            fn () => Teacher::all()->pluck('name', 'id')->toArray(),
-            function ($value) {
+            fn() => Teacher::all()->pluck('name', 'id')->toArray(),
+            function($value) {
                 CRUD::addClause('where', 'teacher_id', $value);
             },
-            function () {
+            function() {
                 // if the filter is NOT active (the GET parameter "checkbox" does not exit)
             }
         );
@@ -209,11 +209,11 @@ 
                 'type' => 'select2',
                 'label' => __('Level'),
             ],
-            fn () => Level::all()->pluck('name', 'id')->toArray(),
-            function ($value) {
+            fn() => Level::all()->pluck('name', 'id')->toArray(),
+            function($value) {
                 CRUD::addClause('where', 'level_id', $value);
             },
-            function () {
+            function() {
                 // if the filter is NOT active (the GET parameter "checkbox" does not exit)
             }
         );
@@ -224,11 +224,11 @@ 
                 'type' => 'select2',
                 'label' => __('Period'),
             ],
-            fn () => \App\Models\Period::all()->sortByDesc('id')->pluck('name', 'id')->toArray(),
-            function ($value) {
+            fn() => \App\Models\Period::all()->sortByDesc('id')->pluck('name', 'id')->toArray(),
+            function($value) {
                 CRUD::addClause('where', 'period_id', $value);
             },
-            function () { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
+            function() { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
                 $period = \App\Models\Period::get_default_period()->id;
                 CRUD::addClause('where', 'period_id', $period);
                 $this->crud->getRequest()->request->add(['period_id' => $period]); // to make the filter look active
@@ -236,13 +236,13 @@ 
         );
 
         CRUD::addFilter(
-            [ // add a "simple" filter called Draft
+            [// add a "simple" filter called Draft
                 'type' => 'simple',
                 'name' => 'parent',
                 'label' => __('Hide Children Courses'),
             ],
             false,
-            function () {
+            function() {
                 CRUD::addClause('parent');
             }
         );
@@ -254,7 +254,7 @@ 
                 'label' => __('Start'),
             ],
             false,
-            function ($value) { // if the filter is active, apply these constraints
+            function($value) { // if the filter is active, apply these constraints
                 $dates = json_decode($value, null, 512, JSON_THROW_ON_ERROR);
                 $this->crud->addClause('where', 'start_date', '>=', $dates->from);
                 $this->crud->addClause('where', 'start_date', '<=', $dates->to.' 23:59:59');
@@ -268,7 +268,7 @@ 
                 'label' => __('End'),
             ],
             false,
-            function ($value) { // if the filter is active, apply these constraints
+            function($value) { // if the filter is active, apply these constraints
                 $dates = json_decode($value, null, 512, JSON_THROW_ON_ERROR);
                 $this->crud->addClause('where', 'end_date', '>=', $dates->from);
                 $this->crud->addClause('where', 'end_date', '<=', $dates->to.' 23:59:59');
@@ -367,7 +367,7 @@ 
                 'tab' => __('Course info'),
             ],
 
-            [   // repeatable
+            [// repeatable
                 'name' => 'sublevels',
                 'label' => __('Course sublevels'),
                 'type' => 'repeatable',
@@ -527,7 +527,7 @@ 
                 'default' => Period::get_enrollments_period()->end,
             ],
 
-            [   // repeatable
+            [// repeatable
                 'name' => 'times',
                 'label' => __('Course Schedule'),
                 'type' => 'repeatable',
@@ -590,14 +590,14 @@ 
                 'init_rows' => 0,
             ],
 
-            [   // view
+            [// view
                 'name' => 'custom-ajax-button',
                 'type' => 'view',
                 'view' => 'courses/schedule-preset-alert',
                 'tab' => __('Schedule'),
             ],
 
-            [   // select_from_array
+            [// select_from_array
                 'name' => 'schedulepreset',
                 'label' => __('Schedule Preset'),
                 'type' => 'select_from_array',
@@ -621,7 +621,7 @@ 
         }
 
         if ($this->crud->getCurrentEntry()->children->count() > 0) {
-            CRUD::addField([   // view
+            CRUD::addField([// view
                 'name' => 'custom-ajax-button',
                 'type' => 'view',
                 'view' => 'courses/parent-course-alert',
@@ -629,7 +629,7 @@ 
         }
 
         if ($this->crud->getCurrentEntry()->parent_course_id !== null) {
-            CRUD::addField([   // view
+            CRUD::addField([// view
                 'name' => 'custom-ajax-button',
                 'type' => 'view',
                 'view' => 'courses/child-course-alert',

app/Http/Controllers/Admin/StudentCrudController.php

Spacing +24 added lines, -24 removed lines

@@ -96,10 +96,10 @@ 
                 'attribute' => 'lastname',
                 'model' => User::class,
                 'orderable' => true,
-                'orderLogic' => fn ($query, $column, $columnDirection) => $query->leftJoin('users', 'users.id', '=', 'students.id')
+                'orderLogic' => fn($query, $column, $columnDirection) => $query->leftJoin('users', 'users.id', '=', 'students.id')
                     ->orderBy('users.lastname', $columnDirection)->select('students.*'),
-                'searchLogic' => function ($query, $column, $searchTerm) {
-                    $query->orWhereHas('user', function ($q) use ($searchTerm) {
+                'searchLogic' => function($query, $column, $searchTerm) {
+                    $query->orWhereHas('user', function($q) use ($searchTerm) {
                         $q->where('lastname', 'like', '%'.$searchTerm.'%');
                     });
                 },
@@ -114,10 +114,10 @@ 
                 'attribute' => 'firstname',
                 'model' => User::class,
                 'orderable' => true,
-                'orderLogic' => fn ($query, $column, $columnDirection) => $query->leftJoin('users', 'users.id', '=', 'students.id')
+                'orderLogic' => fn($query, $column, $columnDirection) => $query->leftJoin('users', 'users.id', '=', 'students.id')
                     ->orderBy('users.firstname', $columnDirection)->select('students.*'),
-                'searchLogic' => function ($query, $column, $searchTerm) {
-                    $query->orWhereHas('user', function ($q) use ($searchTerm) {
+                'searchLogic' => function($query, $column, $searchTerm) {
+                    $query->orWhereHas('user', function($q) use ($searchTerm) {
                         $q->where('firstname', 'like', '%'.$searchTerm.'%');
                     });
                 },
@@ -131,10 +131,10 @@ 
                 'attribute' => 'email',
                 'model' => User::class,
                 'orderable' => true,
-                'orderLogic' => fn ($query, $column, $columnDirection) => $query->leftJoin('users', 'users.id', '=', 'students.id')
+                'orderLogic' => fn($query, $column, $columnDirection) => $query->leftJoin('users', 'users.id', '=', 'students.id')
                     ->orderBy('users.email', $columnDirection)->select('students.*'),
-                'searchLogic' => function ($query, $column, $searchTerm) {
-                    $query->orWhereHas('user', function ($q) use ($searchTerm) {
+                'searchLogic' => function($query, $column, $searchTerm) {
+                    $query->orWhereHas('user', function($q) use ($searchTerm) {
                         $q->where('email', 'like', '%'.$searchTerm.'%');
                     });
                 },
@@ -181,28 +181,28 @@ 
         ]);
 
         CRUD::addFilter(
-            [ // select2 filter
+            [// select2 filter
                 'name' => 'enrolled',
                 'type' => 'select2',
                 'label' => __('Is Enrolled in'),
             ],
-            fn () => Period::all()->pluck('name', 'id')->toArray(),
-            function ($value) {
-                $this->crud->query = $this->crud->query->whereHas('enrollments', fn ($query) => $query->whereHas('course', function ($q) use ($value) {
+            fn() => Period::all()->pluck('name', 'id')->toArray(),
+            function($value) {
+                $this->crud->query = $this->crud->query->whereHas('enrollments', fn($query) => $query->whereHas('course', function($q) use ($value) {
                     $q->where('period_id', $value);
                 }));
             },
-            function () { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
+            function() { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
             }
         );
 
-        CRUD::addFilter([ // select2_multiple filter
+        CRUD::addFilter([// select2_multiple filter
             'name' => 'notenrolled',
             'type' => 'select2_multiple',
             'label' => __('Is Not Enrolled in'),
-        ], fn () => Period::all()->pluck('name', 'id')->toArray(), function ($values) {
+        ], fn() => Period::all()->pluck('name', 'id')->toArray(), function($values) {
             foreach (json_decode($values, null, 512, JSON_THROW_ON_ERROR) as $value) {
-                $this->crud->query = $this->crud->query->whereDoesntHave('enrollments', fn ($query) => $query->whereHas('course', function ($q) use ($value) {
+                $this->crud->query = $this->crud->query->whereDoesntHave('enrollments', fn($query) => $query->whereHas('course', function($q) use ($value) {
                     $q->where('period_id', $value);
                 }));
             }
@@ -214,8 +214,8 @@ 
                 'type' => 'select2',
                 'label' => __('New In'),
             ],
-            fn () => Period::all()->pluck('name', 'id')->toArray(),
-            function ($value) {
+            fn() => Period::all()->pluck('name', 'id')->toArray(),
+            function($value) {
                 CRUD::addClause('newInPeriod', $value);
             }
         );
@@ -225,7 +225,7 @@ 
             'name' => 'institution_id',
             'type' => 'select2',
             'label' => __('Institution'),
-        ], fn () => Institution::all()->pluck('name', 'id')->toArray(), function ($value) {
+        ], fn() => Institution::all()->pluck('name', 'id')->toArray(), function($value) {
             $this->crud->addClause('where', 'institution_id', $value);
         });
 
@@ -233,7 +233,7 @@ 
             'name' => 'lead_type_id',
             'type' => 'select2',
             'label' => __('Lead Status'),
-        ], fn () => LeadType::all()->pluck('name', 'id')->toArray(), function ($value) {
+        ], fn() => LeadType::all()->pluck('name', 'id')->toArray(), function($value) {
             if ($value === '4') {
                 $this->crud->query = $this->crud->query->where('lead_type_id', $value)->orWhere('lead_type_id', null);
             } else {
@@ -366,8 +366,8 @@ 
         $username_parts = array_filter(explode(' ', strtolower($fullName)));
         $username_parts = array_slice($username_parts, -2);
 
-        $part1 = (! empty($username_parts[0])) ? substr($username_parts[0], 0, 3) : '';
-        $part2 = (! empty($username_parts[1])) ? substr($username_parts[1], 0, 8) : '';
+        $part1 = (!empty($username_parts[0])) ? substr($username_parts[0], 0, 3) : '';
+        $part2 = (!empty($username_parts[1])) ? substr($username_parts[1], 0, 8) : '';
         $part3 = random_int(999, 9999);
 
         //str_shuffle to randomly shuffle all characters
@@ -455,7 +455,7 @@ 
     {
         $student = Student::findOrFail($student);
 
-        if (! backpack_user()->isTeacher() && ! backpack_user()->can('enrollments.view')) {
+        if (!backpack_user()->isTeacher() && !backpack_user()->can('enrollments.view')) {
             abort(403);
         }
 

app/Models/Student.php

Spacing +7 added lines, -7 removed lines

@@ -45,8 +45,8 @@ 
 
     public function scopeEnrolled($query)
     {
-        return $query->whereHas('enrollments', function ($q) {
-            return $q->whereHas('course', function ($q) {
+        return $query->whereHas('enrollments', function($q) {
+            return $q->whereHas('course', function($q) {
                 return $q->where('period_id', Period::get_default_period()->id);
             });
         });
@@ -83,7 +83,7 @@ 
 
         return $this->hasMany(Attendance::class)
         ->where('attendance_type_id', 4) // absence
-        ->whereHas('event', fn ($q) => $q->whereHas('course', fn ($c) => $c->where('period_id', $period->id)));
+        ->whereHas('event', fn($q) => $q->whereHas('course', fn($c) => $c->where('period_id', $period->id)));
     }
 
     public function periodAttendance(Period $period = null)
@@ -93,7 +93,7 @@ 
         }
 
         return $this->hasMany(Attendance::class)
-        ->whereHas('event', fn ($q) => $q->whereHas('course', fn ($c) => $c->where('period_id', $period->id)));
+        ->whereHas('event', fn($q) => $q->whereHas('course', fn($c) => $c->where('period_id', $period->id)));
     }
 
     public function contacts()
@@ -154,7 +154,7 @@ 
 
     public function getEmailAttribute(): ?string
     {
-        return $this?->user?->email;
+        return $this ? ->user ? ->email;
     }
 
     public function getNameAttribute(): string
@@ -175,7 +175,7 @@ 
     public function getIsEnrolledAttribute()
     {
         // if the student is currently enrolled
-        if ($this->enrollments()->whereHas('course', fn ($q) => $q->where('period_id', Period::get_default_period()->id))->count() > 0) {
+        if ($this->enrollments()->whereHas('course', fn($q) => $q->where('period_id', Period::get_default_period()->id))->count() > 0) {
             return 1;
         }
 
@@ -274,7 +274,7 @@ 
 
     public function getImageAttribute(): ?string
     {
-        return $this->getMedia('profile-picture')->last()?->getUrl('thumb');
+        return $this->getMedia('profile-picture')->last() ? ->getUrl('thumb');
     }
 
     public function setImageAttribute($value)

app/Listeners/UpdateCourseEvents.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 
             // if there is a coursetime for today, create the event
             if ($course->times->contains('day', $courseStartDate->format('w'))) {
-                Event::create(['course_id' => $course->id, 'teacher_id' => $course->teacher_id, 'room_id' => $course->room_id, 'start' => $courseStartDate->setTimeFromTimeString($course->times->where('day', $courseStartDate->format('w'))->first()->start)->toDateTimeString(), 'end' => $courseStartDate->setTimeFromTimeString($course->times->where('day', $courseStartDate->format('w'))->first()->end)->toDateTimeString(), 'name' => $course->name, 'course_time_id' => $course->times->where('day', $courseStartDate->format('w'))->first()->id, 'exempt_attendance' => $course->exempt_attendance,]);
+                Event::create(['course_id' => $course->id, 'teacher_id' => $course->teacher_id, 'room_id' => $course->room_id, 'start' => $courseStartDate->setTimeFromTimeString($course->times->where('day', $courseStartDate->format('w'))->first()->start)->toDateTimeString(), 'end' => $courseStartDate->setTimeFromTimeString($course->times->where('day', $courseStartDate->format('w'))->first()->end)->toDateTimeString(), 'name' => $course->name, 'course_time_id' => $course->times->where('day', $courseStartDate->format('w'))->first()->id, 'exempt_attendance' => $course->exempt_attendance, ]);
             }
             $courseStartDate->addDay();
         }

app/Listeners/UpdateTeacherEvents.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 
 class UpdateTeacherEvents
 {
-    public function handle(LeaveCreated|LeaveUpdated $event) {
+    public function handle(LeaveCreated | LeaveUpdated $event) {
         $teacher = $event->leave->teacher;
         foreach ($teacher->events()->whereDate('start', '>=', $event->leave->date)->whereDate('end', '<=', $event->leave->date)->get() as $event) {
             $event->teacher_id = null;

app/Listeners/UpdateUsername.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 
 class UpdateUsername
 {
-    public function handle(TeacherUpdated|StudentUpdated|UserUpdated $event) {
+    public function handle(TeacherUpdated | StudentUpdated | UserUpdated $event) {
         if (User::whereUsername($event->user->email)->count() === 0) {
             $event->user->update(['username' => $event->user->email]);
         }