Black\Page\Infrastructure\Persistence\DoctrineORMRepository

Complexity

Total Complexity

8

Size/Duplication

Total Lines	72
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	find()	13	13	2
A	findBySlug()	13	13	2
A	findAll()	4	4	1
A	add()	5	5	1
A	remove()	6	6	1
A	update()	4	4	1

<?php

namespace Black\Page\Infrastructure\Persistence;

use Black\Bridge\Doctrine\Common\Persistence\ORMRepository;
use Black\Page\Domain\Model\WebPage;
use Black\Page\Domain\Model\WebPageId;
use Black\Page\Domain\Model\WebPageRepository;
use Doctrine\ORM\NoResultException;

/**
 * Class DoctrineORMRepository
 */
class DoctrineORMRepository extends ORMRepository implements WebPageRepository

There is one abstract method getClassName in this class; you could implement it, or declare this class as abstract.

This class seems to be duplicated in your project.

{
    /**
     * @param mixed $id
     * @return mixed
     * @throws \Doctrine\ORM\NoResultException
     * @throws \Doctrine\ORM\NonUniqueResultException
     */
    public function find(WebPageId $id)
    {
        $query = $this->getQueryBuilder()
            ->where('p.webPageId.value = :id')
            ->setParameter('id', $id->getValue())
            ->getQuery();

        try {
            return $query->getSingleResult();
        } catch (NoResultException $exception) {

The class Doctrine\ORM\NoResultException does not exist. Did you forget a USE statement, or did you not list all dependencies?

            return null;
        }
    }

    /**
     * @param $slug
     * @return mixed|null
     */
    public function findBySlug($slug)
    {
        $query = $this->getQueryBuilder()
            ->where('p.slug = :slug')
            ->setParameter('slug', $slug)
            ->getQuery();

        try {
            return $query->getSingleResult();
        } catch (NoResultException $exception) {

The class Doctrine\ORM\NoResultException does not exist. Did you forget a USE statement, or did you not list all dependencies?

            return null;
        }
    }

    /**
     * @return mixed
     */
    public function findAll()
    {
        return $this->getQueryBuilder()->getQuery()->execute();
    }

    /**
     * @param WebPage $webpage
     */
    public function add(WebPage $webpage)
    {
        $this->manager->persist($webpage);
        $this->update($webpage);
    }

    /**
     * @param WebPage $webpage
     */
    public function remove(WebPage $webpage)
    {
        $this->manager->remove($webpage);
        $this->update($webpage);

    }

    public function update(WebPage $webpage)
    {
        $this->manager->flush();
    }
}