WyriHaximus\React\Cake\Orm\AsyncTableRegistry

Complexity

Total Complexity

9

Size/Duplication

Total Lines	80
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	7

Test Coverage

Coverage

28.57%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	4	1
B	get()	0	25	3
A	getInstance()	0	9	3
A	reset()	0	4	1
A	info()	0	4	1

<?php

namespace WyriHaximus\React\Cake\Orm;

use Cake\Core\App;
use Cake\Core\Configure;
use Cake\ORM\Table;
use React\EventLoop\LoopInterface;
use WyriHaximus\React\ChildProcess\Pool\PoolUtilizerInterface;

class AsyncTableRegistry implements PoolUtilizerInterface
{
    /**
     * @var AsyncTable[]
     */
    protected static $tables = [];

    /**
     * @var AsyncTableRegistry
     */
    protected static $instance = null;

    /**
     * @var boolean
     */
    protected static $reset = false;

    /**
     * @param LoopInterface $loop
     * @param array $config
     */
    public static function init(LoopInterface $loop, array $config = [])
    {
        Pool::getInstance($loop, $config);
    }

    /**
     * @param Table $table
     *
     * @return AsyncTable

The return type AsyncTable is a trait, and thus cannot be used for type-hinting in PHP. Maybe consider adding an interface and use that for type-hinting?

     */
    public static function get(Table $table)
    {
        $tableName = get_class($table);

        if (isset(static::$tables[$tableName])) {
            return static::$tables[$tableName];
        }

        $asyncTableName = (new AsyncTableGenerator(
            Configure::read('WyriHaximus.React.Cake.Orm.Cache.AsyncTables')
        ))->generate($tableName)->getFQCN();

        $asyncTable = new $asyncTableName();

        if ($asyncTable instanceof AsyncTableInterface) {
            $asyncTable->setUpAsyncTable(
                Pool::getInstance(),
                $table->table(),

It seems like $table->table() targeting Cake\ORM\Table::table() can also be of type boolean; however, WyriHaximus\React\Cake\Orm\AsyncTableInterface::setUpAsyncTable() does only seem to accept string, maybe add an additional type check?

                App::className($tableName, 'Model/Table', 'Table')

It seems like \Cake\Core\App::className($tableName, 'Model/Table', 'Table') targeting Cake\Core\App::className() can also be of type false; however, WyriHaximus\React\Cake\Orm\AsyncTableInterface::setUpAsyncTable() does only seem to accept string, did you maybe forget to handle an error condition?

            );
        }

        static::$tables[$tableName] = $asyncTable;
        return static::$tables[$tableName];
    }

    public static function getInstance()
    {
        if (null === self::$instance || self::$reset) {
            self::$instance = new static();
            self::$reset = false;
        }

        return self::$instance;
    }

    public static function reset()
    {
        self::$reset = true;
    }

    /**
     * @inheritDoc
     */
    public function info()
    {
        return Pool::getInstance()->info();
    }
}