XmlRpcService::callRaw() - Code Metrics - Inspection of "Automagically map an array of arguments to their X..." - php-api-clients/xml-rpc-service - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0ce32f...262efd )

by Cees-Jan

created 2018-01-20 20:16 UTC

XmlRpcService::callRaw() B

↳ Parent: XmlRpcService

Complexity

Conditions	3
Paths	2

Size

Total Lines	32
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	32
rs	8.8571
c	0
b	0
f	0
cc	3
eloc	18
nc	2
nop	2

<?php

namespace ApiClients\Tools\Services\XmlRpc;

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

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);
    }

    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']);
        });
    }
}


GitHub Access Token became invalid

Push — master ( 0ce32f...262efd )

XmlRpcService::callRaw() B

Complexity

Size

Duplication

Importance

Available Fixes

1			<?php
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 function React\Promise\reject;
9			use function React\Promise\resolve;
10			use RingCentral\Psr7\Request;
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);
41			}
42
43			public function callRaw(string $method, array $arguments = [])
44			{
45			$xml = [
46			'methodCall' => [
47			'methodName' => $method,
48			],
49			];
50
51			if (count($arguments) > 0) {
52			$xml['methodCall']['params'] = $arguments;
53			}
54
55			return $this->requestService->request(new Request(
56			'POST',
57			'',
58			[],
59			new XmlStream($xml)
60			))->then(function (ResponseInterface $response) {
61			$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...
62
63			if (isset($xml['methodResponse']['fault'])) {
64			return reject(
65			new XmlRpcError(
66			$xml['methodResponse']['fault']['value']['struct']['member'][1]['value']['string'],
67			(int)$xml['methodResponse']['fault']['value']['struct']['member'][0]['value']['int']
68			)
69			);
70			}
71
72			return resolve($xml['methodResponse']['params']['param']);
73			});
74			}
75			}
76

php-api-clients / xml-rpc-service

GitHub Access Token became invalid

Push — master ( 0ce32f...262efd )

XmlRpcService::callRaw() B

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like