XmlRpcService::parseMessage() - Code Metrics - Inspection of "XmlRpcPayloadParser" - php-api-clients/xml-rpc-service - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 7f68f8...aed5c0 )

by Cees-Jan

created 2018-01-21 20:39 UTC

XmlRpcService::parseMessage() C

↳ Parent: Project

Complexity

Conditions	7
Paths	7

Size

Total Lines	28
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	28
rs	6.7272
c	0
b	0
f	0
cc	7
eloc	14
nc	7
nop	1

<?php declare(strict_types=1);

namespace ApiClients\Tools\Services\XmlRpc;

use ApiClients\Foundation\Transport\Service\RequestService;
use ApiClients\Middleware\Xml\XmlStream;
use Psr\Http\Message\ResponseInterface;
use RingCentral\Psr7\Request;
use function React\Promise\reject;
use function React\Promise\resolve;

class XmlRpcService
{
    /**
     * @var RequestService
     */
    private $requestService;

    /**
     * @param RequestService $requestService
     */
    public function __construct(RequestService $requestService)
    {
        $this->requestService = $requestService;
    }

    public function call(string $method, array $arguments = [])
    {
        $params = [];
        foreach ($arguments as $argument) {
            $params[] = [
                'param' => [
                    'value' => [
                        gettype($argument) => $argument,
                    ],
                ],
            ];
        }

        return $this->callRaw($method, $params)->then(function (array $xml) {
            return XmlRpcPayloadParser::parse($xml);
        });
    }

    public function callRaw(string $method, array $arguments = [])
    {
        $xml = [
            'methodCall' => [
                'methodName' => $method,
            ],
        ];

        if (count($arguments) > 0) {
            $xml['methodCall']['params'] = $arguments;
        }

        return $this->requestService->request(new Request(
            'POST',
            '',
            [],
            new XmlStream($xml)
        ))->then(function (ResponseInterface $response) {
            $xml = $response->getBody()->getParsedContents();
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.
    }
}

            if (isset($xml['methodResponse']['fault'])) {
                return reject(
                    new XmlRpcError(
                        $xml['methodResponse']['fault']['value']['struct']['member'][1]['value']['string'],
                        (int)$xml['methodResponse']['fault']['value']['struct']['member'][0]['value']['int']
                    )
                );
            }

            return resolve($xml['methodResponse']['params']['param']['value']);
        });
    }
}


GitHub Access Token became invalid

Push — master ( 7f68f8...aed5c0 )

XmlRpcService::parseMessage() C

Complexity

Size

Duplication

Importance

Available Fixes

1			<?php declare(strict_types=1);
2
3			namespace ApiClients\Tools\Services\XmlRpc;
4
5			use ApiClients\Foundation\Transport\Service\RequestService;
6			use ApiClients\Middleware\Xml\XmlStream;
7			use Psr\Http\Message\ResponseInterface;
8			use RingCentral\Psr7\Request;
9			use function React\Promise\reject;
10			use function React\Promise\resolve;
11
12			class XmlRpcService
13			{
14			/**
15			* @var RequestService
16			*/
17			private $requestService;
18
19			/**
20			* @param RequestService $requestService
21			*/
22			public function __construct(RequestService $requestService)
23			{
24			$this->requestService = $requestService;
25			}
26
27			public function call(string $method, array $arguments = [])
28			{
29			$params = [];
30			foreach ($arguments as $argument) {
31			$params[] = [
32			'param' => [
33			'value' => [
34			gettype($argument) => $argument,
35			],
36			],
37			];
38			}
39
40			return $this->callRaw($method, $params)->then(function (array $xml) {
41			return XmlRpcPayloadParser::parse($xml);
42			});
43			}
44
45			public function callRaw(string $method, array $arguments = [])
46			{
47			$xml = [
48			'methodCall' => [
49			'methodName' => $method,
50			],
51			];
52
53			if (count($arguments) > 0) {
54			$xml['methodCall']['params'] = $arguments;
55			}
56
57			return $this->requestService->request(new Request(
58			'POST',
59			'',
60			[],
61			new XmlStream($xml)
62			))->then(function (ResponseInterface $response) {
63			$xml = $response->getBody()->getParsedContents();
			0 ignored issues – show Bug introduced 2018-01-20 19:25 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Psr\Http\Message\StreamInterface` as the method `getParsedContents()` does only exist in the following implementations of said interface: `ApiClients\Middleware\Xml\XmlStream`. 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...
64
65			if (isset($xml['methodResponse']['fault'])) {
66			return reject(
67			new XmlRpcError(
68			$xml['methodResponse']['fault']['value']['struct']['member'][1]['value']['string'],
69			(int)$xml['methodResponse']['fault']['value']['struct']['member'][0]['value']['int']
70			)
71			);
72			}
73
74			return resolve($xml['methodResponse']['params']['param']['value']);
75			});
76			}
77			}
78

php-api-clients / xml-rpc-service

GitHub Access Token became invalid

Push — master ( 7f68f8...aed5c0 )

XmlRpcService::parseMessage() C

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like