academico-sis / academico

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/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/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);
 
         $username = $part1.$part2.$part3; //str_shuffle to randomly shuffle all characters

app/Http/Controllers/Admin/SkillCrudController.php

Spacing +9 added lines, -9 removed lines

@@ -31,7 +31,7 @@ 
     protected function setupListOperation()
     {
         CRUD::setColumns([
-            [ // skill type
+            [// skill type
                 'label'     => 'Type',
                 'type'      => 'select',
                 'name'      => 'skill_type',
@@ -44,7 +44,7 @@ 
                 'type' => 'text',
                 'name' => 'name',
             ],
-            [ // skill level
+            [// skill level
                 'label'     => 'Level',
                 'type'      => 'select',
                 'name'      => 'level',
@@ -56,19 +56,19 @@ 
 
         CRUD::enableExportButtons();
 
-        CRUD::addFilter([ // select2 filter
+        CRUD::addFilter([// select2 filter
             'name' => 'level_id',
             'type' => 'select2',
             'label'=> 'Level',
-        ], fn () => Level::all()->pluck('name', 'id')->toArray(), function ($value) { // if the filter is active
+        ], fn() => Level::all()->pluck('name', 'id')->toArray(), function($value) { // if the filter is active
             CRUD::addClause('where', 'level_id', $value);
         });
 
-        CRUD::addFilter([ // select2 filter
+        CRUD::addFilter([// select2 filter
             'name' => 'skill_type_id',
             'type' => 'select2',
             'label'=> 'Type',
-        ], fn () => SkillType::all()->pluck('name', 'id')->toArray(), function ($value) { // if the filter is active
+        ], fn() => SkillType::all()->pluck('name', 'id')->toArray(), function($value) { // if the filter is active
             CRUD::addClause('where', 'skill_type_id', $value);
         });
     }
@@ -77,7 +77,7 @@ 
     {
         CRUD::setValidation(StoreRequest::class);
         CRUD::addFields([
-            [ // skill type
+            [// skill type
                 'label'     => 'Type',
                 'type' => 'select',
                 'name' => 'skill_type_id',
@@ -90,7 +90,7 @@ 
                 'type' => 'text',
                 'name' => 'name',
             ],
-            [ // skill level
+            [// skill level
                 'label'     => 'Level',
                 'type' => 'select',
                 'name' => 'level_id',
@@ -103,7 +103,7 @@ 
                 'type' => 'relationship',
                 'name' => 'skill_type', // the method on your model that defines the relationship
                 'ajax' => true,
-                'inline_create' => [ // specify the entity in singular
+                'inline_create' => [// specify the entity in singular
                     'entity' => 'skilltype', // the entity in singular
                     // OPTIONALS
                     'force_select' => true, // should the inline-created entry be immediately selected?

app/Http/Controllers/EnrollmentController.php

Spacing +2 added lines, -2 removed lines

@@ -183,7 +183,7 @@ 
                         'price_b' => $enrollment->course->price_b,
                         'price_c' => $enrollment->course->price_c,
                     ]),
-                    'studentPriceCategory' => $enrollment->student?->price_category,
+                    'studentPriceCategory' => $enrollment->student ? ->price_category,
                 ]
             );
         }
@@ -221,7 +221,7 @@ 
 
     public function getBalance(Request $request)
     {
-        if (! config('invoicing.invoices_contain_enrollments_only')) {
+        if (!config('invoicing.invoices_contain_enrollments_only')) {
             abort(422, 'Configuration options forbid to access this value');
         }
 

Braces +1 added lines, -2 removed lines

@@ -228,8 +228,7 @@
         $periodId = $request->get('periodId');
         if ($periodId) {
             $pendingBalance = Enrollment::period($periodId)->pending()->sum('balance');
-        }
-        else {
+        } else {
             $pendingBalance = Enrollment::pending()->sum('balance');
         }
 

app/Models/Enrollment.php

Spacing +9 added lines, -9 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';
                 }
             }
@@ -408,7 +408,7 @@ 
 
     public function getBalanceAttribute()
     {
-        if (! config('invoicing.invoices_contain_enrollments_only')) {
+        if (!config('invoicing.invoices_contain_enrollments_only')) {
             abort(422, 'Configuration options forbid to access this value');
         }
 

app/Http/Controllers/Admin/EnrollmentCrudController.php

Spacing +35 added lines, -35 removed lines

@@ -93,103 +93,103 @@ 
             $currency = ['suffix' => config('app.currency_symbol')];
         }
 
-        CRUD::addColumns([['name' => 'id', 'label' => 'ID', 'wrapper' => ['element' => function ($crud, $column, $entry) {
+        CRUD::addColumns([['name' => 'id', 'label' => 'ID', 'wrapper' => ['element' => function($crud, $column, $entry) {
             return $entry->status_id > 2 ? 'del' : 'span';
-        },],],
+        },], ],
 
-            ['label' => __('ID number'), 'type' => 'text', 'name' => 'student.idnumber', 'wrapper' => ['element' => function ($crud, $column, $entry) {
+            ['label' => __('ID number'), 'type' => 'text', 'name' => 'student.idnumber', 'wrapper' => ['element' => function($crud, $column, $entry) {
                 return $entry->status_id > 2 ? 'del' : 'span';
-            },],],
+            },], ],
 
-            ['name' => 'user', 'key' => 'user_lastname', 'attribute' => 'lastname', 'label' => __('Last Name'), 'type' => 'relationship', 'wrapper' => ['element' => function ($crud, $column, $entry) {
+            ['name' => 'user', 'key' => 'user_lastname', 'attribute' => 'lastname', 'label' => __('Last Name'), 'type' => 'relationship', 'wrapper' => ['element' => function($crud, $column, $entry) {
                 return $entry->status_id > 2 ? 'del' : 'span';
             },],
                 'orderable'  => true,
-                'orderLogic' => function ($query, $column, $columnDirection) {
+                'orderLogic' => function($query, $column, $columnDirection) {
                     return $query->leftJoin('users', 'enrollments.student_id', '=', 'users.id')
                         ->orderBy('users.lastname', $columnDirection)->select('enrollments.*');
                 },
-                'searchLogic' => function ($query, $column, $searchTerm) {
-                $query->orWhereHas('student', function ($q) use ($searchTerm) {
-                    $q->whereHas('user', function ($q) use ($searchTerm) {
-                        $q->where('lastname', 'like', '%' . $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.'%');
                     });
                 });
             },],
 
-            ['name' => 'user', 'key' => 'user_firstname', 'attribute' => 'firstname', 'label' => __('First Name'), 'type' => 'relationship', 'wrapper' => ['element' => function ($crud, $column, $entry) {
+            ['name' => 'user', 'key' => 'user_firstname', 'attribute' => 'firstname', 'label' => __('First Name'), 'type' => 'relationship', 'wrapper' => ['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) {
-                        $q->where('firstname', 'like', '%' . $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.'%');
                     });
                 });
             },
-                'orderLogic' => function ($query, $column, $columnDirection) {
+                'orderLogic' => function($query, $column, $columnDirection) {
                     return $query->leftJoin('users', 'enrollments.student_id', '=', 'users.id')
                         ->orderBy('users.firstname', $columnDirection)->select('enrollments.*');
                 },
                 'orderable'  => true,
                 ],
 
-            ['label' => __('Age'), 'name' => 'student_age',],
+            ['label' => __('Age'), 'name' => 'student_age', ],
 
-            ['label' => __('Birthdate'), 'name' => 'student_birthdate',],]);
+            ['label' => __('Birthdate'), 'name' => 'student_birthdate', ], ]);
 
         if ($this->mode === 'global') {
-            CRUD::addColumns([['label' => __('Course'), 'type' => 'select', 'name' => 'course_id', 'entity' => 'course', 'attribute' => 'name', 'model' => Course::class,],
+            CRUD::addColumns([['label' => __('Course'), 'type' => 'select', 'name' => 'course_id', 'entity' => 'course', 'attribute' => 'name', 'model' => Course::class, ],
                 ['type' => 'relationship', 'name' => 'course.period', 'label' => __('Period'), 'attribute' => 'name',
-                    'orderLogic' => function ($query, $column, $columnDirection) {
+                    'orderLogic' => function($query, $column, $columnDirection) {
                         return $query->leftJoin('courses', 'enrollments.course_id', '=', 'courses.id')
                             ->orderBy('courses.period_id', $columnDirection)->select('enrollments.*');
                     },
                     'orderable'  => true,
-                    ],]);
+                    ], ]);
         }
 
-        CRUD::addColumns([['label' => __('Status'), 'type' => 'select', 'name' => 'status_id', 'entity' => 'enrollmentStatus', 'attribute' => 'name', 'model' => EnrollmentStatusType::class, 'wrapper' => ['element' => 'span', 'class' => function ($crud, $column, $entry) {
-            return 'badge badge-pill badge-' . $entry->enrollmentStatus->styling();
-        },],],]);
+        CRUD::addColumns([['label' => __('Status'), 'type' => 'select', 'name' => 'status_id', 'entity' => 'enrollmentStatus', 'attribute' => 'name', 'model' => EnrollmentStatusType::class, 'wrapper' => ['element' => 'span', 'class' => function($crud, $column, $entry) {
+            return 'badge badge-pill badge-'.$entry->enrollmentStatus->styling();
+        },], ], ]);
 
         if (config('invoicing.allow_scheduled_payments')) {
             CRUD::addColumn(['name' => 'scheduledPayments', 'type' => 'relationship', 'label' => __('Scheduled Payments'), // OPTIONAL
-                'attribute' => 'date', 'model' => ScheduledPayment::class,]);
+                'attribute' => 'date', 'model' => ScheduledPayment::class, ]);
         }
 
         CRUD::addColumn(array_merge(['name' => 'price', // The db column name
-            'label' => __('Price'), 'type' => 'number',], $currency));
+            'label' => __('Price'), 'type' => 'number', ], $currency));
 
 
         if (config('invoicing.invoices_contain_enrollments_only')) {
-            CRUD::addColumn(array_merge(['name' => 'balance', 'label' => __('Balance'), 'type' => 'number',], $currency));
+            CRUD::addColumn(array_merge(['name' => 'balance', 'label' => __('Balance'), 'type' => 'number', ], $currency));
         }
 
         CRUD::addColumns([
 
-            ['name' => 'scholarships', 'type' => 'relationship', 'label' => __('Scholarship'), 'attribute' => 'name', 'model' => Scholarship::class,],
+            ['name' => 'scholarships', 'type' => 'relationship', 'label' => __('Scholarship'), 'attribute' => 'name', 'model' => Scholarship::class, ],
 
-            ['label' => __('Email'), 'name' => 'user', 'attribute' => 'email', 'type' => 'relationship',],
+            ['label' => __('Email'), 'name' => 'user', 'attribute' => 'email', 'type' => 'relationship', ],
 
-            ['label' => __('Phone Number'), 'type' => 'select_multiple', 'name' => 'student.phone', 'attribute' => 'phone_number', 'model' => PhoneNumber::class,],]);
+            ['label' => __('Phone Number'), 'type' => 'select_multiple', 'name' => 'student.phone', 'attribute' => 'phone_number', 'model' => PhoneNumber::class, ], ]);
 
         if ($this->mode === 'global') {
-            CRUD::addFilter(['name' => 'status_id', 'type' => 'select2_multiple', 'label' => __('Status'),], fn() => EnrollmentStatusType::all()->pluck('name', 'id')->toArray(), function ($values) {
+            CRUD::addFilter(['name' => 'status_id', 'type' => 'select2_multiple', 'label' => __('Status'), ], 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);
                 }
             });
 
-            CRUD::addFilter(['name' => 'period_id', 'type' => 'select2', 'label' => __('Period'),], fn() => Period::all()->pluck('name', 'id')->toArray(), function ($value) {
+            CRUD::addFilter(['name' => 'period_id', 'type' => 'select2', 'label' => __('Period'), ], fn() => Period::all()->pluck('name', 'id')->toArray(), function($value) {
                 CRUD::addClause('period', $value);
             });
 
-            CRUD::addFilter(['name' => 'scholarship', 'type' => 'select2', 'label' => __('Scholarship'),], fn() => Scholarship::all()->pluck('name', 'id')->toArray(), function ($value) { // if the filter is active
+            CRUD::addFilter(['name' => 'scholarship', 'type' => 'select2', 'label' => __('Scholarship'), ], 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);
                     });
                 }
@@ -234,7 +234,7 @@ 
             'model'     => 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 +4 added lines, -4 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');
         }
     }
@@ -104,7 +104,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) {
@@ -147,7 +147,7 @@ 
     {
         $invoice = Invoice::findOrFail($id)->load('payments');
 
-        if (! backpack_user()->can('enrollments.edit')) {
+        if (!backpack_user()->can('enrollments.edit')) {
             abort(403);
         }
 
@@ -155,7 +155,7 @@ 
             'invoice' => $invoice,
             'availablePaymentMethods' => Paymentmethod::all(),
             'editable' => true,
-            'enrollment' => $invoice->enrollments->first()?->product,
+            'enrollment' => $invoice->enrollments->first() ? ->product,
             'afterSuccessUrl' => $invoice->enrollments->count() > 0 ? "/enrollment/{$invoice->enrollments->first()->product_id}/show" : '/invoice', // TODO fix this, an invoice can theoretically contain several enrollments
         ]);
     }