App\Repository\FreeTimespanRepository

Complexity

Total Complexity

6

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	findAll()	0	3	1
A	findAllByDate()	0	4	1
A	persist()	0	3	1
A	removeAll()	0	14	2
A	removeBetween()	0	9	1

<?php

namespace App\Repository;

use App\Entity\FreeTimespan;
use DateTime;

class FreeTimespanRepository extends AbstractTransactionalRepository implements FreeTimespanRepositoryInterface {

    /**
     * @inheritDoc
     */
    public function findAllByDate(DateTime $dateTime): array {
        return $this->em->getRepository(FreeTimespan::class)
            ->findBy([
                'date' => $dateTime
            ]);
    }

    /**
     * @inheritDoc
     */
    public function findAll(): array {
        return $this->em->getRepository(FreeTimespan::class)
            ->findAll();
    }

    public function persist(FreeTimespan $timespan): void {
        $this->em->persist($timespan);
        $this->flushIfNotInTransaction();
    }

    public function removeAll(?DateTime $dateTime): void {
        $qb = $this->em->createQueryBuilder()
            ->delete(FreeTimespan::class, 't');

        if($dateTime !== null) {
            $qb->where('t.date = :date')
                ->setParameter('date', $dateTime);
        }

        $qb
            ->getQuery()
            ->execute();

        $this->flushIfNotInTransaction();
    }

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

The expression return $this->em->createQueryBuilder()->delete(App\Entity\FreeTimespan::class, 't')->where('t.date >= :start')->andWhere('t.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(FreeTimespan::class, 't')
            ->where('t.date >= :start')
            ->andWhere('t.date <= :end')
            ->setParameter('start', $start)
            ->setParameter('end', $end)
            ->getQuery()
            ->execute();
    }
}