PolderKnowledge\EntityService\Repository\Doctrine\CollectionRepository

Complexity

Total Complexity

13

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

61.11%

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	delete()	0	5	1
A	deleteBy()	0	9	2
A	countBy()	0	4	1
A	find()	0	8	1
A	findAll()	0	4	1
A	findBy()	0	4	1
A	findOneBy()	0	4	1
A	persist()	0	9	3
A	getPersisted()	0	4	1

<?php

namespace PolderKnowledge\EntityService\Repository\Doctrine;

use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\Common\Collections\Collection;
use Doctrine\Common\Collections\Criteria;
use PolderKnowledge\EntityService\Repository\EntityRepositoryInterface;
use PolderKnowledge\EntityService\Repository\Feature\DeletableInterface;
use PolderKnowledge\EntityService\Repository\Feature\ReadableInterface;
use PolderKnowledge\EntityService\Repository\Feature\TransactionAwareInterface;
use PolderKnowledge\EntityService\Repository\Feature\WritableInterface;
use UnexpectedValueException;

class CollectionRepository implements
    EntityRepositoryInterface,
    DeletableInterface,
    ReadableInterface,
    WritableInterface
{
    private $allItems;

    private $persisted;

    public function __construct(array $initialItems = [])
    {
        $this->allItems = new ArrayCollection($initialItems);
        $this->persisted = new ArrayCollection;
    }

    public function delete($entity)
    {
        $this->allItems->removeElement($entity);
        $this->persisted->removeElement($entity);
    }

    public function deleteBy($criteria)
    {
        $itemsToDelete = $this->allItems->matching($criteria);

It seems like $criteria defined by parameter $criteria on line 37 can also be of type array; however, Doctrine\Common\Collections\ArrayCollection::matching() does only seem to accept object<Doctrine\Common\Collections\Criteria>, maybe add an additional type check?

        foreach ($itemsToDelete as $item) {
            $this->allItems->removeElement($item);
            $this->persisted->removeElement($item);
        }
    }

    public function countBy($criteria)
    {
        return $this->allItems->matching($criteria)->count();

It seems like $criteria defined by parameter $criteria on line 47 can also be of type array; however, Doctrine\Common\Collections\ArrayCollection::matching() does only seem to accept object<Doctrine\Common\Collections\Criteria>, maybe add an additional type check?

    }

    public function find($id)
    {
        return $this->allItems->filter(
            function ($entity) use ($id) {
                return $entity->getId() === $id;
            }
        )->first();
    }

    public function findAll()
    {
        return $this->allItems->toArray();
    }

    public function findBy($criteria)
    {
        return $this->allItems->matching($criteria)->toArray();

It seems like $criteria defined by parameter $criteria on line 66 can also be of type array; however, Doctrine\Common\Collections\ArrayCollection::matching() does only seem to accept object<Doctrine\Common\Collections\Criteria>, maybe add an additional type check?

    }

    public function findOneBy($criteria)
    {
        return $this->allItems->matching($criteria)->first();

It seems like $criteria defined by parameter $criteria on line 71 can also be of type array; however, Doctrine\Common\Collections\ArrayCollection::matching() does only seem to accept object<Doctrine\Common\Collections\Criteria>, maybe add an additional type check?

    }

    public function persist($entity)
    {
        if (!$this->allItems->contains($entity)) {
            $this->allItems->add($entity);
        }
        if (!$this->persisted->contains($entity)) {
            $this->persisted->add($entity);
        }
    }

    /**
     * Method outside of the standard interface, for testing purposes
     */
    public function getPersisted(): ArrayCollection
    {
        return clone $this->persisted;
    }
}