Clients\RpcLiteral\ObjectCommand

Complexity

Total Complexity

2

Size/Duplication

Total Lines	57
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	5
Bugs	1	Features	2

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	4	4	1
A	execute()	49	49	1

<?php
namespace Clients\RpcLiteral;

use Clients\InitCommand;
use SoapClient;
use stdClass;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

class ObjectCommand extends InitCommand

This class seems to be duplicated in your project.

{
    protected function configure()
    {
        $this->setName('rpc_literal:object');
    }

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

        $this->soapClient = new SoapClient('http://localhost/wsdl-creator/examples/rpc_literal/ObjectExampleSoapServer.php?wsdl', array(
            'uri' => "http://foo.bar/", 'location' => 'http://localhost/wsdl-creator/examples/rpc_literal/ObjectExampleSoapServer.php',
            'trace' => true, 'cache_wsdl' => WSDL_CACHE_NONE
        ));

        $this->serviceInfo('Client Object - rpc/literal');

        $this->renderMethodsTable();

        $user = new stdClass();
        $user->name = 'john';
        $user->age = 32;
        $response = $this->soapClient->userInfo($user);
        $this->method('userInfo', array($user), $response);

        $response = $this->soapClient->getAgentWithId('peter', 999444);
        $this->method('getAgentWithId', array('peter', 999444), $response);

        $namesInfo = new stdClass();
        $namesInfo->names = array('billy', 'clark');
        $namesInfo->id = 333;
        $response = $this->soapClient->namesForId($namesInfo);
        $this->method('namesForId', array($namesInfo), $response);

        $response = $this->soapClient->getCompanies();
        $this->method('getCompanies', array(), $response);

        $response = $this->soapClient->getListOfAgentsWithId();
        $this->method('getListOfAgentsWithId', array(), $response);

        $payments[0] = new stdClass();

$payments was never initialized. Although not strictly required by PHP, it is generally a good practice to add $payments = array(); before regardless.

        $payments[0]->payment = array(1.21, 3.21, 100.60);
        $payments[0]->user = 'john';
        $payments[1] = new stdClass();
        $payments[1]->payment = array(120.60);
        $payments[1]->user = 'peter';
        $response = $this->soapClient->setPayment($payments);
        $this->method('setPayment', array($payments), $response);

        $response = $this->soapClient->getAgentsWithPayment();
        $this->method('getAgentsWithPayment', array(), $response);

        $response = $this->soapClient->getEmployeesWithAgents();
        $this->method('getEmployeesWithAgents', array(), $response);
    }
}