TaskExecutionRepository::findByStartTime() - Code Metrics - Inspection of "Improved architecture of bundle" - php-task/TaskBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#15)

by Wachter

created 2016-08-29 17:36 UTC

TaskExecutionRepository::findByStartTime() A

↳ Parent: TaskExecutionRepository

Complexity

Conditions	2
Paths	2

Size

Total Lines	15
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

Metric	Value
c	3
b	0
f	0
dl	0
loc	15
rs	9.4285
cc	2
eloc	12
nc	2
nop	2

<?php

namespace Task\TaskBundle\Entity;

use Doctrine\ORM\EntityRepository;
use Doctrine\ORM\NoResultException;
use Task\Execution\TaskExecutionInterface;
use Task\TaskInterface;
use Task\TaskStatus;

class TaskExecutionRepository extends EntityRepository
{
    public function findByStartTime(TaskInterface $task, \DateTime $scheduleTime)
    {
        try {
            return $this->createQueryBuilder('e')
                ->innerJoin('e.task', 't')
                ->where('t.uuid = :uuid')
                ->andWhere('e.scheduleTime = :scheduleTime')
                ->setParameter('uuid', $task->getUuid())
                ->setParameter('scheduleTime', $scheduleTime)
                ->getQuery()
                ->getSingleResult();
        } catch (NoResultException $e) {
            return;
        }
    }

    public function findScheduled(\DateTime $dateTime = null)
    {
        if ($dateTime === null) {
            $dateTime = new \DateTime();
        }

        return $this->createQueryBuilder('e')
            ->innerJoin('e.task', 't')
            ->where('e.status = :status AND e.scheduleTime < :dateTime')
            ->setParameter('status', TaskStatus::PLANNED)
            ->setParameter('dateTime', $dateTime)
            ->getQuery()
            ->getResult();
    }

    public function save(TaskExecutionInterface $execution)

    {
        $this->_em->flush();
    }

    public function add(TaskExecutionInterface $execution)
    {
        $this->_em->persist($execution);
        $this->_em->flush();
    }
}


1			<?php
2
3			namespace Task\TaskBundle\Entity;
4
5			use Doctrine\ORM\EntityRepository;
6			use Doctrine\ORM\NoResultException;
7			use Task\Execution\TaskExecutionInterface;
8			use Task\TaskInterface;
9			use Task\TaskStatus;
10
11			class TaskExecutionRepository extends EntityRepository
12			{
13			public function findByStartTime(TaskInterface $task, \DateTime $scheduleTime)
14			{
15			try {
16			return $this->createQueryBuilder('e')
17			->innerJoin('e.task', 't')
18			->where('t.uuid = :uuid')
19			->andWhere('e.scheduleTime = :scheduleTime')
20			->setParameter('uuid', $task->getUuid())
21			->setParameter('scheduleTime', $scheduleTime)
22			->getQuery()
23			->getSingleResult();
24			} catch (NoResultException $e) {
25			return;
26			}
27			}
28
29			public function findScheduled(\DateTime $dateTime = null)
30			{
31			if ($dateTime === null) {
32			$dateTime = new \DateTime();
33			}
34
35			return $this->createQueryBuilder('e')
36			->innerJoin('e.task', 't')
37			->where('e.status = :status AND e.scheduleTime < :dateTime')
38			->setParameter('status', TaskStatus::PLANNED)
39			->setParameter('dateTime', $dateTime)
40			->getQuery()
41			->getResult();
42			}
43
44			public function save(TaskExecutionInterface $execution)
			0 ignored issues – show Unused Code introduced 2016-08-29 17:24 UTC by Report Bug Copy Issue Report The parameter `$execution` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
45			{
46			$this->_em->flush();
47			}
48
49			public function add(TaskExecutionInterface $execution)
50			{
51			$this->_em->persist($execution);
52			$this->_em->flush();
53			}
54			}
55

php-task / TaskBundle

Pull Request — master (#15)

TaskExecutionRepository::findByStartTime() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like