Wandu\Database\Manager

Complexity

Total Complexity

11

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	6

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	2
A	setEventDispatcher()	0	4	1
A	connect()	0	17	4
A	connection()	0	4	2
A	repository()	0	8	2

<?php
namespace Wandu\Database;

use Wandu\Caster\Caster;
use Wandu\Caster\CastManagerInterface;
use Wandu\Database\Connection\MysqlConnection;
use Wandu\Database\Contracts\ConnectionInterface;
use Wandu\Database\Contracts\Entity\MetadataReaderInterface;
use Wandu\Database\Exception\DriverNotFoundException;
use Wandu\Database\Repository\Repository;
use Wandu\Event\DispatcherInterface;

class Manager
{
    /** @var \Wandu\Database\Contracts\Entity\MetadataReaderInterface */
    protected $reader;
    
    /** @var \Wandu\Caster\Caster */
    protected $caster;
    
    /** @var \Wandu\Event\DispatcherInterface */
    protected $dispatcher;
    
    /** @var \Wandu\Database\Contracts\ConnectionInterface[] */
    protected $connections = [];
    
    /** @var \Wandu\Database\Repository\Repository[] */
    protected $repositories = [];

    public function __construct(
        MetadataReaderInterface $reader,
        CastManagerInterface $caster = null
    ) {
        $this->reader = $reader;
        $this->caster = $caster ?: new Caster();

$caster ?: new \Wandu\Caster\Caster() is of type object<Wandu\Caster\CastManagerInterface>, but the property $caster was declared to be of type object<Wandu\Caster\Caster>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    /**
     * @param \Wandu\Event\DispatcherInterface $dispatcher
     */
    public function setEventDispatcher(DispatcherInterface $dispatcher)
    {
        $this->dispatcher = $dispatcher;
    }
    
    /**
     * @param array|\Wandu\Database\Configuration|\Wandu\Database\Contracts\ConnectionInterface $connection
     * @param string $name
     * @return \Wandu\Database\Contracts\ConnectionInterface
     */
    public function connect($connection, $name = 'default')
    {
        if (!$connection instanceof Configuration) {
            $connection = new Configuration($connection);

It seems like $connection defined by new \Wandu\Database\Configuration($connection) on line 54 can also be of type object<Wandu\Database\Contracts\ConnectionInterface>; however, Wandu\Database\Configuration::__construct() does only seem to accept array, maybe add an additional type check?

        }
        if (!$connection instanceof ConnectionInterface) {
            switch ($connection->getDriver()) {
                case Configuration::DRIVER_MYSQL:
                    $connection = new MysqlConnection($connection, $this->dispatcher);
                    break;
                default:
                    throw new DriverNotFoundException($connection->getDriver());
            }
        }
        $connection->connect();
        return $this->connections[$name] = $connection;
    }

    /**
     * @param string $name
     * @return \Wandu\Database\Contracts\ConnectionInterface
     */
    public function connection($name = 'default')
    {
        return isset($this->connections[$name]) ? $this->connections[$name] : null;
    }

    /**
     * @param string $class
     * @param string $connection
     * @return \Wandu\Database\Repository\Repository
     */
    public function repository(string $class, string $connection = 'default'): Repository
    {
        $repoName = "{$class}@{$connection}";
        if (!isset($this->repositories[$repoName])) {
            $this->repositories[$repoName] = new Repository($this, $this->reader->getMetadataFrom($class), $this->caster);
        }
        return $this->repositories[$repoName];
    }
}