SelfUpdateCommand - Code Metrics - Inspection of "Update dependencies" - gordalina/cachetool - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( f015ee...816b5a )

by Samuel

created 2021-05-11 02:37 UTC

SelfUpdateCommand A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Test Coverage

Coverage

61.53%

Importance

Changes	2
Bugs	0	Features	1

Metric	Value
eloc	14
c	2
b	0
f	1
dl	0
loc	25
ccs	8
cts	13
cp	0.6153
rs	10
wmc	3

2 Methods

Rating	Name	Duplication	Size	Complexity
A	execute()	0	8	2
A	configure()	0	11	1

<?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	24	protected function configure()
20		{
21	24	$app = $this->applicationName;
22
23		$this
24	24	->setAliases(array('self-update'))
25	24	->setDescription("Updates $app to the latest version.")
26	24	->setHelp(
27		<<<EOT
28	24	The <info>self-update</info> command checks github for newer
29	24	versions of $app and if found, installs the latest.
30		EOT
31		);
32	24	}
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 ( f015ee...816b5a )

SelfUpdateCommand A

Complexity

Size/Duplication

Test Coverage

Importance

2 Methods

Duplication Side-by-Side

Filter issues like