BFW\Install\Application

Complexity

Total Complexity

18

Size/Duplication

Total Lines	145
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	4

Importance

Changes	1
Bugs	0	Features	0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	getModulesInstall()	0	4	1
A	initRequest()	0	4	1
A	initSession()	0	4	1
A	initErrors()	0	4	1
A	initCli()	0	4	1
A	addModuleInstall()	0	6	1
A	declareRunSteps()	0	8	1
A	run()	0	8	1
B	installAllModules()	0	20	5
B	installModule()	0	24	5

<?php

namespace BFW\Install;

use \BFW\Helpers\Cli;
use \BFW\Install\ModuleInstall;

/**
 * Application class for install module script
 */
class Application extends \BFW\Application
{
    /**
     * @var \BFW\Install\ModuleInstall[] $modulesInstall Modules to install
     */
    protected static $modulesInstall = [];
    
    /**
     * Getter to static property modulesInstall
     * 
     * @return \BFW\Install\ModuleInstall[]
     */
    public static function getModulesInstall()
    {
        return self::$modulesInstall;
    }
    
    /**
     * {@inheritdoc}
     */
    protected function initRequest()
    {
        return;
    }

    /**
     * {@inheritdoc}
     */
    protected function initSession()
    {
        return;
    }

    /**
     * {@inheritdoc}
     */
    protected function initErrors()
    {
        return;
    }

    /**
     * {@inheritdoc}
     */
    protected function initCli()
    {
        return;
    }
    
    /**
     * Add a new module in the list to install
     * 
     * @param \BFW\Install\ModuleInstall $module
     * 
     * @return void
     */
    public static function addModuleInstall(ModuleInstall $module)
    {
        $moduleName = $module->getName();
        
        self::$modulesInstall[$moduleName] = $module;
    }

    /**
     * {@inheritdoc}
     */
    protected function declareRunSteps()
    {
        $this->runSteps = [
            [$this, 'loadMemcached'],
            [$this, 'loadAllModules'],
            [$this, 'installAllModules']
        ];
    }

    /**
     * {@inheritdoc}
     */
    public function run()
    {
        $runTasks = $this->getSubjectForName('ApplicationTasks');
        
        $runTasks->setNotifyPrefix('BfwAppModulesInstall');

The method setNotifyPrefix() does not exist on BFW\Subjects. Did you maybe mean notify()?

        $runTasks->run();
        $runTasks->sendNotify('bfw_modules_install_done');

The method sendNotify() does not exist on BFW\Subjects. Did you maybe mean notify()?

    }
    
    /**
     * Install all modules in the order of the dependency tree.
     * 
     * @return void
     */
    protected function installAllModules()
    {
        Cli::displayMsgNL('Read all modules to run install script...');
        
        $tree = $this->modules->getLoadTree();

        foreach ($tree as $firstLine) {
            foreach ($firstLine as $secondLine) {
                foreach ($secondLine as $moduleName) {
                    if (!isset(self::$modulesInstall[$moduleName])) {
                        continue;
                    }
                    
                    $this->installModule($moduleName);
                }
            }
        }
        
        Cli::displayMsgNL('All modules have been read.');
    }
    
    /**
     * Install a module
     * 
     * @param string $moduleName The module name
     * 
     * @return void
     */
    protected function installModule($moduleName)
    {
        if (!isset(self::$modulesInstall[$moduleName])) {
            return;
        }
        
        Cli::displayMsgNL(' > Read for module '.$moduleName);
        
        $module         = self::$modulesInstall[$moduleName];
        $installScripts = $module->getSourceInstallScript();
        
        if ($installScripts === '') {
            Cli::displayMsgNL(' >> No script to run.');
            return;
        }
        
        if (is_string($installScripts)) {
            $installScripts = (array) $installScripts;
        }
        
        foreach ($installScripts as $scriptPath) {

The expression $installScripts of type array|boolean is not guaranteed to be traversable. How about adding an additional type check?

            $module->runInstallScript($scriptPath);
        }
    }
}