SchulIT / icc

src/Command/CancelLessonsOnFreeDaysCommand.php

Spacing +3 added lines, -3 removed lines

@@ -35,7 +35,7 @@ 
         $lessons = $this->lessonRepository->findAllByDate($today);
         $io->text(sprintf('Anzahl der Unterrichtsstunden am Tag: %d', count($lessons)));
 
-        foreach($freeTimespans as $timespan) {
+        foreach ($freeTimespans as $timespan) {
             $io->section(sprintf('Markiere %d.-%d. Stunde als Entfall: %s', $timespan->getLessonStart(), $timespan->getLessonEnd(), $timespan->getReason()));
 
             /** @var TimetableLesson[] $affectedLessons */
@@ -44,8 +44,8 @@ 
             $io->text(sprintf('Betroffene Stunden: %d', count($affectedLessons)));
             $count = 0;
 
-            foreach($affectedLessons as $affectedLesson) {
-                if($affectedLesson->getEntries()->count() === 0) {
+            foreach ($affectedLessons as $affectedLesson) {
+                if ($affectedLesson->getEntries()->count() === 0) {
                     $this->cancelHelper->cancelLesson($affectedLesson, $timespan->getReason());
                     $count++;
                 }

src/Book/EntryOverviewHelper.php

Spacing +40 added lines, -40 removed lines

@@ -37,7 +37,7 @@ 
         $entries = $this->entryRepository->findAllByTuition($tuition, $start, $end);
         $comments = $this->commentRepository->findAllByDateAndTuition($tuition, $start, $end);
 
-        return $this->computeOverview([$tuition], $entries, $comments, [ ], $start, $end);
+        return $this->computeOverview([$tuition], $entries, $comments, [], $start, $end);
     }
 
     /**
@@ -47,7 +47,7 @@ 
      * @param Substitution[] $substitutions
      */
     private function computeOverview(array $tuitions, array $entries, array $comments, array $substitutions, DateTime $start, DateTime $end): EntryOverview {
-        if($start > $end) {
+        if ($start > $end) {
             $tmp = $start;
             $start = $end;
             $end = $tmp;
@@ -55,10 +55,10 @@ 
 
         $tuitions = array_filter($tuitions, fn(Tuition $tuition) => $tuition->isBookEnabled());
 
-        $lessons = [ ];
+        $lessons = [];
 
-        foreach($entries as $entry) {
-            if($entry->getTuition() === null) {
+        foreach ($entries as $entry) {
+            if ($entry->getTuition() === null) {
                 // discard entries without tuition
                 continue;
             }
@@ -67,7 +67,7 @@ 
             $absentCount = $this->attendanceRepository->countAbsent($entry);
             $lateCount = $this->attendanceRepository->countLate($entry);
 
-            for($lessonNumber = $entry->getLessonStart(); $lessonNumber <= $entry->getLessonEnd(); $lessonNumber++) {
+            for ($lessonNumber = $entry->getLessonStart(); $lessonNumber <= $entry->getLessonEnd(); $lessonNumber++) {
                 $key = sprintf('%s-%d-%s', $entry->getLesson()->getDate()->format('Y-m-d'), $lessonNumber, $entry->getTuition()->getUuid()->toString());
 
                 $lesson = new Lesson(clone $entry->getLesson()->getDate(), $lessonNumber, null, $entry);
@@ -79,12 +79,12 @@ 
             }
         }
 
-        $weekNumbers = [ ];
+        $weekNumbers = [];
         $currentWeek = clone $start;
-        while($currentWeek < $end) {
-            $weekNumber = (int)$currentWeek->format('W');
+        while ($currentWeek < $end) {
+            $weekNumber = (int) $currentWeek->format('W');
 
-            if(!in_array($weekNumber, $weekNumbers)) {
+            if (!in_array($weekNumber, $weekNumbers)) {
                 $weekNumbers[] = $weekNumber;
             }
 
@@ -94,11 +94,11 @@ 
         $timetableLessons = $this->lessonRepository->findAllByTuitions($start, $end, $tuitions);
 
         $current = clone $start;
-        while($current <= $end) {
+        while ($current <= $end) {
             $dailyLessons = array_filter($timetableLessons, fn(TimetableLesson $lesson) => $lesson->getDate() == $current);
 
-            foreach($dailyLessons as $dailyLesson) {
-                for($lessonNumber = $dailyLesson->getLessonStart(); $lessonNumber <= $dailyLesson->getLessonEnd(); $lessonNumber++) {
+            foreach ($dailyLessons as $dailyLesson) {
+                for ($lessonNumber = $dailyLesson->getLessonStart(); $lessonNumber <= $dailyLesson->getLessonEnd(); $lessonNumber++) {
                     $key = sprintf('%s-%d-%s', $current->format('Y-m-d'), $lessonNumber, $dailyLesson->getTuition()->getUuid()->toString());
 
                     if (!array_key_exists($key, $lessons)) {
@@ -112,19 +112,19 @@ 
             $current = $current->modify('+1 day');
         }
 
-        foreach($substitutions as $substitution) {
+        foreach ($substitutions as $substitution) {
             $tuition = $this->tuitionRepository->findOneBySubstitution($substitution, $this->sectionResolver->getSectionForDate($substitution->getDate()));
 
-            if($tuition === null) {
+            if ($tuition === null) {
                 continue;
             }
 
             $timetableLessons = $this->lessonRepository->findAllByTuitions($substitution->getDate(), $substitution->getDate(), [$tuition]);
 
             // Filter correct lesson
-            foreach($timetableLessons as $dailyLesson) {
-                for($lessonNumber = $dailyLesson->getLessonStart(); $lessonNumber <= $dailyLesson->getLessonEnd(); $lessonNumber++) {
-                    if($substitution->getLessonStart() <= $lessonNumber && $lessonNumber <= $substitution->getLessonEnd()) {
+            foreach ($timetableLessons as $dailyLesson) {
+                for ($lessonNumber = $dailyLesson->getLessonStart(); $lessonNumber <= $dailyLesson->getLessonEnd(); $lessonNumber++) {
+                    if ($substitution->getLessonStart() <= $lessonNumber && $lessonNumber <= $substitution->getLessonEnd()) {
                         $key = sprintf('%s-%d-%s', $substitution->getDate()->format('Y-m-d'), $lessonNumber, $dailyLesson->getTuition()->getUuid()->toString());
 
                         if (!array_key_exists($key, $lessons)) {
@@ -133,14 +133,14 @@ 
 
                         // Set correct entry
                         /** @var LessonEntry $lessonEntry */
-                        foreach($dailyLesson->getEntries() as $lessonEntry) {
-                            if($lessonEntry->getLessonStart() <= $lessonNumber && $lessonNumber <= $lessonEntry->getLessonEnd()) {
+                        foreach ($dailyLesson->getEntries() as $lessonEntry) {
+                            if ($lessonEntry->getLessonStart() <= $lessonNumber && $lessonNumber <= $lessonEntry->getLessonEnd()) {
                                 $lessons[$key]->setEntry($lessonEntry);
                                 break;
                             }
                         }
 
-                        if($lessons[$key]->getLesson() === null) {
+                        if ($lessons[$key]->getLesson() === null) {
                             $lessons[$key]->setLesson($dailyLesson);
                         }
                     }
@@ -159,15 +159,15 @@ 
 
     public function computeOverviewForTeacher(Teacher $teacher, DateTime $start, DateTime $end): EntryOverview {
         $section = $this->sectionResolver->getSectionForDate($start);
-        $entries = [ ];
-        $tuitions = [ ];
-        $comments = [ ];
+        $entries = [];
+        $tuitions = [];
+        $comments = [];
 
-        if($section !== null) {
+        if ($section !== null) {
             $entries = $this->entryRepository->findAllBySubstituteTeacher($teacher, $start, $end);
             $tuitions = $this->tuitionRepository->findAllByTeacher($teacher, $section);
 
-            foreach($tuitions as $tuition) {
+            foreach ($tuitions as $tuition) {
                 $entries = array_merge($entries, $this->entryRepository->findAllByTuition($tuition, $start, $end));
                 $comments = array_merge($comments, $this->commentRepository->findAllByDateAndTuition($tuition, $start, $end));
             }
@@ -175,10 +175,10 @@ 
             $comments = ArrayUtils::unique($comments);
         }
 
-        $substitutions = [ ];
+        $substitutions = [];
 
         $current = clone $start;
-        while($current <= $end) {
+        while ($current <= $end) {
             $substitutions = array_merge(
                 $substitutions,
                 $this->substitutionRepository->findAllForTeacher($teacher, $current)
@@ -194,41 +194,41 @@ 
     public function computeOverviewForGrade(Grade $grade, DateTime $start, DateTime $end): EntryOverview {
         $section = $this->sectionResolver->getSectionForDate($start);
 
-        if($section === null) {
-            return new EntryOverview($start, $end, [ ], [ ], [ ]);
+        if ($section === null) {
+            return new EntryOverview($start, $end, [], [], []);
         }
 
-        if($end > $section->getEnd()) {
+        if ($end > $section->getEnd()) {
             $end = $section->getEnd();
         }
 
         $entries = $this->entryRepository->findAllByGrade($grade, $start, $end);
         $tuitions = $this->tuitionRepository->findAllByGrades([$grade], $section);
 
-        $comments = [ ];
+        $comments = [];
         $section = $this->sectionResolver->getSectionForDate($start);
 
-        if($section !== null) {
+        if ($section !== null) {
             $comments = $this->commentRepository->findAllByDateAndGrade($grade, $section, $start, $end);
         }
 
-        return $this->computeOverview($tuitions, $entries, $comments, [ ], $start, $end);
+        return $this->computeOverview($tuitions, $entries, $comments, [], $start, $end);
     }
 
     /**
      * @return FreeTimespan[]
      */
     public function computeFreeTimespans(DateTime $start, DateTime $end): array {
-        $result = [ ];
+        $result = [];
 
         $categories = array_map(fn(int $id) => $this->appointmentCategoryRepository->findOneById($id), $this->timetableSettings->getCategoryIds());
 
-        if(count($categories) > 0) {
+        if (count($categories) > 0) {
             $appointments = $this->appointmentRepository->findAllStartEnd($start, $end, $categories);
 
-            foreach($appointments as $appointment) {
+            foreach ($appointments as $appointment) {
                 $current = clone $appointment->getStart();
-                while($current < $appointment->getEnd()) {
+                while ($current < $appointment->getEnd()) {
                     $result[] = new FreeTimespan(clone $current, 1, $this->timetableSettings->getMaxLessons(), $appointment->getTitle());
                     $current = $current->modify('+1 day');
                 }
@@ -236,10 +236,10 @@ 
         }
 
         $current = clone $start;
-        while($current <= $end) {
+        while ($current <= $end) {
             $freeTimespans = $this->freeTimespanRepository->findAllByDate($current);
 
-            foreach($freeTimespans as $timespan) {
+            foreach ($freeTimespans as $timespan) {
                 $result[] = new FreeTimespan(clone $timespan->getDate(), $timespan->getStart(), $timespan->getEnd(), 'Vertretungsplan');
             }