ORM\Testing\EntityManagerMock::fetch()

Complexity

Conditions	4
Paths	4

Size

Total Lines	20
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function fetch($class, $primaryKey = null)
    {
        $reflection = new \ReflectionClass($class);
        if (!$reflection->isSubclassOf(Entity::class)) {
            throw new NoEntity($class . ' is not a subclass of Entity');
        }
        /** @var string|Entity $class */

        if ($primaryKey === null) {
            return new EntityFetcherMock($this, $class);
        }

        $primaryKey = $this->buildPrimaryKey($class, (array)$primaryKey);
        $checksum = $this->buildChecksum($primaryKey);

        if (isset($this->map[$class][$checksum])) {
            return $this->map[$class][$checksum];
        }

        return EntityFetcherMock::retrieve($class, $primaryKey);

It seems like $class can also be of type ORM\Entity; however, parameter $class of ORM\Testing\EntityFetcherMock::retrieve() does only seem to accept string, maybe add an additional type check?

    }