Client::__construct() - Code Metrics - Inspection of "多分証明書はこう" - w-takumi/yahoo-shopping-jp-php - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — develop ( fb50a1...a6afd0 )

by Hirofumi

created 2017-06-06 05:46 UTC

Client::__construct() A

↳ Parent: Project

Complexity

Conditions	2
Paths	2

Size

Total Lines	15
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	15
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	10
nc	2
nop	3

<?php

namespace Shippinno\YahooShoppingJp;

use GuzzleHttp\Client as HttpClient;
use GuzzleHttp\Exception\ConnectException as GuzzleConnectException;
use GuzzleHttp\Exception\ServerException as GuzzleServerException;
use GuzzleHttp\Exception\ClientException as GuzzleClientException;
use GuzzleHttp\RequestOptions;
use Psr\Http\Message\ResponseInterface;
use Shippinno\YahooShoppingJp\Api\AbstractApi;
use Shippinno\YahooShoppingJp\Exception\ClientException;
use Shippinno\YahooShoppingJp\Exception\ConnectException;
use Shippinno\YahooShoppingJp\Exception\ExpiredAccessTokenException;
use Shippinno\YahooShoppingJp\Exception\ServerException;
use Shippinno\YahooShoppingJp\Request\AbstractRequest;
use SoapBox\Formatter\Formatter;

class Client
{
//    /**
//     * @var string
//     */
//    const BASE_URL = 'https://circus.shopping.yahooapis.jp/ShoppingWebService/V1/';

    /**
     * @var string
     */
    const BASE_URL = 'https://test.circus.shopping.yahooapis.jp/ShoppingWebService/V1/';

    /**
     * @var HttpClient
     */
    private $httpClient;

    /**
     * @var AbstractApi
     */
    private $api;

    /**
     * @var string
     */
    private $accessToken;

    /**
     * @var string
     */
    private $cert;

    /**
     * @var
     */
    private $sslKey;

    /**
     * @param string $accessToken
     * @param string $cert
     * @param string $sslKey
     * @param HttpClient|null $httpClient
     */
    public function __construct(
        string $accessToken,
        string $cert = null,
        string $sslKey = null,
        HttpClient $httpClient = null
    ) {
        if (null === $httpClient) {
            $httpClient = new HttpClient([
                'base_uri' => self::BASE_URL,
            ]);
        }

        $this->accessToken = $accessToken;
        $this->cert = $cert;
        $this->sslKey = $sslKey;
        $this->httpClient = $httpClient;
    }

    /**
     * @param AbstractRequest $request
     * @return array|mixed
     * @throws ClientException
     * @throws ConnectException
     * @throws ExpiredAccessTokenException
     * @throws ServerException
     */
    public function execute(AbstractRequest $request): array
    {
        $this->setApi($request->api());

        $options = $this->setAuthorizationHeader($this->setRequestParams($request));

        $options = $this->setCertAndSslKey($options);
        
        try {
            $rawResponse = $this->request($options);
        } catch (GuzzleClientException $e) {
            $wwwAuthenticateHeaders = $e->getResponse()->getHeader('WWW-Authenticate');

            if ([] !== $wwwAuthenticateHeaders) {
                if (false !== strpos($wwwAuthenticateHeaders[0], 'error_description="expired token"')) {
                    throw new ExpiredAccessTokenException;
                }
            }

            throw new ClientException($e->getMessage(), $e->getCode(), $e);
        } catch (GuzzleServerException $e) {
            throw new ServerException($e->getMessage(), $e->getCode(), $e);
        } catch (GuzzleConnectException $e) {
            throw new ConnectException($e->getMessage(), $e->getCode(), $e);
        }

        return $this->api->distillResponse($this->decodeResponse($rawResponse));
//        $response = $request->response()->setData($distilledResponse);

    }

    /**
     * @param AbstractApi $api
     */
    private function setApi(AbstractApi $api)
    {
        $this->api = $api;
    }

    /**
     * @param array $options
     * @return array
     */
    private function setCertAndSslKey(array $options): array
    {
        if (null !== $this->cert && null !== $this->sslKey) {
            $options[RequestOptions::CERT] = $this->cert,

            $options[RequestOptions::SSL_KEY] = $this->sslKey;
        }

        return $options;
    }

    /**
     * @param array $options
     * @return array
     */
    private function setAuthorizationHeader(array $options): array
    {
        $options['headers'] = [
            'Authorization' => 'Bearer ' . $this->accessToken,
        ];

        return $options;
    }

    /**
     * @param AbstractRequest $request
     * @return array
     */
    private function setRequestParams(AbstractRequest $request): array
    {
        $options = [];
        if ($this->api->httpMethod()->equals(HttpMethod::GET())) {
            $options = $this->setRequestParamsForGetRequest($options, $request);
        } else {
            if ($this->api->httpMethod()->equals(HttpMethod::POST())) {
                $options = $this->setRequestParamsForPostRequest($options, $request);
            }
        }

        return $options;
    }

    /**
     * @param array $options
     * @param AbstractRequest $request
     * @return array
     */
    private function setRequestParamsForGetRequest(array $options, AbstractRequest $request): array
    {
        $options['query'] = $request->getParams();

        return $options;
    }

    /**
     * @param array $options
     * @param AbstractRequest $request
     * @return array
     */
    private function setRequestParamsForPostRequest(array $options, AbstractRequest $request): array
    {
        if ($this->api->expectsFormFields()) {
            $options['form_params'] = $request->getParams();
        } else {
            $options['body'] = $request->getParams();
        }

        return $options;
    }


    /**
     * @param $options
     * @return mixed|ResponseInterface
     */
    private function request($options)
    {
        return $this->httpClient->request(
            $this->api->httpMethod()->getValue(),
            $this->api->path(),
            $options
        );
    }

    /**
     * @param ResponseInterface $rawResponse
     * @return array
     */
    private function decodeResponse(ResponseInterface $rawResponse): array
    {
        $formatter = Formatter::make($rawResponse->getBody()->getContents(), Formatter::XML);
        return $formatter->toArray();
    }
}


1			<?php
2
3			namespace Shippinno\YahooShoppingJp;
4
5			use GuzzleHttp\Client as HttpClient;
6			use GuzzleHttp\Exception\ConnectException as GuzzleConnectException;
7			use GuzzleHttp\Exception\ServerException as GuzzleServerException;
8			use GuzzleHttp\Exception\ClientException as GuzzleClientException;
9			use GuzzleHttp\RequestOptions;
10			use Psr\Http\Message\ResponseInterface;
11			use Shippinno\YahooShoppingJp\Api\AbstractApi;
12			use Shippinno\YahooShoppingJp\Exception\ClientException;
13			use Shippinno\YahooShoppingJp\Exception\ConnectException;
14			use Shippinno\YahooShoppingJp\Exception\ExpiredAccessTokenException;
15			use Shippinno\YahooShoppingJp\Exception\ServerException;
16			use Shippinno\YahooShoppingJp\Request\AbstractRequest;
17			use SoapBox\Formatter\Formatter;
18
19			class Client
20			{
21			// /**
22			// * @var string
23			// */
24			// const BASE_URL = 'https://circus.shopping.yahooapis.jp/ShoppingWebService/V1/';
25
26			/**
27			* @var string
28			*/
29			const BASE_URL = 'https://test.circus.shopping.yahooapis.jp/ShoppingWebService/V1/';
30
31			/**
32			* @var HttpClient
33			*/
34			private $httpClient;
35
36			/**
37			* @var AbstractApi
38			*/
39			private $api;
40
41			/**
42			* @var string
43			*/
44			private $accessToken;
45
46			/**
47			* @var string
48			*/
49			private $cert;
50
51			/**
52			* @var
53			*/
54			private $sslKey;
55
56			/**
57			* @param string $accessToken
58			* @param string $cert
59			* @param string $sslKey
60			* @param HttpClient\|null $httpClient
61			*/
62			public function __construct(
63			string $accessToken,
64			string $cert = null,
65			string $sslKey = null,
66			HttpClient $httpClient = null
67			) {
68			if (null === $httpClient) {
69			$httpClient = new HttpClient([
70			'base_uri' => self::BASE_URL,
71			]);
72			}
73
74			$this->accessToken = $accessToken;
75			$this->cert = $cert;
76			$this->sslKey = $sslKey;
77			$this->httpClient = $httpClient;
78			}
79
80			/**
81			* @param AbstractRequest $request
82			* @return array\|mixed
83			* @throws ClientException
84			* @throws ConnectException
85			* @throws ExpiredAccessTokenException
86			* @throws ServerException
87			*/
88			public function execute(AbstractRequest $request): array
89			{
90			$this->setApi($request->api());
91
92			$options = $this->setAuthorizationHeader($this->setRequestParams($request));
93
94			$options = $this->setCertAndSslKey($options);
95
96			try {
97			$rawResponse = $this->request($options);
98			} catch (GuzzleClientException $e) {
99			$wwwAuthenticateHeaders = $e->getResponse()->getHeader('WWW-Authenticate');
100
101			if ([] !== $wwwAuthenticateHeaders) {
102			if (false !== strpos($wwwAuthenticateHeaders[0], 'error_description="expired token"')) {
103			throw new ExpiredAccessTokenException;
104			}
105			}
106
107			throw new ClientException($e->getMessage(), $e->getCode(), $e);
108			} catch (GuzzleServerException $e) {
109			throw new ServerException($e->getMessage(), $e->getCode(), $e);
110			} catch (GuzzleConnectException $e) {
111			throw new ConnectException($e->getMessage(), $e->getCode(), $e);
112			}
113
114			return $this->api->distillResponse($this->decodeResponse($rawResponse));
115			// $response = $request->response()->setData($distilledResponse);
			0 ignored issues – show Unused Code Comprehensibility introduced 2017-05-30 03:07 UTC by Report Bug Copy Issue Report `63%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
116			}
117
118			/**
119			* @param AbstractApi $api
120			*/
121			private function setApi(AbstractApi $api)
122			{
123			$this->api = $api;
124			}
125
126			/**
127			* @param array $options
128			* @return array
129			*/
130			private function setCertAndSslKey(array $options): array
131			{
132			if (null !== $this->cert && null !== $this->sslKey) {
133			$options[RequestOptions::CERT] = $this->cert,
			0 ignored issues – show Bug introduced 2017-06-06 05:49 UTC by Report Bug Copy Issue Report This code did not parse for me. Apparently, there is an error somewhere around this line: Syntax error, unexpected ',' Loading history...
134			$options[RequestOptions::SSL_KEY] = $this->sslKey;
135			}
136
137			return $options;
138			}
139
140			/**
141			* @param array $options
142			* @return array
143			*/
144			private function setAuthorizationHeader(array $options): array
145			{
146			$options['headers'] = [
147			'Authorization' => 'Bearer ' . $this->accessToken,
148			];
149
150			return $options;
151			}
152
153			/**
154			* @param AbstractRequest $request
155			* @return array
156			*/
157			private function setRequestParams(AbstractRequest $request): array
158			{
159			$options = [];
160			if ($this->api->httpMethod()->equals(HttpMethod::GET())) {
161			$options = $this->setRequestParamsForGetRequest($options, $request);
162			} else {
163			if ($this->api->httpMethod()->equals(HttpMethod::POST())) {
164			$options = $this->setRequestParamsForPostRequest($options, $request);
165			}
166			}
167
168			return $options;
169			}
170
171			/**
172			* @param array $options
173			* @param AbstractRequest $request
174			* @return array
175			*/
176			private function setRequestParamsForGetRequest(array $options, AbstractRequest $request): array
177			{
178			$options['query'] = $request->getParams();
179
180			return $options;
181			}
182
183			/**
184			* @param array $options
185			* @param AbstractRequest $request
186			* @return array
187			*/
188			private function setRequestParamsForPostRequest(array $options, AbstractRequest $request): array
189			{
190			if ($this->api->expectsFormFields()) {
191			$options['form_params'] = $request->getParams();
192			} else {
193			$options['body'] = $request->getParams();
194			}
195
196			return $options;
197			}
198
199
200			/**
201			* @param $options
202			* @return mixed\|ResponseInterface
203			*/
204			private function request($options)
205			{
206			return $this->httpClient->request(
207			$this->api->httpMethod()->getValue(),
208			$this->api->path(),
209			$options
210			);
211			}
212
213			/**
214			* @param ResponseInterface $rawResponse
215			* @return array
216			*/
217			private function decodeResponse(ResponseInterface $rawResponse): array
218			{
219			$formatter = Formatter::make($rawResponse->getBody()->getContents(), Formatter::XML);
220			return $formatter->toArray();
221			}
222			}
223

w-takumi / yahoo-shopping-jp-php

Push — develop ( fb50a1...a6afd0 )

Client::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like