allejo\stakx\Command\BuildableCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	65
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	5

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	10	1
A	execute()	0	10	1
A	configureBuild()	0	5	1
A	setServiceParameter()	0	4	1

<?php

/**
 * @copyright 2017 Vladimir Jimenez
 * @license   https://github.com/allejo/stakx/blob/master/LICENSE.md MIT
 */

namespace allejo\stakx\Command;

use allejo\stakx\Configuration;
use allejo\stakx\Service;
use allejo\stakx\System\Filesystem;
use allejo\stakx\Website;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * Class BuildableCommand.
 *
 * This abstract class handles configuring the website object
 */
abstract class BuildableCommand extends Command

BuildableCommand does not seem to conform to the naming convention (^Abstract|Factory$).

{
    const NO_CONF = 'no-conf';
    const NO_CLEAN = 'no-clean';
    const USE_DRAFTS = 'use-drafts';

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

    /** @var Website */
    protected $website;

    /** @var Filesystem */
    protected $fs;

Avoid variables with short names like $fs. Configured minimum length is 3.

    /**
     * {@inheritdoc}
     */
    protected function configure()
    {
        $this->fs = new Filesystem();

        $this->addOption('conf', 'c', InputOption::VALUE_REQUIRED, 'The configuration file to be used', $this->fs->absolutePath(Configuration::DEFAULT_NAME));
        $this->addOption('safe', 's', InputOption::VALUE_NONE, 'Disable file system access from Twig');
        $this->addOption(self::NO_CONF, 'l', InputOption::VALUE_NONE, 'Build a Stakx website without a configuration file');
        $this->addOption(self::NO_CLEAN, 'x', InputOption::VALUE_NONE, "Don't clean the _site before recompiling the website");
        $this->addOption(self::USE_DRAFTS, 'd', InputOption::VALUE_NONE, 'Ignore `output: false` in ContentItems and always write a file');
    }

    /**
     * {@inheritdoc}
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $this->website = new Website($output);
        $this->website->setConfLess($input->getOption(self::NO_CONF));
        $this->website->setNoClean($input->getOption(self::NO_CLEAN));

        $this->setServiceParameter($input, self::NO_CONF);
        $this->setServiceParameter($input, self::NO_CLEAN);
        $this->setServiceParameter($input, self::USE_DRAFTS);
    }

    /**
     * Configure the website builder.
     *
     * @param InputInterface $input
     */
    protected function configureBuild(InputInterface $input)
    {
        $this->website->setConfiguration($input->getOption('conf'));
        $this->website->setSafeMode($input->getOption('safe'));
    }

    /**
     * Set a parameter to the Service singleton
     *
     * @param InputInterface $input
     * @param string         $param
     */
    private function setServiceParameter(InputInterface $input, $param)
    {
        Service::setParameter($param, $input->getOption($param));
    }
}