academico-sis / academico

app/Models/Course.php

Spacing +6 added lines, -6 removed lines

@@ -203,7 +203,7 @@ 
      */
     public function skills()
     {
-        return $this->evaluationType?->skills()->orderBy('order');
+        return $this->evaluationType ? ->skills()->orderBy('order');
     }
 
     public function books()
@@ -228,7 +228,7 @@ 
      */
     public function getPendingAttendanceAttribute()
     {
-        $events = Event::where(function ($query) {
+        $events = Event::where(function($query) {
             $query->where('course_id', $this->id);
             $query->where('exempt_attendance', '!=', true);
             $query->where('exempt_attendance', '!=', 1);
@@ -361,7 +361,7 @@ 
             foreach ($courseTimes as $courseTime) {
                 $initial = $daysInitials[$courseTime->day];
 
-                if (! isset($parsedCourseTimes[$initial])) {
+                if (!isset($parsedCourseTimes[$initial])) {
                     $parsedCourseTimes[$initial] = [];
                 }
 
@@ -408,7 +408,7 @@ 
     public function getCourseTeacherNameAttribute()
     {
         if ($this->teacher_id) {
-            return $this->teacher?->name;
+            return $this->teacher ? ->name;
         } else {
             return '-';
         }
@@ -441,7 +441,7 @@ 
 
     public function getAcceptsNewStudentsAttribute(): bool
     {
-        if (! $this->spots || $this->spots == 0) {
+        if (!$this->spots || $this->spots == 0) {
             return true;
         }
 
@@ -464,7 +464,7 @@ 
 
     public function eventsWithExpectedAttendance()
     {
-        return $this->events()->where(function ($query) {
+        return $this->events()->where(function($query) {
             $query->where('exempt_attendance', '!=', true);
             $query->where('exempt_attendance', '!=', 1);
             $query->orWhereNull('exempt_attendance');

app/Console/Commands/genderizeStudents.php

Spacing +3 added lines, -3 removed lines

@@ -196,15 +196,15 @@
 
     public function handle()
     {
-        Student::whereNull('gender_id')->chunkById(10, function ($students) {
-            $query = $students->map(fn ($student) => ['id' => $student->id, 'name' => $this->remove_accents(strtok($student->firstname, ' '))]);
+        Student::whereNull('gender_id')->chunkById(10, function($students) {
+            $query = $students->map(fn($student) => ['id' => $student->id, 'name' => $this->remove_accents(strtok($student->firstname, ' '))]);
 
             $response = Http::get('https://api.genderize.io/?name[]='.$query->pluck('name')->implode('&name[]='));
             foreach ($students as $student) {
                 $firstname = $query->firstWhere('id', $student->id)['name'];
 
                 $student->update([
-                    'gender_id' => match ($response->collect()->where('name', $firstname)->where('probability', '>', 0.9)->where('count', '>', 10)->first()['gender'] ?? '-') {
+                    'gender_id' => match($response->collect()->where('name', $firstname)->where('probability', '>', 0.9)->where('count', '>', 10)->first()['gender'] ?? '-') {
                         'male' => 2,
                         'female' => 1,
                         default => null,

app/Models/EnrollmentStatusType.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 
     public function styling()
     {
-        return match ($this->id) {
+        return match($this->id) {
             1 => 'warning',
             2 => 'info',
             default => 'danger',

app/Models/Period.php

Spacing +4 added lines, -4 removed lines

@@ -24,7 +24,7 @@ 
     {
         parent::boot();
 
-        static::addGlobalScope('order', function (Builder $builder) {
+        static::addGlobalScope('order', function(Builder $builder) {
             $builder->orderBy('year_id')->orderBy('order')->orderBy('id');
         });
     }
@@ -154,7 +154,7 @@ 
                 ->where('courses.period_id', $this->id)
                 ->where('enrollments.deleted_at', null)
                 ->where('enrollments.parent_id', null)
-                ->where(function ($query) {
+                ->where(function($query) {
                     return $query->where('students.gender_id', 0)->orWhereNull('students.gender_id');
                 })
                 ->whereIn('enrollments.status_id', ['1', '2']) // filter out cancelled enrollments, todo make this configurable.
@@ -201,7 +201,7 @@ 
     {
         $period = self::where('id', '<', $this->id)->orderBy('id', 'desc')->first();
 
-        if (! $period == null) {
+        if (!$period == null) {
             return $period;
         } else {
             return self::first();
@@ -281,7 +281,7 @@ 
     public function getCoursesWithPendingAttendanceAttribute()
     {
         // get all courses for period and preload relations
-        $courses = $this->courses()->where(function ($query) {
+        $courses = $this->courses()->where(function($query) {
             $query->where('exempt_attendance', '!=', true);
             $query->where('exempt_attendance', '!=', 1);
             $query->orWhereNull('exempt_attendance');

app/Models/Student.php

Spacing +14 added lines, -14 removed lines

@@ -42,8 +42,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);
             });
         });
@@ -51,8 +51,8 @@ 
 
     public function scopeComputedLeadType($query, $leadTypeId)
     {
-        return match ($leadTypeId) {
-            1 => $query->whereHas('enrollments', fn ($query) => $query->whereHas('course', function ($q) {
+        return match($leadTypeId) {
+            1 => $query->whereHas('enrollments', fn($query) => $query->whereHas('course', function($q) {
                 $q->where('period_id', Period::get_default_period()->id);
             })),
 
@@ -61,15 +61,15 @@ 
             4 => $query
                 ->where('lead_type_id', $leadTypeId)
                 ->orWhere(
-                    function ($query) {
+                    function($query) {
                     $query
                         ->whereNull('lead_type_id')
-                        ->whereHas('enrollments', fn ($query) => $query
-                            ->whereHas('course', function ($q) {
+                        ->whereHas('enrollments', fn($query) => $query
+                            ->whereHas('course', function($q) {
                                 $q->where('period_id', '!=', Period::get_default_period()->id);
                             }))
-                        ->whereDoesntHave('enrollments', fn ($query) => $query
-                            ->whereHas('course', function ($q) {
+                        ->whereDoesntHave('enrollments', fn($query) => $query
+                            ->whereHas('course', function($q) {
                                 $q->where('period_id', Period::get_default_period()->id);
                             }));
                 }
@@ -110,7 +110,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)
@@ -120,7 +120,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()
@@ -181,7 +181,7 @@ 
 
     public function getEmailAttribute(): ?string
     {
-        return $this?->user?->email;
+        return $this ? ->user ? ->email;
     }
 
     public function getNameAttribute(): string
@@ -202,7 +202,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;
         }
 
@@ -301,7 +301,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/Models/Year.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
                 ->where('periods.year_id', $this->id)
                 ->whereIn('enrollments.status_id', ['1', '2']) // filter out cancelled enrollments, todo make this configurable.
                 ->where('enrollments.parent_id', null)->where('enrollments.deleted_at', null)
-                ->where(function ($query) {
+                ->where(function($query) {
                     return $query->where('students.gender_id', 0)->orWhereNull('students.gender_id');
                 })
                 ->distinct('student_id')->count('enrollments.student_id');

app/Providers/AppServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -53,12 +53,12 @@
 
         View::composer(
             ['partials.create_new_contact', 'students.edit-contact'],
-            function ($view) {
+            function($view) {
                 $view->with('contact_types', ContactRelationship::all());
             }
         );
 
-        View::composer('partials.add_book_to_student', function ($view) {
+        View::composer('partials.add_book_to_student', function($view) {
             $view->with('books', Book::all());
             $view->with('statuses', EnrollmentStatusType::all());
         });

app/Http/Controllers/Admin/PaymentCrudController.php

Spacing +3 added lines, -3 removed lines

@@ -50,7 +50,7 @@ 
                 'label' => __('Due Date'),
             ],
             false,
-            function ($value) { // if the filter is active, apply these constraints
+            function($value) { // if the filter is active, apply these constraints
                 $this->crud->addClause('where', 'date', '>=', Carbon::parse($value)->firstOfMonth());
                 $this->crud->addClause('where', 'date', '<=', Carbon::parse($value)->lastOfMonth());
             }
@@ -92,11 +92,11 @@ 
     {
         $payment = Payment::findOrFail($id);
 
-        if (! backpack_user()->can('enrollments.edit')) {
+        if (!backpack_user()->can('enrollments.edit')) {
             abort(403);
         }
 
-        if (! $payment->invoice) {
+        if (!$payment->invoice) {
             abort(404, 'No enrollment found for this payment');
         }
 

app/Http/Controllers/Admin/EnrollmentCrudController.php

Spacing +19 added lines, -19 removed lines

@@ -105,7 +105,7 @@ 
                 'name' => 'id',
                 'label' => 'ID',
                 'wrapper' => [
-                    'element' => function ($crud, $column, $entry) {
+                    'element' => function($crud, $column, $entry) {
                         return $entry->status_id > 2 ? 'del' : 'span';
                     },
                 ],
@@ -116,7 +116,7 @@ 
                 'type' => 'text',
                 'name' => 'student.idnumber',
                 'wrapper' => [
-                    'element' => function ($crud, $column, $entry) {
+                    'element' => function($crud, $column, $entry) {
                         return $entry->status_id > 2 ? 'del' : 'span';
                     },
                 ],
@@ -129,13 +129,13 @@ 
                 'label' => __('Last Name'),
                 'type' => 'relationship',
                 'wrapper' => [
-                    'element' => function ($crud, $column, $entry) {
+                    'element' => function($crud, $column, $entry) {
                         return $entry->status_id > 2 ? 'del' : 'span';
                     },
                 ],
-                'searchLogic' => function ($query, $column, $searchTerm) {
-                    $query->orWhereHas('student', function ($q) use ($searchTerm) {
-                        $q->whereHas('user', function ($q) use ($searchTerm) {
+                'searchLogic' => function($query, $column, $searchTerm) {
+                    $query->orWhereHas('student', function($q) use ($searchTerm) {
+                        $q->whereHas('user', function($q) use ($searchTerm) {
                             $q->where('lastname', 'like', '%'.$searchTerm.'%');
                         });
                     });
@@ -149,13 +149,13 @@ 
                 'label' => __('First Name'),
                 'type' => 'relationship',
                 'wrapper' => [
-                    'element' => function ($crud, $column, $entry) {
+                    'element' => function($crud, $column, $entry) {
                         return $entry->status_id > 2 ? 'del' : 'span';
                     },
                 ],
-                'searchLogic' => function ($query, $column, $searchTerm) {
-                    $query->orWhereHas('student', function ($q) use ($searchTerm) {
-                        $q->whereHas('user', function ($q) use ($searchTerm) {
+                'searchLogic' => function($query, $column, $searchTerm) {
+                    $query->orWhereHas('student', function($q) use ($searchTerm) {
+                        $q->whereHas('user', function($q) use ($searchTerm) {
                             $q->where('firstname', 'like', '%'.$searchTerm.'%');
                         });
                     });
@@ -201,7 +201,7 @@ 
                 'model' => EnrollmentStatusType::class,
                 'wrapper' => [
                     'element' => 'span',
-                    'class' => function ($crud, $column, $entry) {
+                    'class' => function($crud, $column, $entry) {
                         return 'badge badge-pill badge-'.$entry->enrollmentStatus->styling();
                     },
                 ],
@@ -258,8 +258,8 @@ 
                     'type' => 'select2_multiple',
                     'label'=> __('Status'),
                 ],
-                fn () => EnrollmentStatusType::all()->pluck('name', 'id')->toArray(),
-                function ($values) {
+                fn() => EnrollmentStatusType::all()->pluck('name', 'id')->toArray(),
+                function($values) {
                 foreach (json_decode($values, null, 512, JSON_THROW_ON_ERROR) as $value) {
                     CRUD::addClause('orWhere', 'status_id', $value);
                 }
@@ -270,7 +270,7 @@ 
                 'name' => 'period_id',
                 'type' => 'select2',
                 'label'=> __('Period'),
-            ], fn () => Period::all()->pluck('name', 'id')->toArray(), function ($value) {
+            ], fn() => Period::all()->pluck('name', 'id')->toArray(), function($value) {
                 CRUD::addClause('period', $value);
             });
 
@@ -280,12 +280,12 @@ 
                     'type' => 'select2',
                     'label'=> __('Scholarship'),
                 ],
-                fn () => Scholarship::all()->pluck('name', 'id')->toArray(),
-                function ($value) { // if the filter is active
+                fn() => Scholarship::all()->pluck('name', 'id')->toArray(),
+                function($value) { // if the filter is active
                     if ($value == 'all') {
                         CRUD::addClause('whereHas', 'scholarships');
                     } else {
-                        CRUD::addClause('whereHas', 'scholarships', function ($q) use ($value) {
+                        CRUD::addClause('whereHas', 'scholarships', function($q) use ($value) {
                             $q->where('scholarships.id', $value);
                         });
                     }
@@ -305,7 +305,7 @@ 
             ->get();
 
         // get related comments
-        $comments = $enrollment->invoices->map(fn (Invoice $invoice) => $invoice->comments)->flatten()->concat($enrollment->comments);
+        $comments = $enrollment->invoices->map(fn(Invoice $invoice) => $invoice->comments)->flatten()->concat($enrollment->comments);
 
         $scholarships = Scholarship::all();
 
@@ -336,7 +336,7 @@ 
             'model'     => \App\Models\Course::class,
             'attribute' => 'name',
 
-            'options'   => (fn ($query) => $query->orderBy('level_id', 'ASC')->where('period_id', $this->crud->getCurrentEntry()->course->period_id)->get()),
+            'options'   => (fn($query) => $query->orderBy('level_id', 'ASC')->where('period_id', $this->crud->getCurrentEntry()->course->period_id)->get()),
         ]);
 
         CRUD::addField(array_merge([