academico-sis / academico

app/Http/Controllers/HomeController.php

Spacing +5 added lines, -5 removed lines

@@ -46,7 +46,7 @@ 
 
     public function teacher(Request $request)
     {
-        if (! backpack_user()->isTeacher()) {
+        if (!backpack_user()->isTeacher()) {
             abort(403);
         }
 
@@ -70,7 +70,7 @@ 
 
     public function student()
     {
-        if (! backpack_user()->isStudent()) {
+        if (!backpack_user()->isStudent()) {
             abort(403);
         }
 
@@ -88,7 +88,7 @@ 
         $currentPeriod = Period::get_default_period();
         $enrollmentsPeriod = Period::get_enrollments_period();
 
-        if (! backpack_user()->hasRole(['admin', 'secretary'])) {
+        if (!backpack_user()->hasRole(['admin', 'secretary'])) {
             abort(403);
         }
 
@@ -99,14 +99,14 @@ 
 
         $teachers = Teacher::with('user')->get()->toArray();
 
-        $teachers = array_map(function ($teacher) {
+        $teachers = array_map(function($teacher) {
             return [
                 'id' => $teacher['id'],
                 'title' => $teacher['user']['firstname'],
             ];
         }, $teachers);
 
-        $events = array_map(function ($event) {
+        $events = array_map(function($event) {
             return [
                 'title' => $event['name'],
                 'resourceId' => $event['teacher_id'],

app/Http/Controllers/Auth/RegisterController.php

Indentation +3 added lines, -3 removed lines

@@ -169,9 +169,9 @@
         // add photo
         if($request->data['userPicture']) {
             $student
-               ->addMediaFromBase64($request->data['userPicture'])
-               ->usingFileName('profilePicture.jpg')
-               ->toMediaCollection('profile-picture');
+                ->addMediaFromBase64($request->data['userPicture'])
+                ->usingFileName('profilePicture.jpg')
+                ->toMediaCollection('profile-picture');
 
             Log::info('Profile picture added to the student profile');
         }

Spacing +6 added lines, -6 removed lines

@@ -30,11 +30,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;
     }
@@ -83,7 +83,7 @@ 
             $username = $data['email'];
         }
         else {
-            $username = $this->generateUsername($data['firstname'] . ' ' . $data['lastname']);
+            $username = $this->generateUsername($data['firstname'].' '.$data['lastname']);
         }
 
         // create the user
@@ -123,7 +123,7 @@ 
     {
 
         // if registration is closed, deny access
-        if (! config('backpack.base.registration_open')) {
+        if (!config('backpack.base.registration_open')) {
             abort(403, trans('backpack::base.registration_closed'));
         }
 
@@ -167,7 +167,7 @@ 
         Log::info('Profession and institution added to the student profile');
 
         // add photo
-        if($request->data['userPicture']) {
+        if ($request->data['userPicture']) {
             $student
                ->addMediaFromBase64($request->data['userPicture'])
                ->usingFileName('profilePicture.jpg')

Braces +1 added lines, -2 removed lines

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

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');
             CRUD::denyAccess('create');
         }
@@ -67,7 +67,7 @@ 
 
         CRUD::addButtonFromView('line', 'children_badge', 'children_badge', 'beginning');
 
-        if (! $permissions->contains('name', 'courses.delete')) {
+        if (!$permissions->contains('name', 'courses.delete')) {
             CRUD::denyAccess('delete');
         }
 
@@ -189,13 +189,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)
         });
 
@@ -203,13 +203,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)
         });
 
@@ -217,13 +217,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)
         });
 
@@ -232,13 +232,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
@@ -246,13 +246,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');
           }
         );
@@ -263,10 +263,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([
@@ -275,10 +275,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');
             });
     }
 
@@ -354,7 +354,7 @@ 
             ],
 
 
-            [   // repeatable
+            [// repeatable
                 'name'  => 'sublevels',
                 'label' => __('Course sublevels'),
                 'type'  => 'repeatable',
@@ -511,7 +511,7 @@ 
                 'default' => Period::get_enrollments_period()->end,
             ],
 
-            [   // repeatable
+            [// repeatable
                 'name'  => 'times',
                 'label' => __('Course Schedule'),
                 'type'  => 'repeatable',
@@ -574,14 +574,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',
@@ -605,7 +605,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'
@@ -613,7 +613,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/SkillCrudController.php

Spacing +11 added lines, -11 removed lines

@@ -33,7 +33,7 @@ 
     protected function setupListOperation()
     {
         CRUD::setColumns([
-            [ // skill type
+            [// skill type
                 'label'     => 'Type', // Table column heading
                 'type'      => 'select',
                 'name'      => 'skill_type', // the method that defines the relationship in your Model
@@ -46,7 +46,7 @@ 
                 'type' => 'text',
                 'name' => 'name',
             ],
-            [ // skill level
+            [// skill level
                 'label'     => 'Level', // Table column heading
                 'type'      => 'select',
                 'name'      => 'level', // the method that defines the relationship in your Model
@@ -58,23 +58,23 @@ 
 
         CRUD::enableBulkActions();
 
-        CRUD::addFilter([ // select2 filter
+        CRUD::addFilter([// select2 filter
             'name' => 'level_id',
             'type' => 'select2',
             'label'=> 'Level',
-        ], function () {
+        ], function() {
             return 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);
         });
 
-        CRUD::addFilter([ // select2 filter
+        CRUD::addFilter([// select2 filter
             'name' => 'skill_type_id',
             'type' => 'select2',
             'label'=> 'Type',
-        ], function () {
+        ], function() {
             return SkillType::all()->pluck('name', 'id')->toArray();
-        }, function ($value) { // if the filter is active
+        }, function($value) { // if the filter is active
             CRUD::addClause('where', 'skill_type_id', $value);
         });
     }
@@ -83,7 +83,7 @@ 
     {
         CRUD::setValidation(StoreRequest::class);
         CRUD::addFields([
-            [ // skill type
+            [// skill type
                 'label'     => 'Type', // Table column heading
                 'type' => 'select',
                 'name' => 'skill_type_id', // the db column for the foreign key
@@ -96,7 +96,7 @@ 
                 'type' => 'text',
                 'name' => 'name',
             ],
-            [ // skill level
+            [// skill level
                 'label'     => 'Level', // Table column heading
                 'type' => 'select',
                 'name' => 'level_id', // the db column for the foreign key
@@ -109,7 +109,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/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);
             });
         });