WyriHaximus\React\Cake\Orm\Shell\GenerateShell

Complexity

Total Complexity

13

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	9

Test Coverage

Coverage

0%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	all()	0	16	6
A	iteratePath()	0	6	2
A	iterateClasses()	0	9	2
A	getClassByFile()	0	4	1
A	getOptionParser()	0	15	1
A	setupIterator()	0	13	1

<?php declare(strict_types=1);

namespace WyriHaximus\React\Cake\Orm\Shell;

use Cake\Console\Shell;
use Cake\Core\App;
use Cake\Core\Configure;
use Cake\Core\Plugin;
use Roave\BetterReflection\BetterReflection;
use Roave\BetterReflection\Reflector\ClassReflector;
use Roave\BetterReflection\SourceLocator\Type\SingleFileSourceLocator;
use WyriHaximus\React\Cake\Orm\AsyncTableGenerator;

class GenerateShell extends Shell
{
    public function all()
    {
        foreach (App::path('Model/Table') as $path) {
            if (is_dir($path)) {
                $this->iteratePath($path);
            }
        }

        foreach (Plugin::loaded() as $plugin) {

The expression \Cake\Core\Plugin::loaded() of type boolean|array is not guaranteed to be traversable. How about adding an additional type check?

            foreach (App::path('Model/Table', $plugin) as $path) {
                if (is_dir($path)) {
                    $this->iteratePath($path);
                }
            }
        }
    }

    public function iteratePath($path)
    {
        foreach ($this->setupIterator($path) as $item) {
            $this->iterateClasses($this->getClassByFile(current($item)));
        }
    }

    public function iterateClasses($classes)
    {
        foreach ($classes as $class) {
            $className = $class->getName();
            (new AsyncTableGenerator(
                Configure::read('WyriHaximus.React.Cake.Orm.Cache.AsyncTables')
            ))->generate($className, true);
        }
    }

    public function getClassByFile($fileName)
    {
        return (new ClassReflector(new SingleFileSourceLocator($fileName, (new BetterReflection())->astLocator())))->getAllClasses();
    }

    /**
     * Set options for this console.
     *
     * @return \Cake\Console\ConsoleOptionParser
     */
    // @codingStandardsIgnoreStart
    public function getOptionParser()
    {
        // @codingStandardsIgnoreEnd
        return parent::getOptionParser()->addSubcommand(

The expression parent::getOptionParser()->addSubcommand('all', array('short' => 'a', 'help' => __('Searches and pregenerates all async tables it finds.')))->description(__('Async table pregenerator')); of type Cake\Console\ConsoleOptionParser|string adds the type string to the return on line 64 which is incompatible with the return type of the parent method Cake\Console\Shell::getOptionParser of type Cake\Console\ConsoleOptionParser.

The method Cake\Console\ConsoleOptionParser::description() has been deprecated with message: 3.4.0 Use setDescription()/getDescription() instead.

            'all',
            [
                'short' => 'a',
                // @codingStandardsIgnoreStart
                'help' => __('Searches and pregenerates all async tables it finds.'),
                // @codingStandardsIgnoreEnd
            ]
        // @codingStandardsIgnoreStart
        )->description(__('Async table pregenerator'));
        // @codingStandardsIgnoreEnd
    }

    protected function setupIterator($path)
    {
        return new \RegexIterator(new \RecursiveIteratorIterator(
            new \RecursiveDirectoryIterator(
                $path,
                \FilesystemIterator::KEY_AS_PATHNAME |
                \FilesystemIterator::CURRENT_AS_FILEINFO |
                \FilesystemIterator::SKIP_DOTS
            ),
            \RecursiveIteratorIterator::CHILD_FIRST,
            \RecursiveIteratorIterator::CATCH_GET_CHILD
        ), '/.*?.php$/', \RegexIterator::GET_MATCH);
    }
}