PartFire\MangoPayBundle\Command\HookListCommand

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	8	8	1
A	execute()	8	8	1
A	showTitle()	4	4	1
A	getConsoleOutPutter()	4	4	1
A	getHookService()	4	4	1

<?php

/**
 * Created by Graham Owens (gra@partfire.co.uk)
 * Company: PartFire Ltd (www.partfire.co.uk)
 * Console: Discovery
 *
 * User:    gra
 * Date:    13/01/17
 * Time:    16:23
 * Project: PartFire MangoPay Bundle
 * File:    HookListCommand.php
 *
 **/


namespace PartFire\MangoPayBundle\Command;

use Fruitful\IdentityCheckBundle\Entity\IdentityCheck;
use Fruitful\IdentityCheckBundle\Entity\Repository\IdentityCheckFactoryRepository;
use Fruitful\IdentityCheckBundle\Event\IdentityCheckUpdateEvent;
use PartFire\CommonBundle\Services\Output\Cli\ConsoleOutput;
use PartFire\MangoPayBundle\Services\Hook;
use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputDefinition;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\EventDispatcher\EventDispatcher;
use Symfony\Component\EventDispatcher\EventDispatcherInterface;

class HookListCommand extends ContainerAwareCommand

This class seems to be duplicated in your project.

{

    private $output;

    protected function configure()
    {
        $this
            ->setName('partfire:mangopay:hook-list')
            ->setDescription('Shows all hooks registered in Mango')
            ;

    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $this->output = $this->getConsoleOutPutter();
        $this->output->setOutputer($output);

        $this->output->info("Listing all hooks with MangoPay");
        var_dump($this->getHookService()->list());

var_dump($this->getHookService()->list()); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

    }

    private function showTitle($title)
    {
        $this->output->info(str_pad(" " . $title . " ", 80, "-", STR_PAD_BOTH));
    }

    private function getConsoleOutPutter() : ConsoleOutput
    {
        return $this->getContainer()->get('partfire_common.output_console');
    }

    private function getHookService() : Hook
    {
        return $this->getContainer()->get('part_fire_mango_pay.services.hook');
    }

}