academico-sis / academico

app/Http/Controllers/ReportController.php

Spacing +2 added lines, -2 removed lines

@@ -62,7 +62,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);
@@ -163,7 +163,7 @@ 
     {
         $period = Period::get_default_period();
 
-        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/CourseCrudController.php

Indentation +7 added lines, -7 removed lines

@@ -239,10 +239,10 @@ 
             CRUD::addClause('where', 'period_id', $value);
         },
             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
-          }
+                $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
+            }
         );
 
         CRUD::addFilter(
@@ -253,8 +253,8 @@ 
         ],
             false,
             function () {
-              CRUD::addClause('parent');
-          }
+                CRUD::addClause('parent');
+            }
         );
 
         $this->crud->addFilter([
@@ -380,7 +380,7 @@ 
                         'type' => 'number'
                     ], $currency),
 
-                   [
+                    [
                         'name' => 'volume', // The db column name
                         'label' => __('Presential volume'), // Table column heading
                         'suffix' => 'h',

Spacing +26 added lines, -26 removed lines

@@ -56,7 +56,7 @@ 
         CRUD::addClause('internal');
         $permissions = backpack_user()->getAllPermissions();
 
-        if (! $permissions->contains('name', 'courses.edit')) {
+        if (!$permissions->contains('name', 'courses.edit')) {
             CRUD::denyAccess(['update', 'create']);
         }
 
@@ -66,7 +66,7 @@ 
 
         CRUD::addButtonFromView('line', 'children_badge', 'children_badge', 'beginning');
 
-        if (! $permissions->contains('name', 'courses.delete')) {
+        if (!$permissions->contains('name', 'courses.delete')) {
             CRUD::denyAccess(['delete']);
         }
 
@@ -188,13 +188,13 @@ 
             'name' => 'rhythm_id',
             'type' => 'select2',
             'label'=> __('Rhythm'),
-        ], function () {
+        ], function() {
             return Rhythm::all()->pluck('name', 'id')->toArray();
-        }, function ($value) {
+        }, function($value) {
             // if the filter is active
             CRUD::addClause('where', 'rhythm_id', $value);
         },
-        function () {
+        function() {
             // if the filter is NOT active (the GET parameter "checkbox" does not exit)
         });
 
@@ -202,13 +202,13 @@ 
             'name' => 'teacher_id',
             'type' => 'select2',
             'label'=> __('Teacher'),
-        ], function () {
+        ], function() {
             return Teacher::all()->pluck('name', 'id')->toArray();
-        }, function ($value) {
+        }, function($value) {
             // if the filter is active
             CRUD::addClause('where', 'teacher_id', $value);
         },
-        function () {
+        function() {
             // if the filter is NOT active (the GET parameter "checkbox" does not exit)
         });
 
@@ -216,13 +216,13 @@ 
             'name' => 'level_id',
             'type' => 'select2',
             'label'=> __('Level'),
-        ], function () {
+        ], function() {
             return Level::all()->pluck('name', 'id')->toArray();
-        }, function ($value) {
+        }, function($value) {
             // if the filter is active
             CRUD::addClause('where', 'level_id', $value);
         },
-        function () {
+        function() {
             // if the filter is NOT active (the GET parameter "checkbox" does not exit)
         });
 
@@ -231,13 +231,13 @@ 
             'type' => 'select2',
             'label'=> __('Period'),
         ],
-            function () {
+            function() {
             return \App\Models\Period::all()->sortByDesc('id')->pluck('name', 'id')->toArray();
         },
-            function ($value) { // if the filter is active
+            function($value) { // if the filter is active
             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
@@ -245,13 +245,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');
           }
         );
@@ -262,10 +262,10 @@ 
             '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);
                 $this->crud->addClause('where', 'start_date', '>=', $dates->from);
-                $this->crud->addClause('where', 'start_date', '<=', $dates->to . ' 23:59:59');
+                $this->crud->addClause('where', 'start_date', '<=', $dates->to.' 23:59:59');
             });
 
         $this->crud->addFilter([
@@ -274,10 +274,10 @@ 
             '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);
                 $this->crud->addClause('where', 'end_date', '>=', $dates->from);
-                $this->crud->addClause('where', 'end_date', '<=', $dates->to . ' 23:59:59');
+                $this->crud->addClause('where', 'end_date', '<=', $dates->to.' 23:59:59');
             });
     }
 
@@ -353,7 +353,7 @@ 
             ],
 
 
-            [   // repeatable
+            [// repeatable
                 'name'  => 'sublevels',
                 'label' => __('Course sublevels'),
                 'type'  => 'repeatable',
@@ -510,7 +510,7 @@ 
                 'default' => Period::get_enrollments_period()->end,
             ],
 
-            [   // repeatable
+            [// repeatable
                 'name'  => 'times',
                 'label' => __('Course Schedule'),
                 'type'  => 'repeatable',
@@ -573,14 +573,14 @@ 
                 'init_rows' => 0, // number of empty rows to be initialized, by default 1
             ],
 
-            [   // 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',
@@ -604,7 +604,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'
@@ -612,7 +612,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/TeacherCrudController.php

Spacing +13 added lines, -13 removed lines

@@ -48,12 +48,12 @@ 
                 'attribute' => 'lastname', // foreign key attribute that is shown to user
                 'model'     => 'App\Models\User', // foreign key model
                 'orderable' => true,
-                'orderLogic' => function ($query, $column, $columnDirection) {
+                'orderLogic' => function($query, $column, $columnDirection) {
                     return $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.'%');
                     });
                 },
@@ -68,12 +68,12 @@ 
                 'attribute' => 'firstname', // foreign key attribute that is shown to user
                 'model'     => 'App\Models\User', // foreign key model
                 'orderable' => true,
-                'orderLogic' => function ($query, $column, $columnDirection) {
+                'orderLogic' => function($query, $column, $columnDirection) {
                     return $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.'%');
                     });
                 },
@@ -87,12 +87,12 @@ 
                 'attribute' => 'email', // foreign key attribute that is shown to user
                 'model'     => 'App\Models\User', // foreign key model
                 'orderable' => true,
-                'orderLogic' => function ($query, $column, $columnDirection) {
+                'orderLogic' => function($query, $column, $columnDirection) {
                     return $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.'%');
                     });
                 },
@@ -166,7 +166,7 @@ 
             $username = $request->email;
         }
         else {
-            $username = $this->generateUsername($request->firstname . ' ' . $request->lastname);
+            $username = $this->generateUsername($request->firstname.' '.$request->lastname);
         }
 
         $user = User::create([
@@ -191,11 +191,11 @@ 
         $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 = rand(999, 9999);
 
-        $username = $part1. $part2. $part3; //str_shuffle to randomly shuffle all characters
+        $username = $part1.$part2.$part3; //str_shuffle to randomly shuffle all characters
 
         return $username;
     }

Braces +1 added lines, -2 removed lines

@@ -164,8 +164,7 @@
         if (User::where('email', $request->email)->count() === 0)
         {
             $username = $request->email;
-        }
-        else {
+        } else {
             $username = $this->generateUsername($request->firstname . ' ' . $request->lastname);
         }
 

app/Http/Controllers/Admin/ConfigCrudController.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     public function setup()
     {
         CRUD::setModel(\App\Models\Config::class);
-        CRUD::setRoute(config('backpack.base.route_prefix') . '/config');
+        CRUD::setRoute(config('backpack.base.route_prefix').'/config');
         CRUD::setEntityNameStrings('config', 'configs');
     }
 

app/Http/Controllers/Admin/InvoiceCrudController.php

Spacing +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
     public function setup()
     {
         CRUD::setModel(\App\Models\Invoice::class);
-        CRUD::setRoute(config('backpack.base.route_prefix') . '/invoice');
+        CRUD::setRoute(config('backpack.base.route_prefix').'/invoice');
         CRUD::setEntityNameStrings('invoice', 'invoices');
         $this->crud->addButtonFromView('top', 'createInvoice', 'createInvoice', 'start');
     }
@@ -101,7 +101,7 @@ 
     {
         $invoice = Invoice::findOrFail($id)->load('payments');
 
-        if (! backpack_user()->can('enrollments.edit')) {
+        if (!backpack_user()->can('enrollments.edit')) {
             abort(403);
         }
 

app/Http/Controllers/Admin/ResultCrudController.php

Indentation +7 added lines, -7 removed lines

@@ -54,7 +54,7 @@ 
                             $q->where('firstname', 'like', '%'.$searchTerm.'%')
                             ->orWhere('lastname', 'like', '%'.$searchTerm.'%')
                             ->orWhere('email', 'like', '%'.$searchTerm.'%')
-                          ->orWhere('idnumber', 'like', '%'.$searchTerm.'%');
+                            ->orWhere('idnumber', 'like', '%'.$searchTerm.'%');
                         });
                     });
                 },
@@ -88,10 +88,10 @@ 
             'name' => 'noresult',
             'label'=> __('No Result'),
         ],
-          false,
-          function () {
-              CRUD::addClause('noResult');
-          }
+            false,
+            function () {
+                CRUD::addClause('noResult');
+            }
         );
 
         CRUD::addFilter(
@@ -102,8 +102,8 @@ 
         ],
             false,
             function () {
-              CRUD::addClause('real');
-          }
+                CRUD::addClause('real');
+            }
         );
 
         CRUD::addFilter([

Spacing +11 added lines, -11 removed lines

@@ -48,9 +48,9 @@ 
                 'type' => 'select',
                 'name' => 'student', // the method that defines the relationship in your Model
                 'attribute' => 'name', // foreign key attribute that is shown to user
-                '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.'%')
@@ -89,7 +89,7 @@ 
             'label'=> __('No Result'),
         ],
           false,
-          function () {
+          function() {
               CRUD::addClause('noResult');
           }
         );
@@ -101,7 +101,7 @@ 
             'label'=> __('Hide Parents'),
         ],
             false,
-            function () {
+            function() {
               CRUD::addClause('real');
           }
         );
@@ -110,20 +110,20 @@ 
             'name' => 'period_id',
             'type' => 'select2',
             'label'=> __('Period'),
-        ], function () {
+        ], function() {
             return Period::all()->pluck('name', 'id')->toArray();
-        }, function ($value) { // if the filter is active
+        }, 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'),
-        ], function () { // the options that show up in the select2
+        ], function() { // the options that show up in the select2
             return 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) {
+        }, 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/Admin/ExternalCourseCrudController.php

Indentation +8 added lines, -8 removed lines

@@ -109,7 +109,7 @@ 
                 'entity' => 'teacher', // the method that defines the relationship in your Model
                 'attribute' => 'name', // foreign key attribute that is shown to user
                 'model' => \App\Models\Teacher::class, // foreign key model
-		'searchLogic' => false,
+        'searchLogic' => false,
             ],
 
             [
@@ -170,7 +170,7 @@ 
             CRUD::addClause('where', 'rhythm_id', $value);
         },
             function () { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
-          }
+            }
         );
 
         CRUD::addFilter(
@@ -186,7 +186,7 @@ 
             CRUD::addClause('where', 'teacher_id', $value);
         },
             function () { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
-          }
+            }
         );
 
         CRUD::addFilter(
@@ -202,7 +202,7 @@ 
             CRUD::addClause('where', 'level_id', $value);
         },
             function () { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
-          }
+            }
         );
 
         CRUD::addFilter(
@@ -218,10 +218,10 @@ 
             CRUD::addClause('where', 'period_id', $value);
         },
             function () { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
-              $period = 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
-          }
+                $period = 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
+            }
         );
     }
 

Spacing +19 added lines, -19 removed lines

@@ -158,66 +158,66 @@ 
         ]);
 
         CRUD::addFilter(
-            [ // select2 filter
+            [// select2 filter
             'name' => 'rhythm_id',
             'type' => 'select2',
             'label'=> __('Rhythm'),
         ],
-            function () {
+            function() {
             return \App\Models\Rhythm::all()->pluck('name', 'id')->toArray();
         },
-            function ($value) { // if the filter is active
+            function($value) { // if the filter is active
             CRUD::addClause('where', 'rhythm_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 filter
+            [// select2 filter
             'name' => 'teacher_id',
             'type' => 'select2',
             'label'=> __('Teacher'),
         ],
-            function () {
+            function() {
             return \App\Models\Teacher::all()->pluck('name', 'id')->toArray();
         },
-            function ($value) { // if the filter is active
+            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)
           }
         );
 
         CRUD::addFilter(
-            [ // select2 filter
+            [// select2 filter
             'name' => 'level_id',
             'type' => 'select2',
             'label'=> __('Level'),
         ],
-            function () {
+            function() {
             return \App\Models\Level::all()->pluck('name', 'id')->toArray();
         },
-            function ($value) { // if the filter is active
+            function($value) { // if the filter is active
             CRUD::addClause('where', 'level_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 filter
+            [// select2 filter
             'name' => 'period_id',
             'type' => 'select2',
             'label'=> __('Period'),
         ],
-            function () {
+            function() {
             return Period::all()->pluck('name', 'id')->toArray();
         },
-            function ($value) { // if the filter is active
+            function($value) { // if the filter is active
             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 = 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
@@ -363,7 +363,7 @@ 
                 'tab' => __('Course info'),
             ],
 
-            [   // repeatable
+            [// repeatable
                 'name'  => 'times',
                 'label' => __('Course Schedule'),
                 'type'  => 'repeatable',
@@ -402,7 +402,7 @@ 
                 'tab' => __('Schedule'),
             ],
 
-            [   // view
+            [// view
                 'name' => 'custom-ajax-button',
                 'type' => 'view',
                 'view' => 'courses/schedule-preset-alert',
@@ -556,7 +556,7 @@ 
                 'tab' => __('Course info'),
             ],
 
-            [   // repeatable
+            [// repeatable
                 'name'  => 'times',
                 'label' => __('Course Schedule'),
                 'type'  => 'repeatable',

app/Http/Controllers/Admin/PaymentCrudController.php

Spacing +4 added lines, -4 removed lines

@@ -35,7 +35,7 @@ 
     public function setup()
     {
         CRUD::setModel(\App\Models\Payment::class);
-        CRUD::setRoute(config('backpack.base.route_prefix') . '/payment');
+        CRUD::setRoute(config('backpack.base.route_prefix').'/payment');
         CRUD::setEntityNameStrings('payment', 'payments');
 
         $this->crud->enableExportButtons();
@@ -54,7 +54,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());
             });
@@ -95,11 +95,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/GradeTypeCrudController.php

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
                 'type' => "relationship",
                 'name' => 'category', // 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' => 'gradetypecategory', // the entity in singular
                     // OPTIONALS
                     'force_select' => true, // should the inline-created entry be immediately selected?