RebuildSettingsCommand::mergeSettings() - Code Metrics - Inspection of "new feautures plus refactoring" - mosiyash/taisiya-core-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c4223d...e3af10 )

by Nikita

created 2016-10-14 08:53 UTC

RebuildSettingsCommand::mergeSettings() C

↳ Parent: RebuildSettingsCommand

Complexity

Conditions	9
Paths	256

Size

Total Lines	56
Code Lines	37

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	56
rs	5.3148
c	0
b	0
f	0
cc	9
eloc	37
nc	256
nop	1

How to fix Long Method

<?php

namespace Taisiya\CoreBundle\Console\Command\Config;

use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Finder\Finder;
use Symfony\Component\Finder\SplFileInfo;
use Taisiya\CoreBundle\Console\Command\Command;
use Taisiya\CoreBundle\Console\Style\TaisiyaStyle;
use Taisiya\CoreBundle\Exception\RuntimeException;

final class RebuildSettingsCommand extends Command
{
    /**
     * {@inheritdoc}
     */
    protected function configure()
    {
        $this->setName('config:rebuild-settings')
            ->setDescription('Rebuild project settings');
    }

    /**
     * @param InputInterface $input
     * @param OutputInterface $output
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $io = new TaisiyaStyle($input, $output);

        $this->copySettings($io);
        $this->mergeSettings($io);
    }

    /**
     * @param TaisiyaStyle $io
     * @throws RuntimeException
     */
    final protected function copySettings(TaisiyaStyle $io): void

    {
        $io->isVerbose() && $io->writeln('Copy bundles settings to project');

        $finder = Finder::create()
            ->in(TAISIYA_ROOT)
            ->path('vendor')
            ->files()
            ->name('settings.default.php')
            ->filter(function (SplFileInfo $file) {
                return (bool) preg_match('/\/app\/config\//', $file->getPathname());
            });

        foreach ($finder as $file) {exit(var_dump($file->getPathname()));

            $dest = preg_replace('/^.+?\/vendor\/(.+?)\/(.+?)\/.+?$/', TAISIYA_ROOT.'/app/config/\1.\2.settings.default.php', $file->getPathname());
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
            if (!copy($file->getPathname(), $dest)) {
                throw new RuntimeException('Couldn\'t copy '.str_replace(TAISIYA_ROOT.'/', '', $file->getPathname()).' to '.str_replace(TAISIYA_ROOT.'/', '', $dest));
            } else {
                $io->writeln('  - '.str_replace(TAISIYA_ROOT.'/', '', $file->getPathname()).' is copied to <info>'.str_replace(TAISIYA_ROOT.'/', '', $dest).'</info>');
            }
        }
    }

    /**
     * @param TaisiyaStyle $io
     * @throws RuntimeException
     */
    final protected function mergeSettings(TaisiyaStyle $io): void

    {
        $io->isVerbose() && $io->writeln('Merge settings into one file');

        $uname = php_uname('n');
        $files = [];

        $finder = Finder::create()
            ->in(TAISIYA_ROOT)
            ->path('app/config')
            ->files()
            ->name('/\.settings\.default\.php$/');

        foreach ($finder as $file) {
            $files[] = $file->getPathname();
        }

        $finder = Finder::create()
            ->in(TAISIYA_ROOT)
            ->path('app/config')
            ->files()
            ->name('/\.settings\.local\.php$/');

        foreach ($finder as $file) {
            $files[] = $file->getPathname();
        }

        $finder = Finder::create()
            ->in(TAISIYA_ROOT)
            ->path('app/config')
            ->files()
            ->name('/\.settings\.'.preg_quote($uname, '/').'\.local\.php$/');

        foreach ($finder as $file) {
            $files[] = $file->getPathname();
        }

        $files[] = TAISIYA_ROOT.'/app/config/settings.default.php';
        if (file_exists(TAISIYA_ROOT.'/app/config/settings.local.php')) {
            $files[] = TAISIYA_ROOT.'/app/config/settings.local.php';
        }
        if (file_exists(TAISIYA_ROOT.'/app/config/settings.'.$uname.'.local.php')) {
            $files[] = TAISIYA_ROOT.'/app/config/settings.'.$uname.'.local.php';
        }

        $settings = [];
        foreach ($files as $file) {
            $settings = array_merge_recursive($settings, require $file);
        }

        if (!file_put_contents(TAISIYA_ROOT.'/app/config/settings.php', "<?php\n\nreturn ".var_export($settings, true).";\n")) {
            throw new RuntimeException('Couldn\'t write to the file '.TAISIYA_ROOT.'/app/config/settings.php');
        } else {
            $io->writeln('  - writed to <info>app/config/settings.php</info>');
        }
    }
}


1			<?php
2
3			namespace Taisiya\CoreBundle\Console\Command\Config;
4
5			use Symfony\Component\Console\Input\InputInterface;
6			use Symfony\Component\Console\Output\OutputInterface;
7			use Symfony\Component\Finder\Finder;
8			use Symfony\Component\Finder\SplFileInfo;
9			use Taisiya\CoreBundle\Console\Command\Command;
10			use Taisiya\CoreBundle\Console\Style\TaisiyaStyle;
11			use Taisiya\CoreBundle\Exception\RuntimeException;
12
13			final class RebuildSettingsCommand extends Command
14			{
15			/**
16			* {@inheritdoc}
17			*/
18			protected function configure()
19			{
20			$this->setName('config:rebuild-settings')
21			->setDescription('Rebuild project settings');
22			}
23
24			/**
25			* @param InputInterface $input
26			* @param OutputInterface $output
27			*/
28			protected function execute(InputInterface $input, OutputInterface $output)
29			{
30			$io = new TaisiyaStyle($input, $output);
31
32			$this->copySettings($io);
33			$this->mergeSettings($io);
34			}
35
36			/**
37			* @param TaisiyaStyle $io
38			* @throws RuntimeException
39			*/
40			final protected function copySettings(TaisiyaStyle $io): void
			0 ignored issues – show Coding Style introduced 2016-10-14 08:56 UTC by Report Bug Copy Issue Report Unnecessary FINAL modifier in FINAL class Loading history...
41			{
42			$io->isVerbose() && $io->writeln('Copy bundles settings to project');
43
44			$finder = Finder::create()
45			->in(TAISIYA_ROOT)
46			->path('vendor')
47			->files()
48			->name('settings.default.php')
49			->filter(function (SplFileInfo $file) {
50			return (bool) preg_match('/\/app\/config\//', $file->getPathname());
51			});
52
53			foreach ($finder as $file) {exit(var_dump($file->getPathname()));
			0 ignored issues – show Security Debugging Code introduced 2016-10-14 08:56 UTC by Report Bug Copy Issue Report `var_dump($file->getPathname());` looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data. Loading history...
54			$dest = preg_replace('/^.+?\/vendor\/(.+?)\/(.+?)\/.+?$/', TAISIYA_ROOT.'/app/config/\1.\2.settings.default.php', $file->getPathname());
			0 ignored issues – show Unused Code introduced 2016-10-14 08:56 UTC by Report Bug Copy Issue Report `$dest = preg_replace('/^... $file->getPathname());` does not seem to be reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history...
55			if (!copy($file->getPathname(), $dest)) {
56			throw new RuntimeException('Couldn\'t copy '.str_replace(TAISIYA_ROOT.'/', '', $file->getPathname()).' to '.str_replace(TAISIYA_ROOT.'/', '', $dest));
57			} else {
58			$io->writeln(' - '.str_replace(TAISIYA_ROOT.'/', '', $file->getPathname()).' is copied to <info>'.str_replace(TAISIYA_ROOT.'/', '', $dest).'</info>');
59			}
60			}
61			}
62
63			/**
64			* @param TaisiyaStyle $io
65			* @throws RuntimeException
66			*/
67			final protected function mergeSettings(TaisiyaStyle $io): void
			0 ignored issues – show Coding Style introduced 2016-10-14 08:56 UTC by Report Bug Copy Issue Report Unnecessary FINAL modifier in FINAL class Loading history...
68			{
69			$io->isVerbose() && $io->writeln('Merge settings into one file');
70
71			$uname = php_uname('n');
72			$files = [];
73
74			$finder = Finder::create()
75			->in(TAISIYA_ROOT)
76			->path('app/config')
77			->files()
78			->name('/\.settings\.default\.php$/');
79
80			foreach ($finder as $file) {
81			$files[] = $file->getPathname();
82			}
83
84			$finder = Finder::create()
85			->in(TAISIYA_ROOT)
86			->path('app/config')
87			->files()
88			->name('/\.settings\.local\.php$/');
89
90			foreach ($finder as $file) {
91			$files[] = $file->getPathname();
92			}
93
94			$finder = Finder::create()
95			->in(TAISIYA_ROOT)
96			->path('app/config')
97			->files()
98			->name('/\.settings\.'.preg_quote($uname, '/').'\.local\.php$/');
99
100			foreach ($finder as $file) {
101			$files[] = $file->getPathname();
102			}
103
104			$files[] = TAISIYA_ROOT.'/app/config/settings.default.php';
105			if (file_exists(TAISIYA_ROOT.'/app/config/settings.local.php')) {
106			$files[] = TAISIYA_ROOT.'/app/config/settings.local.php';
107			}
108			if (file_exists(TAISIYA_ROOT.'/app/config/settings.'.$uname.'.local.php')) {
109			$files[] = TAISIYA_ROOT.'/app/config/settings.'.$uname.'.local.php';
110			}
111
112			$settings = [];
113			foreach ($files as $file) {
114			$settings = array_merge_recursive($settings, require $file);
115			}
116
117			if (!file_put_contents(TAISIYA_ROOT.'/app/config/settings.php', "<?php\n\nreturn ".var_export($settings, true).";\n")) {
118			throw new RuntimeException('Couldn\'t write to the file '.TAISIYA_ROOT.'/app/config/settings.php');
119			} else {
120			$io->writeln(' - writed to <info>app/config/settings.php</info>');
121			}
122			}
123			}
124

mosiyash / taisiya-core-bundle

Push — master ( c4223d...e3af10 )

RebuildSettingsCommand::mergeSettings() C

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like