Manager - Code Metrics - Inspection of "url/email update" - gpupo/netshoes-sdk - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( 6821c9...7f3ed3 )

by Gilmar

created 2017-10-24 12:12 UTC

Manager A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	112
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	7

Test Coverage

Coverage

77.78%

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	14
c	1
b	0
f	0
lcom	2
cbo	7
dl	0
loc	112
rs	10
ccs	42
cts	54
cp	0.7778

7 Methods

Rating	Name	Size	Complexity
A	setUp()	4	1
A	factoryDecorator()	8	1
A	resolvePrevious()	11	3
A	normalizeShipping()	9	2
A	factoryTranslator()	6	1
B	update()	46	5
A	fetchQueue()	7	1

<?php

/*
 * This file is part of gpupo/netshoes-sdk
 * Created by Gilmar Pupo <[email protected]>
 * For the information of copyright and license you should read the file
 * LICENSE which is distributed with this source code.
 * Para a informação dos direitos autorais e de licença você deve ler o arquivo
 * LICENSE que é distribuído com este código-fonte.
 * Para obtener la información de los derechos de autor y la licencia debe leer
 * el archivo LICENSE que se distribuye con el código fuente.
 * For more information, see <https://www.gpupo.com/>.
 */

namespace Gpupo\NetshoesSdk\Entity\Order;

use Gpupo\CommonSdk\Entity\EntityInterface;
use Gpupo\CommonSdk\Response;
use Gpupo\CommonSdk\Traits\LoadTrait;
use Gpupo\CommonSdk\Traits\TranslatorManagerTrait;
use Gpupo\NetshoesSdk\Entity\AbstractManager;
use Gpupo\NetshoesSdk\Traits\DateTimeTrait;

final class Manager extends AbstractManager
{
    use TranslatorManagerTrait;
    use LoadTrait;
    use DateTimeTrait;

    protected $entity = 'Order';

    /**
     * @codeCoverageIgnore
     */
    protected function setUp()
    {
        $this->maps = $this->loadArrayFromFile(__DIR__.'/map/restful.map.php');
    }

    public function factoryDecorator(Order $order, $decoratorName)
    {
        $className = __NAMESPACE__.'\\Decorator\\'.$decoratorName;
        $instance = new $className();
        $instance->setOrder($order);

        return $instance;
    }

    protected function resolvePrevious(Order $entity)
    {
        try {
            $o = $this->findById($entity->getId());
            if ($o instanceof Order) {
                return $o;
            }
        } catch (\Exception $o) {
            throw new \Exception('Order #'.$entity->getId().' not found on marketplace!');
        }
    }

    protected function normalizeShipping(Order $entity, Order $existent)
    {
        if (1000 > intval($entity->getShipping()->getShippingCode())) {
            $code = $existent->getShipping()->getShippingCode();
            $entity->getShipping()->setShippingCode($code);
        }

        return $entity;
    }

    /**
     * {@inheritdoc}
     */
    public function update(EntityInterface $entity, EntityInterface $existent = null)
    {
        parent::update($entity, $existent);

        $factory204 = function ($message) {
            return new Response([
                'raw'            => '{"message":"'.$message.'"}',
                'httpStatusCode' => 204,
            ]);
        };

        if (empty($existent)) {
            $existent = $this->resolvePrevious($entity);

        }

        if ($entity->getOrderStatus() === $existent->getOrderStatus()) {
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
            $this->log('info', 'Order sem atualização');

            return $factory204('Order status not changed!');
        }

        if ('processing' === $entity->getOrderStatus()) {
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
            return $factory204('Order status not used!');
        }

        $entity = $this->normalizeShipping($entity, $existent);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        if (in_array($entity->getOrderStatus(), ['approved', 'canceled',
            'delivered', 'invoiced', 'shipped', ], true)) {
            $decorator = $this->factoryDecorator($entity, 'Status\\'.ucfirst($entity->getOrderStatus()));
            $json = $decorator->toJson();
            $mapKey = 'to'.ucfirst($entity->getOrderStatus());
            $shipping = $entity->getShipping();
            $code = $shipping->getShippingCode();
            $shipping->toJson();
            $map = $this->factoryMap($mapKey, [
                'orderNumber'  => $entity->getOrderNumber(),
                'itemId'       => $entity->getOrderNumber(),
                'shippingCode' => $code,
            ]);

            return $this->execute($map, $json);
        }

        throw new \InvalidArgumentException('Order Status ['.$entity->getOrderStatus().'] não suportado', 1);
    }

    public function factoryTranslator(array $data = [])
    {
        $translator = new Translator($data);

        return $translator;
    }

    public function fetchQueue($offset = 0, $limit = 50, array $parameters = [])
    {
        return $this->translatorFetch($offset, $limit, array_merge([
            'orderStatus'    => 'approved',
            'orderStartDate' => $this->dateMove('P4D'),
        ], $parameters));
    }
}


1		<?php
2
3		/*
4		* This file is part of gpupo/netshoes-sdk
5		* Created by Gilmar Pupo <[email protected]>
6		* For the information of copyright and license you should read the file
7		* LICENSE which is distributed with this source code.
8		* Para a informação dos direitos autorais e de licença você deve ler o arquivo
9		* LICENSE que é distribuído com este código-fonte.
10		* Para obtener la información de los derechos de autor y la licencia debe leer
11		* el archivo LICENSE que se distribuye con el código fuente.
12		* For more information, see <https://www.gpupo.com/>.
13		*/
14
15		namespace Gpupo\NetshoesSdk\Entity\Order;
16
17		use Gpupo\CommonSdk\Entity\EntityInterface;
18		use Gpupo\CommonSdk\Response;
19		use Gpupo\CommonSdk\Traits\LoadTrait;
20		use Gpupo\CommonSdk\Traits\TranslatorManagerTrait;
21		use Gpupo\NetshoesSdk\Entity\AbstractManager;
22		use Gpupo\NetshoesSdk\Traits\DateTimeTrait;
23
24		final class Manager extends AbstractManager
25		{
26		use TranslatorManagerTrait;
27		use LoadTrait;
28		use DateTimeTrait;
29
30		protected $entity = 'Order';
31
32		/**
33		* @codeCoverageIgnore
34		*/
35		protected function setUp()
36		{
37		$this->maps = $this->loadArrayFromFile(__DIR__.'/map/restful.map.php');
38		}
39
40	1	public function factoryDecorator(Order $order, $decoratorName)
41		{
42	1	$className = __NAMESPACE__.'\\Decorator\\'.$decoratorName;
43	1	$instance = new $className();
44	1	$instance->setOrder($order);
45
46	1	return $instance;
47		}
48
49	1	protected function resolvePrevious(Order $entity)
50		{
51		try {
52	1	$o = $this->findById($entity->getId());
53	1	if ($o instanceof Order) {
54	1	return $o;
55		}
56		} catch (\Exception $o) {
57		throw new \Exception('Order #'.$entity->getId().' not found on marketplace!');
58		}
59		}
60
61	1	protected function normalizeShipping(Order $entity, Order $existent)
62		{
63	1	if (1000 > intval($entity->getShipping()->getShippingCode())) {
64	1	$code = $existent->getShipping()->getShippingCode();
65	1	$entity->getShipping()->setShippingCode($code);
66		}
67
68	1	return $entity;
69		}
70
71		/**
72		* {@inheritdoc}
73		*/
74	7	public function update(EntityInterface $entity, EntityInterface $existent = null)
75		{
76	7	parent::update($entity, $existent);
77
78	7	$factory204 = function ($message) {
79		return new Response([
80		'raw' => '{"message":"'.$message.'"}',
81		'httpStatusCode' => 204,
82		]);
83	7	};
84
85	7	if (empty($existent)) {
86	7	$existent = $this->resolvePrevious($entity);
		0 ignored issues – show Compatibility introduced 2016-07-29 18:35 UTC by Report Bug Copy Issue Report `$entity` of type `object<Gpupo\CommonSdk\Entity\EntityInterface>` is not a sub-type of `object<Gpupo\NetshoesSdk\Entity\Order\Order>`. It seems like you assume a concrete implementation of the interface `Gpupo\CommonSdk\Entity\EntityInterface` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history...
87		}
88
89	7	if ($entity->getOrderStatus() === $existent->getOrderStatus()) {
		0 ignored issues – show Bug introduced 2016-07-27 15:12 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Gpupo\CommonSdk\Entity\EntityInterface` as the method `getOrderStatus()` does only exist in the following implementations of said interface: `Gpupo\NetshoesSdk\Entity\Order\Order`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
90		$this->log('info', 'Order sem atualização');
91
92		return $factory204('Order status not changed!');
93		}
94
95	7	if ('processing' === $entity->getOrderStatus()) {
		0 ignored issues – show Bug introduced 2016-07-27 19:55 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Gpupo\CommonSdk\Entity\EntityInterface` as the method `getOrderStatus()` does only exist in the following implementations of said interface: `Gpupo\NetshoesSdk\Entity\Order\Order`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
96		return $factory204('Order status not used!');
97		}
98
99	7	$entity = $this->normalizeShipping($entity, $existent);
		0 ignored issues – show Compatibility introduced 2016-07-29 18:35 UTC by Report Bug Copy Issue Report `$entity` of type `object<Gpupo\CommonSdk\Entity\EntityInterface>` is not a sub-type of `object<Gpupo\NetshoesSdk\Entity\Order\Order>`. It seems like you assume a concrete implementation of the interface `Gpupo\CommonSdk\Entity\EntityInterface` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history... Documentation introduced 2016-07-29 18:35 UTC by Report Bug Copy Issue Report `$existent` is of type `null\|object<Gpupo\Common...Entity\EntityInterface>`, but the function expects a `object<Gpupo\NetshoesSdk\Entity\Order\Order>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
100
101	7	if (in_array($entity->getOrderStatus(), ['approved', 'canceled',
102	7	'delivered', 'invoiced', 'shipped', ], true)) {
103	6	$decorator = $this->factoryDecorator($entity, 'Status\\'.ucfirst($entity->getOrderStatus()));
104	6	$json = $decorator->toJson();
105	6	$mapKey = 'to'.ucfirst($entity->getOrderStatus());
106	6	$shipping = $entity->getShipping();
107	6	$code = $shipping->getShippingCode();
108	6	$shipping->toJson();
109	5	$map = $this->factoryMap($mapKey, [
110	5	'orderNumber' => $entity->getOrderNumber(),
111	5	'itemId' => $entity->getOrderNumber(),
112	5	'shippingCode' => $code,
113		]);
114
115	5	return $this->execute($map, $json);
116		}
117
118	1	throw new \InvalidArgumentException('Order Status ['.$entity->getOrderStatus().'] não suportado', 1);
119		}
120
121		public function factoryTranslator(array $data = [])
122		{
123		$translator = new Translator($data);
124
125		return $translator;
126		}
127
128	1	public function fetchQueue($offset = 0, $limit = 50, array $parameters = [])
129		{
130	1	return $this->translatorFetch($offset, $limit, array_merge([
131	1	'orderStatus' => 'approved',
132	1	'orderStartDate' => $this->dateMove('P4D'),
133	1	], $parameters));
134		}
135		}
136

gpupo / netshoes-sdk

Push — master ( 6821c9...7f3ed3 )

Manager A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

7 Methods

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like