ChildrenCommand::execute() - Code Metrics - Inspection of "Merge pull request #16 from silverleague/feature/l..." - silverleague/silverstripe-console - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 6e7d18...37a08d )

by Robbie

created 2017-01-03 05:18 UTC

ChildrenCommand::execute() A

↳ Parent: ChildrenCommand

Complexity

Conditions	2
Paths	2

Size

Total Lines	22
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	22
rs	9.2
cc	2
eloc	17
nc	2
nop	2

<?php

namespace SilverLeague\Console\Command\Object;

use SilverLeague\Console\Command\SilverStripeCommand;
use SilverStripe\Core\ClassInfo;
use Symfony\Component\Console\Helper\Table;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * List all child classes of a given class name
 *
 * @package silverstripe-console
 * @author  Robbie Averill <[email protected]>
 */
class ChildrenCommand extends SilverStripeCommand
{
    /**
     * {@inheritDoc}
     */
    protected function configure()
    {
        $this
            ->setName('object:children')
            ->setDescription('List all child classes of a given class, e.g. "Page"')
            ->addArgument('object', InputArgument::REQUIRED, 'The class to find children for');
    }

    /**
     * {@inheritDoc}
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $object = $input->getArgument('object');
        $classes = (array) ClassInfo::subclassesFor($object);
        // Remove the class itself
        array_shift($classes);
        if (!$classes) {

            $output->writeln('There are no child classes for ' . $object);
            return;
        }
        sort($classes);
        $rows = array_map(function ($class) {
            return [$class];
        }, $classes);

        $output->writeln('<info>Child classes for ' . $object . ':</info>');
        $table = new Table($output);
        $table
            ->setHeaders(['Class name'])
            ->setRows($rows)
            ->render();
    }
}


1			<?php
2
3			namespace SilverLeague\Console\Command\Object;
4
5			use SilverLeague\Console\Command\SilverStripeCommand;
6			use SilverStripe\Core\ClassInfo;
7			use Symfony\Component\Console\Helper\Table;
8			use Symfony\Component\Console\Input\InputArgument;
9			use Symfony\Component\Console\Input\InputInterface;
10			use Symfony\Component\Console\Output\OutputInterface;
11
12			/**
13			* List all child classes of a given class name
14			*
15			* @package silverstripe-console
16			* @author Robbie Averill <[email protected]>
17			*/
18			class ChildrenCommand extends SilverStripeCommand
19			{
20			/**
21			* {@inheritDoc}
22			*/
23			protected function configure()
24			{
25			$this
26			->setName('object:children')
27			->setDescription('List all child classes of a given class, e.g. "Page"')
28			->addArgument('object', InputArgument::REQUIRED, 'The class to find children for');
29			}
30
31			/**
32			* {@inheritDoc}
33			*/
34			protected function execute(InputInterface $input, OutputInterface $output)
35			{
36			$object = $input->getArgument('object');
37			$classes = (array) ClassInfo::subclassesFor($object);
38			// Remove the class itself
39			array_shift($classes);
40			if (!$classes) {
			0 ignored issues – show Bug Best Practice introduced 2017-01-03 05:00 UTC by Report Bug Copy Issue Report The expression `$classes` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
41			$output->writeln('There are no child classes for ' . $object);
42			return;
43			}
44			sort($classes);
45			$rows = array_map(function ($class) {
46			return [$class];
47			}, $classes);
48
49			$output->writeln('<info>Child classes for ' . $object . ':</info>');
50			$table = new Table($output);
51			$table
52			->setHeaders(['Class name'])
53			->setRows($rows)
54			->render();
55			}
56			}
57

silverleague / silverstripe-console

Push — master ( 6e7d18...37a08d )

ChildrenCommand::execute() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like