Infuse\Console\OptimizeCommand

Complexity

Total Complexity

9

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	6	1
A	execute()	0	14	2
A	cacheRouteTable()	0	11	1
A	cacheConfig()	0	33	5

<?php

/**
 * @author Jared King <j@jaredtking.com>
 *
 * @link http://jaredtking.com
 *
 * @copyright 2015 Jared King
 * @license MIT
 */
namespace Infuse\Console;

use Infuse\HasApp;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

class OptimizeCommand extends Command
{
    use HasApp;

    protected function configure()
    {
        $this
            ->setName('optimize')
            ->setDescription('Optimizes the app');
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $output->writeln('Optimizing app');

        if ($this->app['config']->get('router.cacheFile')) {
            $this->cacheRouteTable($output);
        } else {
            $output->writeln('The route table could be cached with the router.cacheFile setting');
        }

        $this->cacheConfig($output);

        return 0;
    }

    private function cacheRouteTable(OutputInterface $output)
    {
        $output->writeln('-- Caching route table');

        $cacheFile = $this->app['config']->get('router.cacheFile');
        @unlink($cacheFile);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        $this->app['router']->getDispatcher();

        $output->writeln('Success!');
    }

    private function cacheConfig(OutputInterface $output)
    {
        $output->writeln('-- Caching configuration');

        $configFile = INFUSE_BASE_DIR.'/config.php';
        $originalConfigFile = INFUSE_BASE_DIR.'/config.original.php';

        // we are going to cache the already loaded config,
        // unless the loaded config comes from a cached version
        $settings = $this->app['config']->all();
        if (file_exists($originalConfigFile)) {
            $settings = include $originalConfigFile;
        }

        // build a hard-coded version of the application configuration
        // that does not include any configuration building logic
        $configStr = "<?php\n// THIS FILE IS AUTO-GENERATED\n";
        $configStr .= 'return ';
        $configStr .= var_export($settings, true);
        $configStr .= ';';

        // move the original config file to a backup
        if (file_exists($configFile) && !file_exists($originalConfigFile)) {
            rename($configFile, $originalConfigFile);
        }

        // flush out 
        if (file_put_contents($configFile, $configStr)) {
            $output->writeln('Success!');
        } else {
            $output->writeln('Could not write config.php');
        }
    }
}