Slick\Orm\Repository\EntityRepository

Complexity

Total Complexity

4

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	get()	0	8	2
B	load()	0	24	2

<?php

/**
 * This file is part of slick/orm package
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Slick\Orm\Repository;

use Slick\Database\Sql;
use Slick\Orm\EntityInterface;
use Slick\Orm\EntityMapperInterface;
use Slick\Orm\RepositoryInterface;

/**
 * Class EntityRepository
 *
 * @package Slick\Orm\Repository
 * @author  Filipe Silva <silvam.filipe@gmail.com>
 */
class EntityRepository extends AbstractRepository implements
    RepositoryInterface
{

    /**
     * Gets an entity by its id
     *
     * @param mixed $entityId
     *
     * @return EntityInterface|null
     */
    public function get($entityId)
    {
        $entity = $this->getIdentityMap()->get($entityId, false);
        if ($entity === false) {
            $entity = $this->load($entity);
        }
        return $entity;
    }

    /**
     * Loads entity from database
     *
     * @param $entityId
     *
     * @return null|EntityInterface
     */
    protected function load($entityId)
    {
        $table = $this->getEntityDescriptor()->getTableName();
        $primaryKey = $this->getEntityDescriptor()
            ->getPrimaryKey()
            ->getField();

        $data = Sql::createSql($this->getAdapter())
            ->select($this->getEntityDescriptor()->getTableName())
            ->where(
                [

array("{$table}.{$primaryKey} = :id" => array(':id' => $entityId)) is of type array<string|integer,array<string,?,{":id":"?"}>>, but the function expects a string.

                    "{$table}.{$primaryKey} = :id" => [
                        ':id' => $entityId
                    ]
                ]
            )
            ->first();
        $entity = null;
        if ($data) {
            $entity = $this->getEntityMapper()->createFrom($data);
            $this->getIdentityMap()->set($entity);

It seems like $entity defined by $this->getEntityMapper()->createFrom($data) on line 69 can also be of type array<integer,object<Slick\Orm\EntityMapperInterface>> or object<Slick\Orm\Entity\EntityCollection>; however, Slick\Orm\Repository\IdentityMapInterface::set() does only seem to accept object<Slick\Orm\EntityInterface>, maybe add an additional type check?

        }
        return $entity;
    }
}