Passed
Push — master ( 8bc982...4b4510 )
by Thomas
05:57
created
app/Http/Controllers/Admin/EnrollmentCrudController.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -69,9 +69,9 @@  discard block
 block discarded – undo
69 69
                 'name' => 'user.lastname',
70 70
                 'label' => __('Last Name'),
71 71
                 'type' => 'text',
72
-                'searchLogic' => function ($query, $column, $searchTerm) {
73
-                    $query->orWhereHas('student', function ($q) use ($searchTerm) {
74
-                        $q->whereHas('user', function ($q) use ($searchTerm) {
72
+                'searchLogic' => function($query, $column, $searchTerm) {
73
+                    $query->orWhereHas('student', function($q) use ($searchTerm) {
74
+                        $q->whereHas('user', function($q) use ($searchTerm) {
75 75
                             $q->where('lastname', 'like', '%'.$searchTerm.'%');
76 76
                         });
77 77
                     });
@@ -82,9 +82,9 @@  discard block
 block discarded – undo
82 82
                 'name' => 'user.firstname',
83 83
                 'label' => __('First Name'),
84 84
                 'type' => 'text',
85
-                'searchLogic' => function ($query, $column, $searchTerm) {
86
-                    $query->orWhereHas('student', function ($q) use ($searchTerm) {
87
-                        $q->whereHas('user', function ($q) use ($searchTerm) {
85
+                'searchLogic' => function($query, $column, $searchTerm) {
86
+                    $query->orWhereHas('student', function($q) use ($searchTerm) {
87
+                        $q->whereHas('user', function($q) use ($searchTerm) {
88 88
                             $q->where('firstname', 'like', '%'.$searchTerm.'%');
89 89
                         });
90 90
                     });
@@ -133,10 +133,10 @@  discard block
 block discarded – undo
133 133
             'name' => 'status_id',
134 134
             'type' => 'select2_multiple',
135 135
             'label'=> __('Status'),
136
-        ], function () {
136
+        ], function() {
137 137
             return EnrollmentStatusType::all()->pluck('name', 'id')->toArray();
138 138
         },
139
-          function ($values) { // if the filter is active
139
+          function($values) { // if the filter is active
140 140
               foreach (json_decode($values) as $key => $value) {
141 141
                   CRUD::addClause('orWhere', 'status_id', $value);
142 142
               }
@@ -146,9 +146,9 @@  discard block
 block discarded – undo
146 146
             'name' => 'period_id',
147 147
             'type' => 'select2',
148 148
             'label'=> __('Period'),
149
-        ], function () {
149
+        ], function() {
150 150
             return Period::all()->pluck('name', 'id')->toArray();
151
-        }, function ($value) { // if the filter is active
151
+        }, function($value) { // if the filter is active
152 152
             CRUD::addClause('period', $value);
153 153
         });
154 154
     }
Please login to merge, or discard this patch.
app/Console/Kernel.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,12 +29,12 @@
 block discarded – undo
29 29
      */
30 30
     protected function schedule(Schedule $schedule)
31 31
     {
32
-        $schedule->call(function () {
32
+        $schedule->call(function() {
33 33
             Log::info('Sending attendance reminders');
34 34
             (new Attendance)->remindPendingAttendance();
35 35
         })->dailyAt('08:15');
36 36
 
37
-        $schedule->call(function () {
37
+        $schedule->call(function() {
38 38
             Log::info('Checking default periods');
39 39
 
40 40
             // when we finish the current period; remove the manual override to automatically fallbackto the next one
Please login to merge, or discard this patch.
app/Providers/TelescopeServiceProvider.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
 
21 21
         $this->hideSensitiveRequestDetails();
22 22
 
23
-        Telescope::filter(function (IncomingEntry $entry) {
23
+        Telescope::filter(function(IncomingEntry $entry) {
24 24
             if ($this->app->isLocal()) {
25 25
                 return true;
26 26
             }
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
      */
62 62
     protected function gate()
63 63
     {
64
-        Gate::define('viewTelescope', function ($user) {
64
+        Gate::define('viewTelescope', function($user) {
65 65
             return in_array($user->email, [
66 66
                 //
67 67
             ]);
Please login to merge, or discard this patch.
app/Providers/AppServiceProvider.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@
 block discarded – undo
37 37
         }
38 38
 
39 39
         View::composer(
40
-            ['partials.create_new_contact', 'students.edit-contact'], function ($view) {
40
+            ['partials.create_new_contact', 'students.edit-contact'], function($view) {
41 41
                 $view->with('contact_types', ContactRelationship::all());
42 42
             });
43 43
     }
Please login to merge, or discard this patch.
app/Providers/AuthServiceProvider.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
          * a user is allowed to view the course attendance sheet if they are the teacher for this course,
30 30
          * or if they have explicit permission to view all course attendance sheets
31 31
          */
32
-        Gate::define('view-course-attendance', function ($user, $course) {
32
+        Gate::define('view-course-attendance', function($user, $course) {
33 33
             return ($user->teacher_id == $course->teacher_id) || $user->can('attendance.view');
34 34
         });
35 35
 
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
          * if they are the teacher for this course,
39 39
          * or if they have explicit permission to view all event attendance sheets
40 40
          */
41
-        Gate::define('view-event-attendance', function ($user, $event) {
41
+        Gate::define('view-event-attendance', function($user, $event) {
42 42
             return $event->teacher_id == $user->teacher_id || $event->course->teacher_id == $user->teacher_id || $user->can('attendance.view');
43 43
         });
44 44
 
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
          * if they are the teacher for the course,
48 48
          * or if they have explicit permission to edit any attendance sheets
49 49
          */
50
-        Gate::define('edit-attendance', function ($user, $event) {
50
+        Gate::define('edit-attendance', function($user, $event) {
51 51
             return $event->teacher_id == $user->teacher_id || $event->course->teacher_id == $user->teacher_id || $user->can('attendance.edit');
52 52
         });
53 53
 
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
          * teachers are allowed to view their own calendar,
56 56
          * and users with explicit permission can view all calendars
57 57
          */
58
-        Gate::define('view-teacher-calendar', function ($user, $teacher) {
58
+        Gate::define('view-teacher-calendar', function($user, $teacher) {
59 59
             return $user->teacher_id == $teacher->id || $user->can('calendars.view');
60 60
         });
61 61
 
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
          * teachers are allowed to view their own courses,
64 64
          * and users with explicit permission can view all courses
65 65
          */
66
-        Gate::define('view-course', function ($user, $course) {
66
+        Gate::define('view-course', function($user, $course) {
67 67
             return $user->teacher_id == $course->teacher_id || $user->can('courses.view');
68 68
         });
69 69
 
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
          * if they are a teacher
73 73
          * of if they have explicit permission to view any result
74 74
          */
75
-        Gate::define('view-enrollment', function ($user, $enrollment) {
75
+        Gate::define('view-enrollment', function($user, $enrollment) {
76 76
             return $user->student_id == $enrollment->student_id || $user->isTeacher() || $user->can('evaluation.view');
77 77
         });
78 78
 
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
          * if the user is the teacher of the course
81 81
          * of if they have explicit permission to enroll students
82 82
          */
83
-        Gate::define('enroll-in-course', function ($user, $course) {
83
+        Gate::define('enroll-in-course', function($user, $course) {
84 84
             return ($course->teacher_id == $user->teacher_id) || $user->can('enrollments.edit');
85 85
         });
86 86
 
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
          * if the user is a teacher
89 89
          * of if they have explicit permission to enroll students
90 90
          */
91
-        Gate::define('enroll-students', function ($user) {
91
+        Gate::define('enroll-students', function($user) {
92 92
             return $user->isTeacher() || $user->can('enrollments.edit');
93 93
         });
94 94
 
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
          * teachers are allowed to view their own hours,
97 97
          * and users with explicit permission can view all hours
98 98
          */
99
-        Gate::define('view-teacher-hours', function ($user, $teacher) {
99
+        Gate::define('view-teacher-hours', function($user, $teacher) {
100 100
             return $user->teacher_id == $teacher->id || $user->can('hr.view');
101 101
         });
102 102
 
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
          * teachers are allowed to edit results for their own students
105 105
          * as well as users with explicit permission to edit any result
106 106
          */
107
-        Gate::define('edit-result', function ($user, $enrollment) {
107
+        Gate::define('edit-result', function($user, $enrollment) {
108 108
             return $user->teacher_id == $enrollment->course->teacher_id || $user->can('evaluation.edit');
109 109
         });
110 110
     }
Please login to merge, or discard this patch.
app/Http/Controllers/CourseController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -18,9 +18,9 @@  discard block
 block discarded – undo
18 18
     public function __invoke(Builder $query, $value, string $property)
19 19
     {
20 20
         $value = collect($value)->toArray();
21
-        $query->where(function (Builder $query) use ($value) {
21
+        $query->where(function(Builder $query) use ($value) {
22 22
             $query->whereIn('level_id', $value)
23
-                    ->orWhereHas('children', function (Builder $query) use ($value) {
23
+                    ->orWhereHas('children', function(Builder $query) use ($value) {
24 24
                         $query->whereIn('level_id', $value);
25 25
                     });
26 26
         });
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 
56 56
     public function search()
57 57
     {
58
-        return QueryBuilder::for(Course::class)->where('campus_id', 1)
58
+        return QueryBuilder::for (Course::class)->where('campus_id', 1)
59 59
         ->with('room')->withCount('events')->withCount('children')->withCount('enrollments')
60 60
         ->allowedFilters([
61 61
             'name',
Please login to merge, or discard this patch.
app/Models/Student.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
         parent::boot();
32 32
 
33 33
         // when deleting a student, also delete any enrollments, grades and attendance related to this student
34
-        static::deleting(function (self $student) {
34
+        static::deleting(function(self $student) {
35 35
             Attendance::where('student_id', $student->id)->delete();
36 36
             Enrollment::where('student_id', $student->id)->delete();
37 37
         });
@@ -68,8 +68,8 @@  discard block
 block discarded – undo
68 68
 
69 69
         return $this->hasMany(Attendance::class)
70 70
         ->where('attendance_type_id', 4) // absence
71
-        ->whereHas('event', function ($q) use ($period) {
72
-            return $q->whereHas('course', function ($c) use ($period) {
71
+        ->whereHas('event', function($q) use ($period) {
72
+            return $q->whereHas('course', function($c) use ($period) {
73 73
                 return $c->where('period_id', $period->id);
74 74
             });
75 75
         });
@@ -82,8 +82,8 @@  discard block
 block discarded – undo
82 82
         }
83 83
 
84 84
         return $this->hasMany(Attendance::class)
85
-        ->whereHas('event', function ($q) use ($period) {
86
-            return $q->whereHas('course', function ($c) use ($period) {
85
+        ->whereHas('event', function($q) use ($period) {
86
+            return $q->whereHas('course', function($c) use ($period) {
87 87
                 return $c->where('period_id', $period->id);
88 88
             });
89 89
         });
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
     public function getIsEnrolledAttribute()
176 176
     {
177 177
         // if the student is currently enrolled
178
-        if ($this->enrollments()->whereHas('course', function ($q) {
178
+        if ($this->enrollments()->whereHas('course', function($q) {
179 179
             return $q->where('period_id', $this->currentPeriod);
180 180
         })->count() > 0) {
181 181
             return 1;
Please login to merge, or discard this patch.