SelfUpdateCommand::configure() - Code Metrics - Inspection of "Allow SymfonyHttpClient config in file (#177)" - gordalina/cachetool - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 816b5a...9f44d9 )

by Samuel

created 2021-05-11 18:42 UTC

SelfUpdateCommand::configure() A

↳ Parent: SelfUpdateCommand

Complexity

Conditions	1
Paths	1

Size

Total Lines	11
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	1
eloc	9
c	1
b	0
f	1
nc	1
nop	0
dl	0
loc	11
ccs	6
cts	6
cp	1
crap	1
rs	9.9666

<?php

/*
 * This file is part of CacheTool.
 *
 * (c) Samuel Gordalina <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace CacheTool\Command;

use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

class SelfUpdateCommand extends \SelfUpdate\SelfUpdateCommand
{
    protected function configure()
    {
        $app = $this->applicationName;

        $this
            ->setAliases(array('self-update'))
            ->setDescription("Updates $app to the latest version.")
            ->setHelp(
                <<<EOT
  The <info>self-update</info> command checks github for newer
  versions of $app and if found, installs the latest.
  EOT
            );
    }

    protected function execute(InputInterface $input, OutputInterface $output) {
        $result = parent::execute($input, $output);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();

        if (is_int($result)) {

           return $result;
        }

        return 0;
    }
}


1		<?php
2
3		/*
4		* This file is part of CacheTool.
5		*
6		* (c) Samuel Gordalina <[email protected]>
7		*
8		* For the full copyright and license information, please view the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		namespace CacheTool\Command;
13
14		use Symfony\Component\Console\Input\InputInterface;
15		use Symfony\Component\Console\Output\OutputInterface;
16
17		class SelfUpdateCommand extends \SelfUpdate\SelfUpdateCommand
18		{
19	25	protected function configure()
20		{
21	25	$app = $this->applicationName;
22
23		$this
24	25	->setAliases(array('self-update'))
25	25	->setDescription("Updates $app to the latest version.")
26	25	->setHelp(
27		<<<EOT
28		The <info>self-update</info> command checks github for newer
29	25	versions of $app and if found, installs the latest.
30		EOT
31		);
32	25	}
33
34		protected function execute(InputInterface $input, OutputInterface $output) {
35		$result = parent::execute($input, $output);
		0 ignored issues – show Bug introduced 2021-03-18 05:19 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$result` is correct as `parent::execute($input, $output)` targeting `SelfUpdate\SelfUpdateCommand::execute()` seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
36
37		if (is_int($result)) {
		0 ignored issues – show introduced 2021-03-18 05:19 UTC by Report Bug Copy Issue Report The condition `is_int($result)` is always `false`. Loading history...
38		return $result;
39		}
40
41		return 0;
42		}
43		}
44

gordalina / cachetool

Push — master ( 816b5a...9f44d9 )

SelfUpdateCommand::configure() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like