Passed
Push — master ( 3d9c9a...9bf1ea )
by Thomas
06:35 queued 11s
created
app/Models/Event.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@
 block discarded – undo
18 18
         parent::boot();
19 19
 
20 20
         // before adding an event, we check that the teacher is available
21
-        static::saving(function ($event) {
21
+        static::saving(function($event) {
22 22
             $teacher = Teacher::find($event->teacher_id);
23 23
             // if the teacher is on leave on the day of the event
24 24
             if (($event->teacher_id !== null) && $teacher->leaves->contains('date', Carbon::parse($event->start)->toDateString())) {
Please login to merge, or discard this patch.
app/Models/Enrollment.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
         parent::boot();
28 28
 
29 29
         // when creating a new enrollment, also add past attendance
30
-        static::created(function (self $enrollment) {
30
+        static::created(function(self $enrollment) {
31 31
             $events = $enrollment->course->events->where('start', '<', (new Carbon())->toDateString());
32 32
             foreach ($events as $event) {
33 33
                 $event->attendance()->create([
@@ -58,11 +58,11 @@  discard block
 block discarded – undo
58 58
     public function scopeWithoutChildren($query)
59 59
     {
60 60
         return $query
61
-            ->where(function ($query) {
61
+            ->where(function($query) {
62 62
                 $query->whereDoesntHave('childrenEnrollments')
63 63
                 ->where('parent_id', null);
64 64
             })
65
-            ->orWhere(function ($query) {
65
+            ->orWhere(function($query) {
66 66
                 $query->where('parent_id', null);
67 67
             })
68 68
             ->get();
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 
85 85
     public function scopePeriod(Builder $query, $period)
86 86
     {
87
-        return $query->whereHas('course', function ($q) use ($period) {
87
+        return $query->whereHas('course', function($q) use ($period) {
88 88
             $q->where('period_id', $period);
89 89
         });
90 90
     }
@@ -293,13 +293,13 @@  discard block
 block discarded – undo
293 293
 
294 294
         // delete attendance records related to the enrollment
295 295
         $attendances = $this->course->attendance->where('student_id', $this->student->id);
296
-        Attendance::destroy($attendances->map(function ($item, $key) {
296
+        Attendance::destroy($attendances->map(function($item, $key) {
297 297
             return $item->id;
298 298
         }));
299 299
 
300 300
         foreach ($this->course->children as $child) {
301 301
             $attendances = $child->attendance->where('student_id', $this->student->id);
302
-            Attendance::destroy($attendances->map(function ($item, $key) {
302
+            Attendance::destroy($attendances->map(function($item, $key) {
303 303
                 return $item->id;
304 304
             }));
305 305
         }
Please login to merge, or discard this patch.
app/Http/Controllers/CourseController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@
 block discarded – undo
41 41
 
42 42
     public function search()
43 43
     {
44
-        return QueryBuilder::for(Course::class)->where('campus_id', 1)
44
+        return QueryBuilder::for (Course::class)->where('campus_id', 1)
45 45
         ->with('room')->withCount('events')->withCount('children')->withCount('enrollments')
46 46
         ->allowedFilters([
47 47
             'name',
Please login to merge, or discard this patch.
app/Traits/FiltersSearchableLevels.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -10,8 +10,8 @@
 block discarded – undo
10 10
     public function __invoke(Builder $query, $value, string $property)
11 11
     {
12 12
         $value = collect($value)->toArray();
13
-        $query->where(function (Builder $query) use ($value) {
14
-            $query->whereIn('level_id', $value)->orWhereHas('children', function (Builder $query) use ($value) {
13
+        $query->where(function(Builder $query) use ($value) {
14
+            $query->whereIn('level_id', $value)->orWhereHas('children', function(Builder $query) use ($value) {
15 15
                 $query->whereIn('level_id', $value);
16 16
             });
17 17
         });
Please login to merge, or discard this patch.
app/Http/Controllers/CourseSkillEvaluationController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@
 block discarded – undo
69 69
 
70 70
         $student_skills = $enrollment->skill_evaluations;
71 71
 
72
-        $skills = $course->skills->map(function ($skill, $key) use ($student_skills) {
72
+        $skills = $course->skills->map(function($skill, $key) use ($student_skills) {
73 73
             $skill['status'] = $student_skills->where('skill_id', $skill->id)->first()->skill_scale_id ?? null;
74 74
 
75 75
             return $skill;
Please login to merge, or discard this patch.
app/Http/Middleware/SetLocale.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@
 block discarded – undo
17 17
     private function setLocale(string $locale)
18 18
     {
19 19
         // Check if is allowed and set default locale if not
20
-        if (! language()->allowed($locale)) {
20
+        if (!language()->allowed($locale)) {
21 21
             $locale = config('app.locale');
22 22
         }
23 23
 
Please login to merge, or discard this patch.
app/Http/Controllers/GradeController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@
 block discarded – undo
52 52
 
53 53
         $gradeType = GradeType::findOrFail($request->input('grade_type_id'));
54 54
 
55
-        if (! $course->grade_types->contains($gradeType->id)) {
55
+        if (!$course->grade_types->contains($gradeType->id)) {
56 56
             $course->grade_types()->attach($gradeType->id);
57 57
         }
58 58
 
Please login to merge, or discard this patch.
app/Providers/AuthServiceProvider.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
          * a user is allowed to edit the course grades if they are the teacher for this course,
31 31
          * or if they have explicit permission to do so
32 32
          */
33
-        Gate::define('edit-course-grades', function ($user, $course) {
33
+        Gate::define('edit-course-grades', function($user, $course) {
34 34
             return $user->isTeacher() && $user->teacher_id == $course->teacher_id || $user->can('evaluation.edit');
35 35
         });
36 36
 
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
          * a user is allowed to view the course attendance sheet if they are the teacher for this course,
39 39
          * or if they have explicit permission to view all course attendance sheets
40 40
          */
41
-        Gate::define('view-course-attendance', function ($user, $course) {
41
+        Gate::define('view-course-attendance', function($user, $course) {
42 42
             return $user->isTeacher() && $user->teacher_id == $course->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 this course,
48 48
          * or if they have explicit permission to view all event attendance sheets
49 49
          */
50
-        Gate::define('view-event-attendance', function ($user, $event) {
50
+        Gate::define('view-event-attendance', function($user, $event) {
51 51
             return ($event->teacher_id == $user->teacher_id) || ($event->course->teacher_id == $user->teacher_id) || $user->can('attendance.view');
52 52
         });
53 53
 
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
          * if they are the teacher for the course,
57 57
          * or if they have explicit permission to edit any attendance sheets
58 58
          */
59
-        Gate::define('edit-attendance', function ($user, $event) {
59
+        Gate::define('edit-attendance', function($user, $event) {
60 60
             return ($event->teacher_id == $user->teacher_id) || ($event->course->teacher_id == $user->teacher_id) || $user->can('attendance.edit');
61 61
         });
62 62
 
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
          * teachers are allowed to view their own calendar,
65 65
          * and users with explicit permission can view all calendars
66 66
          */
67
-        Gate::define('view-teacher-calendar', function ($user, $teacher) {
67
+        Gate::define('view-teacher-calendar', function($user, $teacher) {
68 68
             return ($user->isTeacher() && $user->teacher_id == $teacher->id) || $user->can('calendars.view');
69 69
         });
70 70
 
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
          * teachers are allowed to view their own courses,
73 73
          * and users with explicit permission can view all courses
74 74
          */
75
-        Gate::define('view-course', function ($user, Course $course) {
75
+        Gate::define('view-course', function($user, Course $course) {
76 76
             return ($user->isTeacher() && $user->teacher_id == $course->teacher_id) || $user->can('courses.view');
77 77
         });
78 78
 
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
          * if they are a teacher
82 82
          * of if they have explicit permission to view any result
83 83
          */
84
-        Gate::define('view-enrollment', function ($user, $enrollment) {
84
+        Gate::define('view-enrollment', function($user, $enrollment) {
85 85
             return ($user->isStudent() && $user->student_id == $enrollment->student_id) || $user->isTeacher() || $user->can('evaluation.view');
86 86
         });
87 87
 
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
          * if the user is the teacher of the course
90 90
          * of if they have explicit permission to enroll students
91 91
          */
92
-        Gate::define('enroll-in-course', function ($user, $course) {
92
+        Gate::define('enroll-in-course', function($user, $course) {
93 93
             return $course->teacher_id == $user->teacher_id || $user->can('enrollments.edit');
94 94
         });
95 95
 
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
          * if the user is a teacher
98 98
          * of if they have explicit permission to enroll students
99 99
          */
100
-        Gate::define('enroll-students', function ($user) {
100
+        Gate::define('enroll-students', function($user) {
101 101
             return $user->isTeacher() || $user->can('enrollments.edit');
102 102
         });
103 103
 
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
          * teachers are allowed to view their own hours,
106 106
          * and users with explicit permission can view all hours
107 107
          */
108
-        Gate::define('view-teacher-hours', function ($user, $teacher) {
108
+        Gate::define('view-teacher-hours', function($user, $teacher) {
109 109
             return ($user->isTeacher() && $user->teacher_id == $teacher->id) || $user->can('hr.view');
110 110
         });
111 111
 
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
          * teachers are allowed to edit results for their own students
114 114
          * as well as users with explicit permission to edit any result
115 115
          */
116
-        Gate::define('edit-result', function ($user, $enrollment) {
116
+        Gate::define('edit-result', function($user, $enrollment) {
117 117
             return ($user->isTeacher() && $user->teacher_id == $enrollment->course->teacher_id) || $user->can('evaluation.edit');
118 118
         });
119 119
     }
Please login to merge, or discard this patch.