Mouf\Database\TDBM\Controllers\TdbmController

Complexity

Total Complexity

14

Size/Duplication

Total Lines	157
Duplicated Lines	14.65 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes	13
Bugs	0	Features	2

3 Methods

Rating	Name	Duplication	Size	Complexity
B	defaultAction()	17	44	5
A	generate()	0	9	1
B	generateDaos()	6	59	8

<?php

namespace Mouf\Database\TDBM\Controllers;

use Mouf\Composer\ClassNameMapper;
use Mouf\Controllers\AbstractMoufInstanceController;
use Mouf\Database\TDBM\TDBMService;
use Mouf\Database\TDBM\Utils\TDBMDaoGenerator;
use Mouf\MoufManager;
use Mouf\InstanceProxy;

/**
 * The controller to generate automatically the Beans, Daos, etc...
 * Sweet!
 *
 * @Component
 */
class TdbmController extends AbstractMoufInstanceController
{
    /**
     * @var HtmlBlock
     */
    public $content;

    protected $daoNamespace;
    protected $beanNamespace;
    protected $daoFactoryName;
    protected $daoFactoryInstanceName;
    protected $autoloadDetected;
    protected $storeInUtc;

    /**
     * Admin page used to display the DAO generation form.
     *
     * @Action
     * //@Admin
     */
    public function defaultAction($name, $selfedit = 'false')
    {
        $this->initController($name, $selfedit);

        // Fill variables
        if ($this->moufManager->getVariable('tdbmDefaultSourceDirectory_'.$name) != null) {
            $this->daoNamespace = $this->moufManager->getVariable('tdbmDefaultDaoNamespace_'.$name);
            $this->beanNamespace = $this->moufManager->getVariable('tdbmDefaultBeanNamespace_'.$name);
            $this->daoFactoryName = $this->moufManager->getVariable('tdbmDefaultDaoFactoryName_'.$name);
            $this->daoFactoryInstanceName = $this->moufManager->getVariable('tdbmDefaultDaoFactoryInstanceName_'.$name);
            $this->storeInUtc = $this->moufManager->getVariable('tdbmDefaultStoreInUtc_'.$name);
            $this->useCustomComposer = $this->moufManager->getVariable('tdbmDefaultUseCustomComposer_'.$name);
            $this->composerFile = $this->moufManager->getVariable('tdbmDefaultComposerFile_'.$name);
        } else {
            $this->daoNamespace = $this->moufManager->getVariable('tdbmDefaultDaoNamespace');
            $this->beanNamespace = $this->moufManager->getVariable('tdbmDefaultBeanNamespace');
            $this->daoFactoryName = $this->moufManager->getVariable('tdbmDefaultDaoFactoryName');
            $this->daoFactoryInstanceName = $this->moufManager->getVariable('tdbmDefaultDaoFactoryInstanceName');
            $this->storeInUtc = $this->moufManager->getVariable('tdbmDefaultStoreInUtc');
            $this->useCustomComposer = $this->moufManager->getVariable('tdbmDefaultUseCustomComposer');
            $this->composerFile = $this->moufManager->getVariable('tdbmDefaultComposerFile');
        }

        if ($this->daoNamespace == null && $this->beanNamespace == null) {

This code seems to be duplicated across your project.

            $classNameMapper = ClassNameMapper::createFromComposerFile(__DIR__.'/../../../../../../../../composer.json');

            $autoloadNamespaces = $classNameMapper->getManagedNamespaces();
            if ($autoloadNamespaces) {
                $this->autoloadDetected = true;
                $rootNamespace = $autoloadNamespaces[0];
                $this->daoNamespace = $rootNamespace.'Dao';
                $this->beanNamespace = $rootNamespace.'Dao\\Bean';
            } else {
                $this->autoloadDetected = false;
                $this->daoNamespace = 'YourApplication\\Dao';
                $this->beanNamespace = 'YourApplication\\Dao\\Bean';
            }
        } else {
            $this->autoloadDetected = true;
        }

        $this->content->addFile(dirname(__FILE__).'/../../../../views/tdbmGenerate.php', $this);
        $this->template->toHtml();
    }

    /**
     * This action generates the DAOs and Beans for the TDBM service passed in parameter.
     *
     * @Action
     *
     * @param string $name
     * @param bool   $selfedit
     */
    public function generate($name, $daonamespace, $beannamespace, $daofactoryclassname, $daofactoryinstancename, $storeInUtc = 0, $selfedit = 'false', $useCustomComposer = false, $composerFile = '')
    {
        $this->initController($name, $selfedit);

        self::generateDaos($this->moufManager, $name, $daonamespace, $beannamespace, $daofactoryclassname, $daofactoryinstancename, $selfedit, $storeInUtc, $useCustomComposer, $composerFile);

        // TODO: better: we should redirect to a screen that list the number of DAOs generated, etc...
        header('Location: '.ROOT_URL.'ajaxinstance/?name='.urlencode($name).'&selfedit='.$selfedit);
    }

    /**
     * This function generates the DAOs and Beans for the TDBM service passed in parameter.
     *
     * @param MoufManager $moufManager
     * @param string      $name
     * @param string      $daonamespace
     * @param string      $beannamespace
     * @param string      $daofactoryclassname
     * @param string      $daofactoryinstancename
     * @param string      $selfedit
     * @param bool        $storeInUtc
     *
     * @throws \Mouf\MoufException
     */
    public static function generateDaos(MoufManager $moufManager, $name, $daonamespace, $beannamespace, $daofactoryclassname, $daofactoryinstancename, $selfedit = 'false', $storeInUtc = null, $useCustomComposer = null, $composerFile = null)
    {
        $moufManager->setVariable('tdbmDefaultDaoNamespace_'.$name, $daonamespace);
        $moufManager->setVariable('tdbmDefaultBeanNamespace_'.$name, $beannamespace);
        $moufManager->setVariable('tdbmDefaultDaoFactoryName_'.$name, $daofactoryclassname);
        $moufManager->setVariable('tdbmDefaultDaoFactoryInstanceName_'.$name, $daofactoryinstancename);
        $moufManager->setVariable('tdbmDefaultStoreInUtc_'.$name, $storeInUtc);
        $moufManager->setVariable('tdbmDefaultUseCustomComposer_'.$name, $useCustomComposer);
        $moufManager->setVariable('tdbmDefaultComposerFile_'.$name, $composerFile);

        // In case of instance renaming, let's use the last used settings
        $moufManager->setVariable('tdbmDefaultDaoNamespace', $daonamespace);
        $moufManager->setVariable('tdbmDefaultBeanNamespace', $beannamespace);
        $moufManager->setVariable('tdbmDefaultDaoFactoryName', $daofactoryclassname);
        $moufManager->setVariable('tdbmDefaultDaoFactoryInstanceName', $daofactoryinstancename);
        $moufManager->setVariable('tdbmDefaultStoreInUtc', $storeInUtc);
        $moufManager->setVariable('tdbmDefaultUseCustomComposer', $useCustomComposer);
        $moufManager->setVariable('tdbmDefaultComposerFile', $composerFile);

        // Remove first and last slash in namespace.
        if (strpos($daonamespace, '\\') === 0) {
            $daonamespace = substr($daonamespace, 1);
        }
        if (strpos($daonamespace, '\\') === strlen($daonamespace) - 1) {

This code seems to be duplicated across your project.

            $daonamespace = substr($daonamespace, 0, strlen($daonamespace) - 1);
        }
        if (strpos($beannamespace, '\\') === 0) {
            $beannamespace = substr($beannamespace, 1);
        }
        if (strpos($beannamespace, '\\') === strlen($beannamespace) - 1) {

This code seems to be duplicated across your project.

            $beannamespace = substr($beannamespace, 0, strlen($beannamespace) - 1);
        }

        $tdbmService = new InstanceProxy($name);
        /* @var $tdbmService TDBMService */
        $tables = $tdbmService->generateAllDaosAndBeans($daofactoryclassname, $daonamespace, $beannamespace, $storeInUtc, ($useCustomComposer ? $composerFile : null));

It seems like $storeInUtc defined by parameter $storeInUtc on line 115 can also be of type null; however, Mouf\Database\TDBM\TDBMService::generateAllDaosAndBeans() does only seem to accept boolean, maybe add an additional type check?

        $moufManager->declareComponent($daofactoryinstancename, $daonamespace.'\\'.$daofactoryclassname, false, MoufManager::DECLARE_ON_EXIST_KEEP_INCOMING_LINKS);

        $tableToBeanMap = [];

        //$tdbmServiceDescriptor = $moufManager->getInstanceDescriptor('tdbmService');

64% of this comment could be valid code. Did you maybe forget this after debugging?

        foreach ($tables as $table) {
            $daoName = TDBMDaoGenerator::getDaoNameFromTableName($table);

            $instanceName = TDBMDaoGenerator::toVariableName($daoName);
            if (!$moufManager->instanceExists($instanceName)) {
                $moufManager->declareComponent($instanceName, $daonamespace.'\\'.$daoName);
            }
            $moufManager->setParameterViaConstructor($instanceName, 0, $name, 'object');
            $moufManager->bindComponentViaSetter($daofactoryinstancename, 'set'.$daoName, $instanceName);

            $tableToBeanMap[$table] = $beannamespace.'\\'.TDBMDaoGenerator::getBeanNameFromTableName($table);
        }
        $tdbmServiceDescriptor = $moufManager->getInstanceDescriptor($name);
        $tdbmServiceDescriptor->getSetterProperty('setTableToBeanMap')->setValue($tableToBeanMap);
        $moufManager->rewriteMouf();
    }
}