academico-sis / academico

app/Http/Controllers/Admin/StudentCrudController.php

Spacing +30 added lines, -30 removed lines

@@ -88,12 +88,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.'%');
                     });
                 },
@@ -108,12 +108,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', '=', 'students.id')
                         ->orderBy('users.firstname', $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('firstname', 'like', '%'.$searchTerm.'%');
                     });
                 },
@@ -127,12 +127,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', '=', 'students.id')
                         ->orderBy('users.email', $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('email', 'like', '%'.$searchTerm.'%');
                     });
                 },
@@ -160,32 +160,32 @@ 
 
         ]);
 
-        CRUD::addFilter([ // select2 filter
+        CRUD::addFilter([// select2 filter
             'name' => 'enrolled',
             'type' => 'select2',
             'label'=> __('Is Enrolled in'),
-        ], function () {
+        ], function() {
             return Period::all()->pluck('name', 'id')->toArray();
-        }, function ($value) { // if the filter is active
-            $this->crud->query = $this->crud->query->whereHas('enrollments', function ($query) use ($value) {
-                return $query->whereHas('course', function ($q) use ($value) {
+        }, function($value) { // if the filter is active
+            $this->crud->query = $this->crud->query->whereHas('enrollments', function($query) use ($value) {
+                return $query->whereHas('course', function($q) use ($value) {
                     $q->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)
           });
 
-        CRUD::addFilter([ // select2_multiple filter
+        CRUD::addFilter([// select2_multiple filter
             'name' => 'notenrolled',
             'type' => 'select2_multiple',
             'label'=> __('Is Not Enrolled in'),
-        ], function () { // the options that show up in the select2
+        ], function() { // the options that show up in the select2
             return Period::all()->pluck('name', 'id')->toArray();
-        }, function ($values) { // if the filter is active
+        }, function($values) { // if the filter is active
             foreach (json_decode($values) as $value) {
-                $this->crud->query = $this->crud->query->whereDoesntHave('enrollments', function ($query) use ($value) {
-                    return $query->whereHas('course', function ($q) use ($value) {
+                $this->crud->query = $this->crud->query->whereDoesntHave('enrollments', function($query) use ($value) {
+                    return $query->whereHas('course', function($q) use ($value) {
                         $q->where('period_id', $value);
                     });
                 });
@@ -197,9 +197,9 @@ 
             'name'  => 'institution_id',
             'type'  => 'select2',
             'label' => __('Institution'),
-        ], function () {
+        ], function() {
             return Institution::all()->pluck('name', 'id')->toArray();
-        }, function ($value) { // if the filter is active
+        }, function($value) { // if the filter is active
             $this->crud->addClause('where', 'institution_id', $value);
         });
 
@@ -207,10 +207,10 @@ 
             'name'  => 'status_type_id',
             'type'  => 'select2',
             'label' => __('Lead Status'),
-        ], function () {
+        ], function() {
             return LeadType::all()->pluck('name', 'id')->toArray();
-        }, function ($value) {
-            $this->crud->addClause('where','lead_type_id', $value);
+        }, function($value) {
+            $this->crud->addClause('where', 'lead_type_id', $value);
         });
     }
 
@@ -300,11 +300,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;
     }
@@ -322,7 +322,7 @@ 
             $username = $request->email;
         }
         else {
-            $username = $this->generateUsername($request->firstname . ' ' . $request->lastname);
+            $username = $this->generateUsername($request->firstname.' '.$request->lastname);
         }
 
         // update the user info
@@ -391,7 +391,7 @@ 
     {
         $student = Student::findOrFail($student);
 
-        if (! backpack_user()->isTeacher() && ! backpack_user()->can('enrollments.view')) {
+        if (!backpack_user()->isTeacher() && !backpack_user()->can('enrollments.view')) {
             abort(403);
         }
 

app/Models/Period.php

Spacing +4 added lines, -4 removed lines

@@ -75,7 +75,7 @@ 
     {
         parent::boot();
 
-        static::addGlobalScope('order', function (Builder $builder) {
+        static::addGlobalScope('order', function(Builder $builder) {
             $builder->orderBy('year_id')->orderBy('order')->orderBy('id');
         });
     }
@@ -88,7 +88,7 @@ 
     {
         $configPeriod = Config::where('name', 'current_period');
 
-        if($configPeriod->exists()) {
+        if ($configPeriod->exists()) {
             
             $currentPeriod = $configPeriod->first();
 
@@ -225,7 +225,7 @@ 
     {
         $period = self::where('id', '<', $this->id)->orderBy('id', 'desc')->first();
 
-        if (! $period == null) {
+        if (!$period == null) {
             return $period;
         } else {
             return self::first();
@@ -302,7 +302,7 @@ 
     public function getCoursesWithPendingAttendanceAttribute()
     {
         // get all courses for period and preload relations
-        $courses = $this->courses()->where(function ($query) {
+        $courses = $this->courses()->where(function($query) {
             $query->where('exempt_attendance', '!=', true);
             $query->where('exempt_attendance', '!=', 1);
             $query->orWhereNull('exempt_attendance');