academico-sis / academico

app/Http/Controllers/Admin/EventCrudController.php

Indentation +15 added lines, -15 removed lines

@@ -119,15 +119,15 @@ 
             ],
             false,
             function ($value) { // if the filter is active, apply these constraints
-              $dates = json_decode($value, null, 512, JSON_THROW_ON_ERROR);
-
-              if ($dates->from) {
-                  CRUD::addClause('where', 'start', '>=', $dates->from);
-              }
-              if ($dates->to) {
-                  CRUD::addClause('where', 'start', '<=', $dates->to.' 23:59:59');
-              }
-          }
+                $dates = json_decode($value, null, 512, JSON_THROW_ON_ERROR);
+
+                if ($dates->from) {
+                    CRUD::addClause('where', 'start', '>=', $dates->from);
+                }
+                if ($dates->to) {
+                    CRUD::addClause('where', 'start', '<=', $dates->to.' 23:59:59');
+                }
+            }
         );
 
         CRUD::addFilter(
@@ -138,10 +138,10 @@ 
             ],
             false,
             function ($value) { // if the filter is active, apply these constraints
-              $this->crud->query->where('course_id', null);
-          },
+                $this->crud->query->where('course_id', null);
+            },
             function () { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
-          }
+            }
         );
 
         CRUD::addFilter(
@@ -152,8 +152,8 @@ 
             ],
             false,
             function ($value) { // if the filter is active, apply these constraints
-              CRUD::addClause('unassigned');
-          }
+                CRUD::addClause('unassigned');
+            }
         );
 
         CRUD::addFilter(
@@ -167,7 +167,7 @@ 
             CRUD::addClause('where', 'teacher_id', $value);
         },
             function () { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
-          }
+            }
         );
     }
 

Spacing +9 added lines, -9 removed lines

@@ -112,13 +112,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) {
@@ -137,10 +137,10 @@ 
                 'label'=> __('Events with no course'),
             ],
             false,
-            function ($value) { // if the filter is active, apply these constraints
+            function($value) { // 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)
           }
         );
 
@@ -151,22 +151,22 @@ 
                 'label'=> __('Events with no teacher'),
             ],
             false,
-            function ($value) { // if the filter is active, apply these constraints
+            function($value) { // 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) { // if the filter is active
+            fn() => Teacher::all()->pluck('name', 'id')->toArray(),
+            function($value) { // if the filter is active
             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/ResultCrudController.php

Indentation +3 added lines, -3 removed lines

@@ -58,7 +58,7 @@ 
                             $q->where('firstname', 'like', '%'.$searchTerm.'%')
                             ->orWhere('lastname', 'like', '%'.$searchTerm.'%')
                             ->orWhere('email', 'like', '%'.$searchTerm.'%')
-                          ->orWhere('idnumber', 'like', '%'.$searchTerm.'%');
+                            ->orWhere('idnumber', 'like', '%'.$searchTerm.'%');
                         });
                     });
                 },
@@ -95,8 +95,8 @@ 
             ],
             false,
             function () {
-              CRUD::addClause('noResult');
-          }
+                CRUD::addClause('noResult');
+            }
         );
 
         CRUD::addFilter(

Spacing +9 added lines, -9 removed lines

@@ -52,9 +52,9 @@ 
                 'type' => 'select',
                 'name' => 'student',
                 'attribute' => 'name',
-                '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.'%')
                             ->orWhere('lastname', 'like', '%'.$searchTerm.'%')
                             ->orWhere('email', 'like', '%'.$searchTerm.'%')
@@ -94,7 +94,7 @@ 
                 'label'=> __('No Result'),
             ],
             false,
-            function () {
+            function() {
               CRUD::addClause('noResult');
           }
         );
@@ -106,7 +106,7 @@ 
                 'label'=> __('Hide Parents'),
             ],
             false,
-            function () {
+            function() {
                 CRUD::addClause('real');
             }
         );
@@ -115,16 +115,16 @@ 
             'name' => 'period_id',
             'type' => 'select2',
             'label'=> __('Period'),
-        ], fn () => Period::all()->pluck('name', 'id')->toArray(), function ($value) { // if the filter is active
+        ], fn() => Period::all()->pluck('name', 'id')->toArray(), function($value) { // if the filter is active
             CRUD::addClause('period', $value);
         });
 
-        CRUD::addFilter([ // select2_multiple filter
+        CRUD::addFilter([// select2_multiple filter
             'name' => 'result',
             'type' => 'select2',
             'label'=> __('Result'),
-        ], fn () => ResultType::all()->pluck('name', 'id')->toArray(), function ($value) { // if the filter is active
-            $this->crud->query = $this->crud->query->whereHas('result', function ($query) use ($value) {
+        ], fn() => ResultType::all()->pluck('name', 'id')->toArray(), function($value) { // if the filter is active
+            $this->crud->query = $this->crud->query->whereHas('result', function($query) use ($value) {
                 $query->where('result_type_id', $value);
             });
         });

app/Http/Controllers/ReportController.php

Spacing +3 added lines, -3 removed lines

@@ -63,7 +63,7 @@ 
         $year_data = [];
         $years = []; // New array
 
-        if (! isset($request->period)) {
+        if (!isset($request->period)) {
             $startperiod = Period::find(Config::where('name', 'first_external_period')->first()->value ?? Period::first()->id);
         } else {
             $startperiod = Period::find($request->period);
@@ -203,7 +203,7 @@ 
             ->where('id', '>=', $startperiod->id)
             ->get()
             ->groupBy('year_id')
-            ->map(function ($yearData) {
+            ->map(function($yearData) {
                 $yearPeriods = [];
 
                 foreach ($yearData as $period) {
@@ -307,7 +307,7 @@ 
 
     private function getStartperiod(Request $request)
     {
-        if (! isset($request->period)) {
+        if (!isset($request->period)) {
             $startperiod = Period::find(Config::where('name', 'first_period')->first()->value);
         } else {
             $startperiod = Period::find($request->period);

app/Http/Controllers/Admin/EnrollmentCrudController.php

Spacing +18 added lines, -18 removed lines

@@ -92,7 +92,7 @@ 
                 'name' => 'id',
                 'label' => 'ID',
                 'wrapper' => [
-                    'element' => function ($crud, $column, $entry) {
+                    'element' => function($crud, $column, $entry) {
                         return $entry->status_id > 2 ? 'del' : 'span';
                     },
                 ],
@@ -103,7 +103,7 @@ 
                 'type' => 'text',
                 'name' => 'student.idnumber',
                 'wrapper' => [
-                    'element' => function ($crud, $column, $entry) {
+                    'element' => function($crud, $column, $entry) {
                         return $entry->status_id > 2 ? 'del' : 'span';
                     },
                 ],
@@ -116,13 +116,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.'%');
                         });
                     });
@@ -136,13 +136,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.'%');
                         });
                     });
@@ -188,7 +188,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();
                     },
                 ],
@@ -236,8 +236,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);
                 }
@@ -248,7 +248,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);
             });
 
@@ -258,12 +258,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);
                         });
                     }
@@ -303,7 +303,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([

app/Http/Controllers/Admin/InvoiceCrudController.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,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');
         }
     }
@@ -114,7 +114,7 @@ 
     {
         $invoice = Invoice::findOrFail($id)->load('payments');
 
-        if (! backpack_user()->can('enrollments.edit')) {
+        if (!backpack_user()->can('enrollments.edit')) {
             abort(403);
         }
 

app/Http/Controllers/InvoiceController.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
 
         // persist the products
         foreach ($request->products as $f => $product) {
-            $productType = match ($product['type']) {
+            $productType = match($product['type']) {
                 'enrollment' => Enrollment::class,
                 'scheduledPayment' => ScheduledPayment::class,
                 'fee' => Fee::class,

app/Models/Enrollment.php

Spacing +8 added lines, -8 removed lines

@@ -60,11 +60,11 @@ 
     public function scopeWithoutChildren($query)
     {
         return $query
-            ->where(function ($query) {
+            ->where(function($query) {
                 $query->whereDoesntHave('childrenEnrollments')
                 ->where('parent_id', null);
             })
-            ->orWhere(function ($query) {
+            ->orWhere(function($query) {
                 $query->where('parent_id', null);
             })
             ->get();
@@ -86,7 +86,7 @@ 
 
     public function scopePeriod(Builder $query, int $periodId)
     {
-        return $query->whereHas('course', function ($q) use ($periodId) {
+        return $query->whereHas('course', function($q) use ($periodId) {
             $q->where('period_id', $periodId);
         });
     }
@@ -180,7 +180,7 @@ 
     // also includes invoices for this enrollment's scheduled payments.
     public function relatedInvoices()
     {
-        $scheduledPaymentsInvoices = $this->scheduledPayments->map(fn (ScheduledPayment $scheduledPayment) => $scheduledPayment->invoices());
+        $scheduledPaymentsInvoices = $this->scheduledPayments->map(fn(ScheduledPayment $scheduledPayment) => $scheduledPayment->invoices());
 
         return $this->invoices()->concat($scheduledPaymentsInvoices)->flatten(1);
     }
@@ -332,7 +332,7 @@ 
         }
 
         // if enabled, retrieve the default price category for the student
-        if (config('invoicing.price_categories_enabled') && $this->student?->price_category) {
+        if (config('invoicing.price_categories_enabled') && $this->student ? ->price_category) {
             $price_category = $this->student->price_category;
 
             return $this->course->$price_category ?? 0;
@@ -362,11 +362,11 @@ 
 
         // delete attendance records related to the enrollment
         $attendances = $this->course->attendance->where('student_id', $this->student->id);
-        Attendance::destroy($attendances->map(fn ($item, $key) => $item->id));
+        Attendance::destroy($attendances->map(fn($item, $key) => $item->id));
 
         foreach ($this->course->children as $child) {
             $attendances = $child->attendance->where('student_id', $this->student->id);
-            Attendance::destroy($attendances->map(fn ($item, $key) => $item->id));
+            Attendance::destroy($attendances->map(fn($item, $key) => $item->id));
         }
 
         $this->delete();
@@ -397,7 +397,7 @@ 
                 }
 
                 // if one book is expired
-                if ($this->student && $this->student->books->where('id', $book->id)->filter(fn ($book) => $book->pivot->expiry_date == null || $book->pivot->expiry_date > Carbon::now())->count() == 0) {
+                if ($this->student && $this->student->books->where('id', $book->id)->filter(fn($book) => $book->pivot->expiry_date == null || $book->pivot->expiry_date > Carbon::now())->count() == 0) {
                     return 'EXP';
                 }
             }

app/Models/ScheduledPayment.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
 
     public function scopeStatus(Builder $query, $status)
     {
-        return match ($status) {
+        return match($status) {
             '2' => $query->where('status', 2),
             '1' => $query->where('status', 1),
             default => $query,
@@ -119,7 +119,7 @@ 
 
     public function getStatusTypeNameAttribute()
     {
-        return match ($this->status) {
+        return match($this->status) {
             2 => __('Paid'),
             1 => __('Pending'),
             default => '-',

app/Console/Commands/compare_invoice_pricesCommand.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
         foreach (Invoice::all() as $invoice) {
             if ($invoice->total_price !== $invoice->totalPrice()) {
                 $anomalies++;
-                echo "\ninvoice ".$invoice->id. ' should be ' . $invoice->total_price . ' vs ' . $invoice->totalPrice() .' real ';
+                echo "\ninvoice ".$invoice->id.' should be '.$invoice->total_price.' vs '.$invoice->totalPrice().' real ';
             }
         }
         echo "\n".$anomalies.' anomalies detected';