Manager - Code Metrics - Inspection of "Fix date format" - gpupo/netshoes-sdk - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 2ae3d8...1a6d7e )

by Gilmar

created 2017-03-22 17:09 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

82%

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
ccs	41
cts	50
cp	0.82
rs	10

7 Methods

Rating	Name	Size	Complexity
A	setUp()	4	1
A	factoryDecorator()	8	1
A	resolvePrevious()	11	3
A	normalizeShipping()	9	2
B	update()	46	5
A	factoryTranslator()	6	1
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 DateInterval;
use DateTime;
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 DateInterval;
18		use DateTime;
19		use Gpupo\CommonSdk\Entity\EntityInterface;
20		use Gpupo\CommonSdk\Response;
21		use Gpupo\CommonSdk\Traits\LoadTrait;
22		use Gpupo\CommonSdk\Traits\TranslatorManagerTrait;
23		use Gpupo\NetshoesSdk\Entity\AbstractManager;
24		use Gpupo\NetshoesSdk\Traits\DateTimeTrait;
25
26		final class Manager extends AbstractManager
27		{
28		use TranslatorManagerTrait;
29		use LoadTrait;
30		use DateTimeTrait;
31
32		protected $entity = 'Order';
33
34		/**
35		* @codeCoverageIgnore
36		*/
37		protected function setUp()
38		{
39		$this->maps = $this->loadArrayFromFile(__DIR__.'/map/restful.map.php');
40		}
41
42	1	public function factoryDecorator(Order $order, $decoratorName)
43		{
44	1	$className = __NAMESPACE__.'\\Decorator\\'.$decoratorName;
45	1	$instance = new $className();
46	1	$instance->setOrder($order);
47
48	1	return $instance;
49		}
50
51	1	protected function resolvePrevious(Order $entity)
52		{
53		try {
54	1	$o = $this->findById($entity->getId());
55	1	if ($o instanceof Order) {
56	1	return $o;
57		}
58		} catch (\Exception $o) {
59		throw new \Exception('Order #'.$entity->getId().' not found on marketplace!');
60		}
61		}
62
63	1	protected function normalizeShipping(Order $entity, Order $existent)
64		{
65	1	if (1000 > intval($entity->getShipping()->getShippingCode())) {
66	1	$code = $existent->getShipping()->getShippingCode();
67	1	$entity->getShipping()->setShippingCode($code);
68		}
69
70	1	return $entity;
71		}
72
73		/**
74		* {@inheritdoc}
75		*/
76	7	public function update(EntityInterface $entity, EntityInterface $existent = null)
77		{
78	7	parent::update($entity, $existent);
79
80	7	$factory204 = function ($message) {
81		return new Response([
82		'raw' => '{"message":"'.$message.'"}',
83		'httpStatusCode' => 204,
84		]);
85	7	};
86
87	7	if (empty($existent)) {
88	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...
89		}
90
91	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...
92		$this->log('info', 'Order sem atualização');
93
94		return $factory204('Order status not changed!');
95		}
96
97	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...
98		return $factory204('Order status not used!');
99		}
100
101	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...
102
103	7	if (in_array($entity->getOrderStatus(), ['approved', 'canceled',
104	7	'delivered', 'invoiced', 'shipped', ], true)) {
105	6	$decorator = $this->factoryDecorator($entity, 'Status\\'.ucfirst($entity->getOrderStatus()));
106	6	$json = $decorator->toJson();
107	6	$mapKey = 'to'.ucfirst($entity->getOrderStatus());
108	6	$shipping = $entity->getShipping();
109	6	$code = $shipping->getShippingCode();
110	6	$shipping->toJson();
111	5	$map = $this->factoryMap($mapKey, [
112	5	'orderNumber' => $entity->getOrderNumber(),
113	5	'itemId' => $entity->getOrderNumber(),
114	5	'shippingCode' => $code,
115		]);
116
117	5	return $this->execute($map, $json);
118		}
119
120	1	throw new \InvalidArgumentException('Order Status ['.$entity->getOrderStatus().'] não suportado', 1);
121		}
122
123		public function factoryTranslator(array $data = [])
124		{
125		$translator = new Translator($data);
126
127		return $translator;
128		}
129
130	1	public function fetchQueue($offset = 0, $limit = 50, array $parameters = [])
131		{
132	1	return $this->translatorFetch($offset, $limit, array_merge([
133	1	'orderStatus' => 'approved',
134	1	'orderStartDate' => $this->dateMove('P4D'),
135		], $parameters));
136		}
137		}
138

gpupo / netshoes-sdk

Push — master ( 2ae3d8...1a6d7e )

Manager A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

7 Methods

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like