ThemeSetupCommand::execute() - Code Metrics - Inspection of "Merge pull request #52 from takeit/split" - superdesk/web-publisher - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 530ce9...1b2ae5 )

by Paweł

created 2016-02-16 10:14 UTC

ThemeSetupCommand::execute() B

↳ Parent: ThemeSetupCommand

Complexity

Conditions	8
Paths	38

Size

Total Lines	58
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	58
rs	7.1977
cc	8
eloc	36
nc	38
nop	2

How to fix Long Method

<?php

/**
 * This file is part of the Superdesk Web Publisher Fixtures Bundle.
 *
 * Copyright 2015 Sourcefabric z.u. and contributors.
 *
 * For the full copyright and license information, please see the
 * AUTHORS and LICENSE files distributed with this source code.
 *
 * @copyright 2015 Sourcefabric z.ú.
 * @license http://www.superdesk.org/license
 */
namespace SWP\Bundle\FixturesBundle\Command;

use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Filesystem\Filesystem;
use Symfony\Component\Console\Question\ConfirmationQuestion;

class ThemeSetupCommand extends ContainerAwareCommand
{
    const DEFAULT_THEME_NAME = 'theme_1';

    /**
     * {@inheritdoc}
     */
    protected function configure()
    {
        $this
            ->setName('theme:setup')
            ->setDescription('Sets (copies) demo theme for development purposes.')
            ->addArgument(
                'name',
                InputArgument::OPTIONAL,
                'Theme name',
                null
            )
            ->addOption(
                'delete',
                null,
                InputOption::VALUE_NONE,
                'If set, theme will be removed from the application.'
            )
            ->addOption(
                'force',
                'f',
                InputOption::VALUE_NONE,
                'If set, forces to execute an action without confirmation.'
            )
            ->setHelp(
<<<'EOT'
The <info>theme:setup</info> command copies theme to your application themes folder (app/Resources/themes):

  <info>./app/console theme:setup</info>

You can also optionally specify the delete (<info>--delete</info>) option to delete theme by name:

  <info>./app/console theme:setup <name> --delete</info>

To force an action, you need to add an option: <info>--force</info>:

  <info>./app/console theme:setup <name> --delete --force</info>

Demo theme can be found in "SWPFixturesBundle/Resources/themes".
EOT
            )
        ;
    }

    /**
     * {@inheritdoc}
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $fileSystem = new Filesystem();
        $kernel = $this->getContainer()->get('kernel');
        $name = $input->getArgument('name');
        $helper = $this->getHelper('question');
        $force = true === $input->getOption('force');

        if (!$name) {
            $name = $this->getActiveThemeName();
        }

        try {
            if ($input->getOption('delete')) {
                $question = new ConfirmationQuestion(
                    '<question>This will override your current theme: "'.$name.'", if exists. Continue with this action? (yes/no)<question> <comment>[yes]</comment> ',
                    true,
                    '/^(y|j)/i'
                );

                if (!$force) {
                    if (!$helper->ask($input, $output, $question)) {
                        return;
                    }
                }

                $fileSystem->remove($kernel->getRootDir().'/Resources/themes/'.$name);

                $output->writeln('<info>Theme "'.$name.'" has been deleted successfully!</info>');

                return true;
            }

            $question = new ConfirmationQuestion(
                '<question>This will override your current theme: "'.$name.'", if exists. Continue with this action? (yes/no)<question> <comment>[yes]</comment> ',
                true,
                '/^(y|j)/i'
            );

            if (!$force) {
                if (!$helper->ask($input, $output, $question)) {
                    return;
                }
            }

            $fileSystem->mirror(
                $kernel->locateResource('@SWPFixturesBundle/Resources/themes/'.self::DEFAULT_THEME_NAME),
                $kernel->getRootDir().'/Resources/themes/'.$name,
                null,
                ['override' => true, 'delete' => true]
            );

            $output->writeln('<info>Theme "'.$name.'" has been setup successfully!</info>');
        } catch (\Exception $e) {
            $output->writeln('<error>Theme "'.$name.'" could not be setup!</error>');
            $output->writeln('<error>Stacktrace: '.$e->getMessage().'</error>');
        }
    }

    private function getActiveThemeName()
    {
        return $this->getContainer()->getParameter('active_theme');
    }
}


1			<?php
2
3			/**
4			* This file is part of the Superdesk Web Publisher Fixtures Bundle.
5			*
6			* Copyright 2015 Sourcefabric z.u. and contributors.
7			*
8			* For the full copyright and license information, please see the
9			* AUTHORS and LICENSE files distributed with this source code.
10			*
11			* @copyright 2015 Sourcefabric z.ú.
12			* @license http://www.superdesk.org/license
13			*/
14			namespace SWP\Bundle\FixturesBundle\Command;
15
16			use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
17			use Symfony\Component\Console\Input\InputArgument;
18			use Symfony\Component\Console\Input\InputInterface;
19			use Symfony\Component\Console\Output\OutputInterface;
20			use Symfony\Component\Console\Input\InputOption;
21			use Symfony\Component\Filesystem\Filesystem;
22			use Symfony\Component\Console\Question\ConfirmationQuestion;
23
24			class ThemeSetupCommand extends ContainerAwareCommand
25			{
26			const DEFAULT_THEME_NAME = 'theme_1';
27
28			/**
29			* {@inheritdoc}
30			*/
31			protected function configure()
32			{
33			$this
34			->setName('theme:setup')
35			->setDescription('Sets (copies) demo theme for development purposes.')
36			->addArgument(
37			'name',
38			InputArgument::OPTIONAL,
39			'Theme name',
40			null
41			)
42			->addOption(
43			'delete',
44			null,
45			InputOption::VALUE_NONE,
46			'If set, theme will be removed from the application.'
47			)
48			->addOption(
49			'force',
50			'f',
51			InputOption::VALUE_NONE,
52			'If set, forces to execute an action without confirmation.'
53			)
54			->setHelp(
55			<<<'EOT'
56			The <info>theme:setup</info> command copies theme to your application themes folder (app/Resources/themes):
57
58			<info>./app/console theme:setup</info>
59
60			You can also optionally specify the delete (<info>--delete</info>) option to delete theme by name:
61
62			<info>./app/console theme:setup <name> --delete</info>
63
64			To force an action, you need to add an option: <info>--force</info>:
65
66			<info>./app/console theme:setup <name> --delete --force</info>
67
68			Demo theme can be found in "SWPFixturesBundle/Resources/themes".
69			EOT
70			)
71			;
72			}
73
74			/**
75			* {@inheritdoc}
76			*/
77			protected function execute(InputInterface $input, OutputInterface $output)
78			{
79			$fileSystem = new Filesystem();
80			$kernel = $this->getContainer()->get('kernel');
81			$name = $input->getArgument('name');
82			$helper = $this->getHelper('question');
83			$force = true === $input->getOption('force');
84
85			if (!$name) {
86			$name = $this->getActiveThemeName();
87			}
88
89			try {
90			if ($input->getOption('delete')) {
91			$question = new ConfirmationQuestion(
92			'<question>This will override your current theme: "'.$name.'", if exists. Continue with this action? (yes/no)<question> <comment>[yes]</comment> ',
93			true,
94			'/^(y\|j)/i'
95			);
96
97			if (!$force) {
98			if (!$helper->ask($input, $output, $question)) {
99			return;
100			}
101			}
102
103			$fileSystem->remove($kernel->getRootDir().'/Resources/themes/'.$name);
104
105			$output->writeln('<info>Theme "'.$name.'" has been deleted successfully!</info>');
106
107			return true;
108			}
109
110			$question = new ConfirmationQuestion(
111			'<question>This will override your current theme: "'.$name.'", if exists. Continue with this action? (yes/no)<question> <comment>[yes]</comment> ',
112			true,
113			'/^(y\|j)/i'
114			);
115
116			if (!$force) {
117			if (!$helper->ask($input, $output, $question)) {
118			return;
119			}
120			}
121
122			$fileSystem->mirror(
123			$kernel->locateResource('@SWPFixturesBundle/Resources/themes/'.self::DEFAULT_THEME_NAME),
124			$kernel->getRootDir().'/Resources/themes/'.$name,
125			null,
126			['override' => true, 'delete' => true]
127			);
128
129			$output->writeln('<info>Theme "'.$name.'" has been setup successfully!</info>');
130			} catch (\Exception $e) {
131			$output->writeln('<error>Theme "'.$name.'" could not be setup!</error>');
132			$output->writeln('<error>Stacktrace: '.$e->getMessage().'</error>');
133			}
134			}
135
136			private function getActiveThemeName()
137			{
138			return $this->getContainer()->getParameter('active_theme');
139			}
140			}
141

superdesk / web-publisher

Push — master ( 530ce9...1b2ae5 )

ThemeSetupCommand::execute() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like