Passed
Pull Request — master (#234)
by
unknown
06:11
created
app/Http/Controllers/Admin/LeaveCrudController.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -66,25 +66,25 @@  discard block
 block discarded – undo
66 66
             ],
67 67
         ]);
68 68
 
69
-        CRUD::addFilter([ // select2 filter
69
+        CRUD::addFilter([// select2 filter
70 70
             'name' => 'teacher_id',
71 71
             'type' => 'select2',
72 72
             'label'=> __('Teacher'),
73
-        ], function () {
73
+        ], function() {
74 74
             return \App\Models\Teacher::all()->pluck('name', 'id')->toArray();
75
-        }, function ($value) { // if the filter is active
75
+        }, function($value) { // if the filter is active
76 76
             CRUD::addClause('where', 'teacher_id', $value);
77 77
         },
78
-          function () { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
78
+          function() { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
79 79
           });
80 80
 
81
-        CRUD::addFilter([ // daterange filter
81
+        CRUD::addFilter([// daterange filter
82 82
             'type' => 'date_range',
83 83
             'name' => 'from_to',
84 84
             'label'=> __('Date range'),
85 85
         ],
86 86
           false,
87
-          function ($value) { // if the filter is active, apply these constraints
87
+          function($value) { // if the filter is active, apply these constraints
88 88
               $dates = json_decode($value);
89 89
               CRUD::addClause('where', 'date', '>=', $dates->from);
90 90
               CRUD::addClause('where', 'date', '<=', $dates->to.' 23:59:59');
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
                 'model' => \App\Models\LeaveType::class, // foreign key model
117 117
             ],
118 118
 
119
-            [   // date_range
119
+            [// date_range
120 120
                 'name' => ['start_date', 'end_date'], // db columns for start_date & end_date
121 121
                 'label' => 'Event Date Range',
122 122
                 'type' => 'date_range',
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
                 'model' => \App\Models\LeaveType::class, // foreign key model
161 161
             ],
162 162
 
163
-            [   // datepicker
163
+            [// datepicker
164 164
                 'name' => 'date',
165 165
                 'label' => 'Event Date',
166 166
                 'type' => 'date',
Please login to merge, or discard this patch.
app/Http/Controllers/Admin/EventCrudController.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -105,13 +105,13 @@  discard block
 block discarded – undo
105 105
 
106 106
         ]);
107 107
 
108
-        CRUD::addFilter([ // daterange filter
108
+        CRUD::addFilter([// daterange filter
109 109
             'type' => 'date_range',
110 110
             'name' => 'from_to',
111 111
             'label'=> __('Date range'),
112 112
         ],
113 113
           false,
114
-          function ($value) { // if the filter is active, apply these constraints
114
+          function($value) { // if the filter is active, apply these constraints
115 115
               $dates = json_decode($value);
116 116
 
117 117
               if ($dates->from) {
@@ -128,10 +128,10 @@  discard block
 block discarded – undo
128 128
             'label'=> __('Events with no course'),
129 129
         ],
130 130
           false,
131
-          function ($value) { // if the filter is active, apply these constraints
131
+          function($value) { // if the filter is active, apply these constraints
132 132
               $this->crud->query->where('course_id', null);
133 133
           },
134
-          function () { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
134
+          function() { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
135 135
           });
136 136
 
137 137
         CRUD::addFilter([
@@ -140,20 +140,20 @@  discard block
 block discarded – undo
140 140
             'label'=> __('Events with no teacher'),
141 141
         ],
142 142
           false,
143
-          function ($value) { // if the filter is active, apply these constraints
143
+          function($value) { // if the filter is active, apply these constraints
144 144
               $this->crud->query->where('teacher_id', null);
145 145
           });
146 146
 
147
-        CRUD::addFilter([ // select2 filter
147
+        CRUD::addFilter([// select2 filter
148 148
             'name' => 'teacher_id',
149 149
             'type' => 'select2',
150 150
             'label'=> __('Teacher'),
151
-        ], function () {
151
+        ], function() {
152 152
             return \App\Models\Teacher::all()->pluck('name', 'id')->toArray();
153
-        }, function ($value) { // if the filter is active
153
+        }, function($value) { // if the filter is active
154 154
             CRUD::addClause('where', 'teacher_id', $value);
155 155
         },
156
-          function () { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
156
+          function() { // if the filter is NOT active (the GET parameter "checkbox" does not exit)
157 157
           });
158 158
     }
159 159
 
Please login to merge, or discard this patch.
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.