| @@ -34,7 +34,7 @@ | ||
| 34 | 34 | parent::boot(); | 
| 35 | 35 | |
| 36 | 36 | // when deleting a user, also delete any teacher and students accounts associated to this user. | 
| 37 | -        static::deleting(function (self $user) { | |
| 37 | +        static::deleting(function(self $user) { | |
| 38 | 38 |              if ($user->student()->exists()) { | 
| 39 | 39 |                  Attendance::where('student_id', $user->student->id)->delete(); | 
| 40 | 40 |                  Enrollment::where('student_id', $user->student->id)->delete(); | 
| @@ -176,7 +176,7 @@ discard block | ||
| 176 | 176 | */ | 
| 177 | 177 | public function getPendingAttendanceAttribute() | 
| 178 | 178 |      { | 
| 179 | -        $events = Event::where(function ($query) { | |
| 179 | +        $events = Event::where(function($query) { | |
| 180 | 180 |              $query->where('course_id', $this->id); | 
| 181 | 181 |              $query->where('exempt_attendance', '!=', true); | 
| 182 | 182 |              $query->where('exempt_attendance', '!=', 1); | 
| @@ -247,7 +247,7 @@ discard block | ||
| 247 | 247 |              foreach ($courseTimes as $courseTime) { | 
| 248 | 248 | $initial = $daysInitials[$courseTime->day]; | 
| 249 | 249 | |
| 250 | -                if (! isset($parsedCourseTimes[$initial])) { | |
| 250 | +                if (!isset($parsedCourseTimes[$initial])) { | |
| 251 | 251 | $parsedCourseTimes[$initial] = []; | 
| 252 | 252 | } | 
| 253 | 253 | |
| @@ -322,7 +322,7 @@ discard block | ||
| 322 | 322 | |
| 323 | 323 | public function eventsWithExpectedAttendance() | 
| 324 | 324 |      { | 
| 325 | -        return $this->events()->where(function ($query) { | |
| 325 | +        return $this->events()->where(function($query) { | |
| 326 | 326 |              $query->where('exempt_attendance', '!=', true); | 
| 327 | 327 |              $query->where('exempt_attendance', '!=', 1); | 
| 328 | 328 |              $query->orWhereNull('exempt_attendance'); | 
| @@ -186,15 +186,15 @@ | ||
| 186 | 186 | $eventsWithMissingAttendance = []; | 
| 187 | 187 | |
| 188 | 188 | $eventsWithExpectedAttendance = $this->events() | 
| 189 | -        ->where(function ($query) { | |
| 189 | +        ->where(function($query) { | |
| 190 | 190 |              $query->where('exempt_attendance', '!=', true); | 
| 191 | 191 |              $query->where('exempt_attendance', '!=', 1); | 
| 192 | 192 |              $query->orWhereNull('exempt_attendance'); | 
| 193 | 193 | }) | 
| 194 | 194 |          ->where('course_id', '!=', null) | 
| 195 | -        ->whereHas('course', function (Builder $query) use ($period) { | |
| 195 | +        ->whereHas('course', function(Builder $query) use ($period) { | |
| 196 | 196 |              return $query->where('period_id', $period->id) | 
| 197 | -                ->where(function ($query) { | |
| 197 | +                ->where(function($query) { | |
| 198 | 198 |                      $query->where('exempt_attendance', '!=', true); | 
| 199 | 199 |                      $query->where('exempt_attendance', '!=', 1); | 
| 200 | 200 |                      $query->orWhereNull('exempt_attendance'); | 
| @@ -208,8 +208,8 @@ | ||
| 208 | 208 |                  foreach ($course->enrollments as $enrollment) { | 
| 209 | 209 | // if a student has no attendance record for the class (event) | 
| 210 | 210 |                      $hasNotAttended = $course->attendance->where('student_id', $enrollment->student_id) | 
| 211 | -                     ->where('event_id', $event->id) | |
| 212 | - ->isEmpty(); | |
| 211 | +                        ->where('event_id', $event->id) | |
| 212 | + ->isEmpty(); | |
| 213 | 213 | |
| 214 | 214 | // count one and break loop | 
| 215 | 215 |                      if ($hasNotAttended) { | 
| @@ -134,7 +134,7 @@ discard block | ||
| 134 | 134 |      { | 
| 135 | 135 |          $period = self::where('id', '<', $this->id)->orderBy('id', 'desc')->first(); | 
| 136 | 136 | |
| 137 | -        if (! $period == null) { | |
| 137 | +        if (!$period == null) { | |
| 138 | 138 | return $period; | 
| 139 | 139 |          } else { | 
| 140 | 140 | return self::first(); | 
| @@ -195,7 +195,7 @@ discard block | ||
| 195 | 195 | public function getCoursesWithPendingAttendanceAttribute() | 
| 196 | 196 |      { | 
| 197 | 197 | // get all courses for period and preload relations | 
| 198 | -        $courses = $this->courses()->where(function ($query) { | |
| 198 | +        $courses = $this->courses()->where(function($query) { | |
| 199 | 199 |              $query->where('exempt_attendance', '!=', true); | 
| 200 | 200 |              $query->where('exempt_attendance', '!=', 1); | 
| 201 | 201 |              $query->orWhereNull('exempt_attendance'); | 
| @@ -20,14 +20,14 @@ | ||
| 20 | 20 | parent::boot(); | 
| 21 | 21 | |
| 22 | 22 | // do not save already existing leaves | 
| 23 | -        static::saving(function (self $leave) { | |
| 23 | +        static::saving(function(self $leave) { | |
| 24 | 24 |              if (self::where('teacher_id', $leave->teacher_id)->where('date', $leave->date)->count() > 0) { | 
| 25 | 25 | return false; | 
| 26 | 26 | } | 
| 27 | 27 | }); | 
| 28 | 28 | |
| 29 | 29 | // when a leave is, we detach the events from the teacher | 
| 30 | -        static::saved(function (self $leave) { | |
| 30 | +        static::saved(function(self $leave) { | |
| 31 | 31 |              $events = Event::where('teacher_id', $leave->teacher_id)->whereDate('start', $leave->date)->get(); | 
| 32 | 32 |              foreach ($events as $event) { | 
| 33 | 33 | $event->teacher_id = null; | 
| @@ -21,7 +21,7 @@ | ||
| 21 | 21 | parent::boot(); | 
| 22 | 22 | |
| 23 | 23 | // when a result is added, send a notification | 
| 24 | -        static::saved(function (self $result) { | |
| 24 | +        static::saved(function(self $result) { | |
| 25 | 25 | Mail::to($result->enrollment->student->user->email) | 
| 26 | 26 | ->locale($result->enrollment->student->locale) | 
| 27 | 27 | ->queue(new ResultNotification($result->enrollment->course, $result->enrollment->student->user)); | 
| @@ -15,12 +15,12 @@ | ||
| 15 | 15 | parent::boot(); | 
| 16 | 16 | |
| 17 | 17 | // when a coursetime is added, we should create corresponding events | 
| 18 | -        static::created(function ($coursetime) { | |
| 18 | +        static::created(function($coursetime) { | |
| 19 | 19 | $coursetime->create_events(); | 
| 20 | 20 | }); | 
| 21 | 21 | |
| 22 | 22 | // when a coursetime is deleted, we should delete all associated future events | 
| 23 | -        static::deleted(function ($coursetime) { | |
| 23 | +        static::deleted(function($coursetime) { | |
| 24 | 24 | $coursetime->events()->delete(); | 
| 25 | 25 | // todo delete only future events | 
| 26 | 26 | }); | 
| @@ -23,7 +23,7 @@ | ||
| 23 | 23 | parent::boot(); | 
| 24 | 24 | |
| 25 | 25 | // when an attendance record is added, we check if this is an absence | 
| 26 | -        static::saved(function (self $attendance) { | |
| 26 | +        static::saved(function(self $attendance) { | |
| 27 | 27 |              if ($attendance->attendance_type_id == 4) { // todo move to configurable settings | 
| 28 | 28 |                  Log::info('Absence marked for student '.$attendance->student->name); | 
| 29 | 29 | // will check the record again and send a notification if it hasn't changed | 
| @@ -29,7 +29,7 @@ discard block | ||
| 29 | 29 | parent::boot(); | 
| 30 | 30 | |
| 31 | 31 | // when deleting a student, also delete any enrollments, grades and attendance related to this student | 
| 32 | -        static::deleting(function (self $student) { | |
| 32 | +        static::deleting(function(self $student) { | |
| 33 | 33 |              Attendance::where('student_id', $student->id)->delete(); | 
| 34 | 34 |              Enrollment::where('student_id', $student->id)->delete(); | 
| 35 | 35 | }); | 
| @@ -66,8 +66,8 @@ discard block | ||
| 66 | 66 | |
| 67 | 67 | return $this->hasMany(Attendance::class) | 
| 68 | 68 |          ->where('attendance_type_id', 4) // absence | 
| 69 | -        ->whereHas('event', function ($q) use ($period) { | |
| 70 | -            return $q->whereHas('course', function ($c) use ($period) { | |
| 69 | +        ->whereHas('event', function($q) use ($period) { | |
| 70 | +            return $q->whereHas('course', function($c) use ($period) { | |
| 71 | 71 |                  return $c->where('period_id', $period->id); | 
| 72 | 72 | }); | 
| 73 | 73 | }); | 
| @@ -80,8 +80,8 @@ discard block | ||
| 80 | 80 | } | 
| 81 | 81 | |
| 82 | 82 | return $this->hasMany(Attendance::class) | 
| 83 | -        ->whereHas('event', function ($q) use ($period) { | |
| 84 | -            return $q->whereHas('course', function ($c) use ($period) { | |
| 83 | +        ->whereHas('event', function($q) use ($period) { | |
| 84 | +            return $q->whereHas('course', function($c) use ($period) { | |
| 85 | 85 |                  return $c->where('period_id', $period->id); | 
| 86 | 86 | }); | 
| 87 | 87 | }); | 
| @@ -173,7 +173,7 @@ discard block | ||
| 173 | 173 | public function getIsEnrolledAttribute() | 
| 174 | 174 |      { | 
| 175 | 175 | // if the student is currently enrolled | 
| 176 | -        if ($this->enrollments()->whereHas('course', function ($q) { | |
| 176 | +        if ($this->enrollments()->whereHas('course', function($q) { | |
| 177 | 177 |              return $q->where('period_id', Period::get_default_period()->id); | 
| 178 | 178 |          })->count() > 0) { | 
| 179 | 179 | return 1; |