Agavi\Config\DatabaseConfigHandler::execute()

Complexity

Conditions	13
Paths	63

Size

Total Lines	74
Code Lines	39

Duplication

Lines	5
Ratio	6.76 %

Importance

Changes

0

<?php
    public function execute(XmlConfigDomDocument $document)
    {
        // set up our default namespace
        $document->setDefaultNamespace(self::XML_NAMESPACE, 'databases');
        
        $databases = array();
        $default = null;
        foreach ($document->getConfigurationElements() as $configuration) {
            if (!$configuration->hasChildren('databases')) {
                continue;
            }
            
            $databasesElement = $configuration->getChild('databases');
            
            // make sure we have a default database exists
            if (!$databasesElement->hasAttribute('default') && $default === null) {
                // missing default database
                $error = 'Configuration file "%s" must specify a default database configuration';
                $error = sprintf($error, $document->documentURI);

                throw new ParseException($error);
            }
            if ($databasesElement->hasAttribute('default')) {
                $default = $databasesElement->getAttribute('default');
            }

            // let's do our fancy work
            /** @var XmlConfigDomElement $database */
            foreach ($configuration->get('databases') as $database) {
                $name = $database->getAttribute('name');

                if (!isset($databases[$name])) {
                    $databases[$name] = array('parameters' => array());

                    if (!$database->hasAttribute('class')) {
                        $error = 'Configuration file "%s" specifies database "%s" with missing class key';
                        $error = sprintf($error, $document->documentURI, $name);

                        throw new ParseException($error);
                    }
                }

                $databases[$name]['class'] = $database->hasAttribute('class') ? $database->getAttribute('class') : $databases[$name]['class'];

                $databases[$name]['parameters'] = $database->getAgaviParameters($databases[$name]['parameters']);
            }
        }

        if (!$databases) {

The expression $databases of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            // we have no connections
            $error = 'Configuration file "%s" does not contain any database connections.';
            $error = sprintf($error, $document->documentURI);
            throw new ConfigurationException($error);
        }

        $data = array();

        foreach ($databases as $name => $db) {
            // append new data
            $data[] = sprintf('$database = new %s();', $db['class']);
            $data[] = sprintf('$this->databases[%s] = $database;', var_export($name, true));
            $data[] = sprintf('$database->initialize($this, %s);', var_export($db['parameters'], true));
        }

        if (!isset($databases[$default])) {

This code seems to be duplicated across your project.

            $error = 'Configuration file "%s" specifies undefined default database "%s".';
            $error = sprintf($error, $document->documentURI, $default);
            throw new ConfigurationException($error);
        }

        $data[] = sprintf('$this->defaultDatabaseName = %s;', var_export($default, true));

        return $this->generate($data, $document->documentURI);
    }
}