SymphonyCms\Installer\Lib\ConsoleInstaller

Complexity

Total Complexity

8

Size/Duplication

Total Lines	112
Duplicated Lines	2.68 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	2
Bugs	1	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	setOverride()	0	6	1
B	install()	3	37	5
A	bootstrap()	0	20	1

<?php

namespace SymphonyCms\Installer\Lib;

use Configuration;
use DateTimeObj;
use Exception;
use Lang;
use Psr\Log\LoggerInterface;
use Symphony;
use SymphonyCms\Installer\Steps;

class ConsoleInstaller
{
    /**
     * @var LoggerInterface
     */
    protected $logger;

    /**
     * @var Configuration
     */
    protected $configuration;

    /**
     * @var bool
     */
    protected $overwrite = false;

    /**
     * @param LoggerInterface $logger
     * @param Configuration $configuration
     */
    public function __construct(LoggerInterface $logger, Configuration $configuration)
    {
        $this->logger = $logger;
        $this->configuration = $configuration;

        // Symphony expects a couple of constants to always exist, so lets do a basic bootstrap.
        $this->bootstrap();
    }

    /**
     * Allow the Installer and it's steps to attempt to install Symphony at all costs,
     * including removing and deleting existing data. Use with caution.
     *
     * @param bool $overwrite
     * @return $this
     */
    public function setOverride($overwrite = false)
    {
        $this->overwrite = $overwrite;

        return $this;
    }

    /**
     * Installs Symphony using the configuration information available.
     *
     * @param array $data
     * @return bool
     */
    public function install(array $data)
    {
        $steps = [
            // Create database
            Steps\CreateDatabase::class,
            // Create manifest folder structure
            Steps\CreateManifest::class,
            // Write .htaccess
            Steps\CreateHtaccess::class,
            // Create or import the workspace
            Steps\Workspace::class,
            // Enable extensions
            Steps\EnableExtensions::class
        ];

        try {
            foreach ($steps as $step) {
                $installStep = new $step($this->logger);
                $installStep->setOverride($this->overwrite);

                if (false === $installStep->handle($this->configuration, $data)) {
                    throw new Exception(sprintf('Aborting installation, %s step failed.', $step));
                }
            }
        } catch (Exception $ex) {
            $this->logger->error($ex->getMessage());
            return false;
        }

        if (false === Symphony::Configuration()->write(CONFIG, $this->configuration->get('write_mode', 'file'))) {

$this->configuration->get('write_mode', 'file') is of type array|string, but the function expects a integer|null.

            $this->logger->error('Could not create config file ‘' . CONFIG . '’. Check permission on /manifest.');
            return false;
        }

        $this->logger->info('Symphony has been installed.');
        return true;
    }

    /**
     * Bootstrap the environment to the bare minimum for Symphony to 'run'.
     */
    private function bootstrap()
    {
        // Initialize date/time
        define_safe('__SYM_DATE_FORMAT__', $this->configuration->get('date_format', 'region'));

It seems like $this->configuration->get('date_format', 'region') targeting Configuration::get() can also be of type array; however, define_safe() does only seem to accept string, maybe add an additional type check?

        define_safe('__SYM_TIME_FORMAT__', $this->configuration->get('time_format', 'region'));

It seems like $this->configuration->get('time_format', 'region') targeting Configuration::get() can also be of type array; however, define_safe() does only seem to accept string, maybe add an additional type check?

        define_safe(
            '__SYM_DATETIME_FORMAT__',
            __SYM_DATE_FORMAT__ . $this->configuration->get('datetime_separator', 'region') . __SYM_TIME_FORMAT__
        );
        DateTimeObj::setSettings($this->configuration->get('region'));

It seems like $this->configuration->get('region') targeting Configuration::get() can also be of type string; however, DateTimeObj::setSettings() does only seem to accept array, maybe add an additional type check?

        Lang::initialize();
        Symphony::setDatabase();
        Symphony::initialiseExtensionManager();
        Symphony::initialiseConfiguration($this->configuration->get());

It seems like $this->configuration->get() targeting Configuration::get() can also be of type string; however, Symphony::initialiseConfiguration() does only seem to accept array, maybe add an additional type check?

        define('INSTALL', DOCROOT . '/install');
        define('INSTALL_LOGS', MANIFEST . '/logs');
        define('INSTALL_URL', URL . '/install');
    }
}