ClassCommand::fire() - Code Metrics - clarkeash/factory - Measure and Improve Code Quality continuously with Scrutinizer

ClassCommand::fire() A
last analyzed 2016-02-01 00:41 UTC

↳ Parent: ClassCommand

Complexity

Conditions	2
Paths	2

Size

Total Lines	18
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	18
rs	9.4285
cc	2
eloc	9
nc	2
nop	0

<?php

namespace Factory\Console\Make;

use Factory\Composer;
use Factory\Console\BaseCommand;
use Factory\Generators\ClassGenerator;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputOption;

class ClassCommand extends BaseCommand
{
    /**
     * @var \Factory\Generators\ClassGenerator
     */
    private $generator;
    /**
     * @var \Factory\Composer
     */
    private $composer;

    public function __construct(Composer $composer, ClassGenerator $generator)
    {
        parent::__construct();
        $this->generator = $generator;
        $this->composer = $composer;
    }
    public function configure()
    {
        $this->setName('make:class');
        $this->setDescription('Create a class.');

        $this->addArgument('name', InputArgument::REQUIRED, 'The name of the class to make.');
        $this->addOption('abstract', null, InputOption::VALUE_NONE, 'The class should be abstract.');
        $this->addOption('force', null, InputOption::VALUE_NONE, 'The class should overwrite an existing one.');
    }

    public function fire()
    {
        $data = [
            'name' => $this->argument('name'),
            'abstract' => $this->option('abstract'),
        ];

        $result = $this->generator->setData($data)->setForce($this->option('force'))->make();
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        if ($result)
        {
            $this->info('Class: ' . $this->composer->getClassPath($this->argument('name')) . ' created');
        }
        else
        {
            $this->error('Something went wrong!');
        }
    }
}


1			<?php
2
3			namespace Factory\Console\Make;
4
5			use Factory\Composer;
6			use Factory\Console\BaseCommand;
7			use Factory\Generators\ClassGenerator;
8			use Symfony\Component\Console\Input\InputArgument;
9			use Symfony\Component\Console\Input\InputOption;
10
11			class ClassCommand extends BaseCommand
12			{
13			/**
14			* @var \Factory\Generators\ClassGenerator
15			*/
16			private $generator;
17			/**
18			* @var \Factory\Composer
19			*/
20			private $composer;
21
22			public function __construct(Composer $composer, ClassGenerator $generator)
23			{
24			parent::__construct();
25			$this->generator = $generator;
26			$this->composer = $composer;
27			}
28			public function configure()
29			{
30			$this->setName('make:class');
31			$this->setDescription('Create a class.');
32
33			$this->addArgument('name', InputArgument::REQUIRED, 'The name of the class to make.');
34			$this->addOption('abstract', null, InputOption::VALUE_NONE, 'The class should be abstract.');
35			$this->addOption('force', null, InputOption::VALUE_NONE, 'The class should overwrite an existing one.');
36			}
37
38			public function fire()
39			{
40			$data = [
41			'name' => $this->argument('name'),
42			'abstract' => $this->option('abstract'),
43			];
44
45			$result = $this->generator->setData($data)->setForce($this->option('force'))->make();
			0 ignored issues – show Documentation introduced 2016-01-28 23:37 UTC by Report Bug Copy Issue Report `$this->option('force')` is of type `string\|array`, but the function expects a `boolean`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
46
47			if ($result)
48			{
49			$this->info('Class: ' . $this->composer->getClassPath($this->argument('name')) . ' created');
50			}
51			else
52			{
53			$this->error('Something went wrong!');
54			}
55			}
56			}
57

clarkeash / factory

ClassCommand::fire() A last analyzed 2016-02-01 00:41 UTC

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like

ClassCommand::fire() A
last analyzed 2016-02-01 00:41 UTC