App\Repository\TeacherAbsenceRepository

Complexity

Total Complexity

9

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	remove()	0	3	1
A	persist()	0	3	1
B	getPaginator()	0	34	6
A	removeRange()	0	9	1

<?php

namespace App\Repository;

use App\Entity\Teacher;
use App\Entity\TeacherAbsence;
use App\Entity\TeacherAbsenceLesson;
use DateTime;
use Doctrine\ORM\Tools\Pagination\Paginator;

class TeacherAbsenceRepository extends AbstractRepository implements TeacherAbsenceRepositoryInterface {

    public function getPaginator(int $itemsPerPage, int &$page, bool $hideProcessed, ?DateTime $startDate, ?DateTime $endDate, ?Teacher $teacher = null): Paginator {
        $qb = $this->em->createQueryBuilder()
            ->select('a')
            ->from(TeacherAbsence::class, 'a')
            ->orderBy('a.from.date', 'desc');

        if($hideProcessed) {
            $qb->andWhere('a.processedAt IS NULL');
        }

        if($startDate !== null) {
            $qb->andWhere('a.from.date >= :start');
        }

        if($endDate !== null) {
            $qb->andWhere('a.until.date <= :end');
        }

        if($teacher !== null) {
            $qb->andWhere('a.teacher = :teacher')
                ->setParameter('teacher', $teacher);
        }

        if($page < 1) {
            $page = 1;
        }

        $offset = ($page - 1) * $itemsPerPage;
        $paginator = new Paginator($qb);
        $paginator->getQuery()
            ->setMaxResults($itemsPerPage)
            ->setFirstResult($offset);

        return $paginator;
    }

    public function persist(TeacherAbsence|TeacherAbsenceLesson $absenceOrLesson): void {
        $this->em->persist($absenceOrLesson);
        $this->em->flush();
    }

    public function remove(TeacherAbsence|TeacherAbsenceLesson $absenceOrLesson): void {
        $this->em->remove($absenceOrLesson);
        $this->em->flush();
    }

    public function removeRange(DateTime $start, DateTime $end): int {
        return $this->em->createQueryBuilder()

The expression return $this->em->createQueryBuilder()->delete(App\Entity\TeacherAbsence::class, 't')->where('t.until.date >= :start')->andWhere('t.until.date <= :end')->setParameter('start', $start)->setParameter('end', $end)->getQuery()->execute() could return the type array<mixed,mixed> which is incompatible with the type-hinted return integer. Consider adding an additional type-check to rule them out.

            ->delete(TeacherAbsence::class, 't')
            ->where('t.until.date >= :start')
            ->andWhere('t.until.date <= :end')
            ->setParameter('start', $start)
            ->setParameter('end', $end)
            ->getQuery()
            ->execute();
    }
}