IsmpApi::__construct() - Code Metrics - Inspection of "Add marked_products end-point" - lamoda/crpt-ismp-api-client - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#10)

unknown

created 2020-02-11 09:29 UTC

IsmpApi::__construct() A

↳ Parent: IsmpApi

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	5
c	0
b	0
f	0
rs	10
ccs	4
cts	4
cp	1
cc	1
nc	1
nop	2
crap	1

<?php

declare(strict_types=1);

namespace Lamoda\IsmpClient\V3;

use GuzzleHttp\ClientInterface;
use GuzzleHttp\Exception\BadResponseException;
use GuzzleHttp\RequestOptions;
use Lamoda\IsmpClient\Exception\IsmpGeneralErrorException;
use Lamoda\IsmpClient\V3\Dto\AuthCertKeyResponse;
use Lamoda\IsmpClient\V3\Dto\AuthCertRequest;
use Lamoda\IsmpClient\V3\Dto\AuthCertResponse;
use Lamoda\IsmpClient\V3\Dto\FacadeDocBodyResponse;
use Lamoda\IsmpClient\V3\Dto\FacadeCisListResponse;
use Lamoda\IsmpClient\V3\Dto\FacadeDocListV2Query;
use Lamoda\IsmpClient\V3\Dto\FacadeDocListV2Response;
use Lamoda\IsmpClient\V3\Dto\FacadeMarkedProductsResponse;
use Lamoda\IsmpClient\V3\Dto\FacadeOrderDetailsResponse;
use Lamoda\IsmpClient\V3\Dto\FacadeOrderRequest;
use Lamoda\IsmpClient\V3\Dto\FacadeOrderResponse;
use Lamoda\IsmpClient\V3\Dto\DocumentCreateRequest;
use Lamoda\IsmpClient\Exception\IsmpRequestErrorException;
use Lamoda\IsmpClient\Serializer\SerializerInterface;
use Lamoda\IsmpClient\V3\Dto\ProductInfoResponse;

final class IsmpApi implements IsmpApiInterface
{
    /**
     * @var ClientInterface
     */
    private $client;
    /**
     * @var SerializerInterface
     */
    private $serializer;

    public function __construct(ClientInterface $client, SerializerInterface $serializer)
    {
        $this->client = $client;
        $this->serializer = $serializer;
    }

    public function authCertKey(): AuthCertKeyResponse
    {
        $result = $this->request('GET', '/api/v3/auth/cert/key');

        /* @noinspection PhpIncompatibleReturnTypeInspection */
        return $this->serializer->deserialize(AuthCertKeyResponse::class, $result);
    }

    public function authCert(AuthCertRequest $request): AuthCertResponse

    {
        $body = $this->serializer->serialize($request);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $result = $this->request('POST', '/api/v3/auth/cert/', $body);

        /* @noinspection PhpIncompatibleReturnTypeInspection */
        return $this->serializer->deserialize(AuthCertResponse::class, $result);
    }

    public function facadeOrder(string $token, FacadeOrderRequest $request): FacadeOrderResponse

    {
        $body = $this->serializer->serialize($request);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $result = $this->request('POST', '/api/v3/facade/order/', $body, null, $token);

        /* @noinspection PhpIncompatibleReturnTypeInspection */
        return $this->serializer->deserialize(FacadeOrderResponse::class, $result);
    }

    public function facadeOrderDetails(string $token, string $orderId): FacadeOrderDetailsResponse

    {
        $result = $this->request('GET', sprintf('/api/v3/facade/order/%s/details', $orderId), null, null, $token);

        /* @noinspection PhpIncompatibleReturnTypeInspection */
        return $this->serializer->deserialize(FacadeOrderDetailsResponse::class, $result);
    }

    public function facadeDocListV2(string $token, FacadeDocListV2Query $query): FacadeDocListV2Response
    {
        $result = $this->request('GET', '/api/v3/facade/doc/listV2', null, $query->toQueryArray(), $token);

        /* @noinspection PhpIncompatibleReturnTypeInspection */
        return $this->serializer->deserialize(FacadeDocListV2Response::class, $result);
    }

    public function facadeDocBody(string $token, string $docId): FacadeDocBodyResponse

    {
        $result = $this->request('GET', sprintf('/api/v3/facade/doc/%s/body', $docId), null, null, $token);

        /* @noinspection PhpIncompatibleReturnTypeInspection */
        return $this->serializer->deserialize(FacadeDocBodyResponse::class, $result);
    }

    public function facadeCisList(string $token, string $cis): FacadeCisListResponse

    {
        $response = $this->request('GET', '/api/v3/facade/cis/cis_list', null, ['cis' => $cis], $token);

        /* @noinspection PhpIncompatibleReturnTypeInspection */
        return $this->serializer->deserialize(FacadeCisListResponse::class, $response);
    }

    public function facadeMarkedProducts(string $token, string $uit): FacadeMarkedProductsResponse

    {
        $response = $this->request('GET', '/api/v3/facade/marked_products', null, ['uit' => $uit], $token);

        /** @noinspection PhpIncompatibleReturnTypeInspection */
        return $this->serializer->deserialize(FacadeMarkedProductsResponse::class, $response);
    }

    public function lkDocumentsCreate(string $token, DocumentCreateRequest $request): string
    {
        assert($request->getType() !== null, 'Document type is required for lkDocumentsCreate');
        assert($request->getProductGroup() !== null, 'Product group is required for lkDocumentsCreate');

        $body = $this->serializer->serialize($request);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        return $this->request(
            'POST',
            '/api/v3/lk/documents/create',
            $body,
            ['pg' => $request->getProductGroup()],
            $token
        );
    }

    public function lkImportSend(string $token, DocumentCreateRequest $request): string
    {
        $body = $this->serializer->serialize($request);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        return $this->request('POST', '/api/v3/lk/import/send', $body, null, $token);
    }

    public function lkReceiptSend(string $token, DocumentCreateRequest $request): string
    {
        $body = $this->serializer->serialize($request);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        return $this->request('POST', '/api/v3/lk/receipt/send', $body, null, $token);
    }

    public function lkDocumentsShipmentCreate(string $token, DocumentCreateRequest $request): string
    {
        $body = $this->serializer->serialize($request);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        return $this->request('POST', '/api/v3/lk/documents/shipment/create', $body, null, $token);
    }

    public function lkDocumentsAcceptanceCreate(string $token, DocumentCreateRequest $request): string
    {
        $body = $this->serializer->serialize($request);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        return $this->request('POST', '/api/v3/lk/documents/acceptance/create', $body, null, $token);
    }

    public function productInfo(string $token, array $gtins): ProductInfoResponse
    {
        $gtinsList = implode(',', $gtins);

        $result = $this->request('GET', 'api/v3/product/info', null, [
            'gtins' => $gtinsList,
        ], $token);

        /* @noinspection PhpIncompatibleReturnTypeInspection */
        return $this->serializer->deserialize(ProductInfoResponse::class, $result);
    }

    private function request(string $method, string $uri, $body = null, $query = null, string $token = null): string
    {
        $options = [
            RequestOptions::BODY => $body,
            RequestOptions::HEADERS => [
                'Content-Type' => 'application/json',
            ],
            RequestOptions::HTTP_ERRORS => true,
            RequestOptions::QUERY => $query,
        ];

        if ($token) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $options[RequestOptions::HEADERS]['Authorization'] = 'Bearer ' . $token;
        }

        $uri = ltrim($uri, '/');

        try {
            $result = $this->client->request($method, $uri, $options);
        } catch (\Throwable $exception) {
            /* @noinspection PhpUnhandledExceptionInspection */
            throw $this->handleRequestException($exception);
        }

        return (string) $result->getBody();
    }

    private function handleRequestException(\Throwable $exception): \Throwable
    {
        if ($exception instanceof BadResponseException) {
            $response = $exception->getResponse();
            $responseBody = $response ? (string) $response->getBody() : '';
            $responseCode = $response ? $response->getStatusCode() : 0;

            return IsmpRequestErrorException::becauseOfErrorResponse($responseCode, $responseBody, $exception);
        }

        return IsmpGeneralErrorException::becauseOfError($exception);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Lamoda\IsmpClient\V3;
6
7			use GuzzleHttp\ClientInterface;
8			use GuzzleHttp\Exception\BadResponseException;
9			use GuzzleHttp\RequestOptions;
10			use Lamoda\IsmpClient\Exception\IsmpGeneralErrorException;
11			use Lamoda\IsmpClient\V3\Dto\AuthCertKeyResponse;
12			use Lamoda\IsmpClient\V3\Dto\AuthCertRequest;
13			use Lamoda\IsmpClient\V3\Dto\AuthCertResponse;
14			use Lamoda\IsmpClient\V3\Dto\FacadeDocBodyResponse;
15			use Lamoda\IsmpClient\V3\Dto\FacadeCisListResponse;
16			use Lamoda\IsmpClient\V3\Dto\FacadeDocListV2Query;
17			use Lamoda\IsmpClient\V3\Dto\FacadeDocListV2Response;
18			use Lamoda\IsmpClient\V3\Dto\FacadeMarkedProductsResponse;
19			use Lamoda\IsmpClient\V3\Dto\FacadeOrderDetailsResponse;
20			use Lamoda\IsmpClient\V3\Dto\FacadeOrderRequest;
21			use Lamoda\IsmpClient\V3\Dto\FacadeOrderResponse;
22			use Lamoda\IsmpClient\V3\Dto\DocumentCreateRequest;
23			use Lamoda\IsmpClient\Exception\IsmpRequestErrorException;
24			use Lamoda\IsmpClient\Serializer\SerializerInterface;
25			use Lamoda\IsmpClient\V3\Dto\ProductInfoResponse;
26
27			final class IsmpApi implements IsmpApiInterface
28			{
29			/**
30			* @var ClientInterface
31			*/
32			private $client;
33			/**
34			* @var SerializerInterface
35			*/
36			private $serializer;
37
38	15		public function __construct(ClientInterface $client, SerializerInterface $serializer)
39			{
40	15		$this->client = $client;
41	15		$this->serializer = $serializer;
42	15		}
43
44	3		public function authCertKey(): AuthCertKeyResponse
45			{
46	3		$result = $this->request('GET', '/api/v3/auth/cert/key');
47
48			/* @noinspection PhpIncompatibleReturnTypeInspection */
49	1		return $this->serializer->deserialize(AuthCertKeyResponse::class, $result);
50			}
51
52	1	View Code Duplication	public function authCert(AuthCertRequest $request): AuthCertResponse
			0 ignored issues – show Duplication introduced 2019-12-10 14:55 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
53			{
54	1		$body = $this->serializer->serialize($request);
			0 ignored issues – show Documentation introduced 2019-12-10 14:55 UTC by Report Bug Copy Issue Report `$request` is of type `object<Lamoda\IsmpClient\V3\Dto\AuthCertRequest>`, but the function expects a `object<Lamoda\IsmpClient\Serializer\object>`. 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...
55	1		$result = $this->request('POST', '/api/v3/auth/cert/', $body);
56
57			/* @noinspection PhpIncompatibleReturnTypeInspection */
58	1		return $this->serializer->deserialize(AuthCertResponse::class, $result);
59			}
60
61	1	View Code Duplication	public function facadeOrder(string $token, FacadeOrderRequest $request): FacadeOrderResponse
			0 ignored issues – show Duplication introduced 2019-12-10 14:55 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
62			{
63	1		$body = $this->serializer->serialize($request);
			0 ignored issues – show Documentation introduced 2019-12-10 14:55 UTC by Report Bug Copy Issue Report `$request` is of type `object<Lamoda\IsmpClient...Dto\FacadeOrderRequest>`, but the function expects a `object<Lamoda\IsmpClient\Serializer\object>`. 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...
64	1		$result = $this->request('POST', '/api/v3/facade/order/', $body, null, $token);
65
66			/* @noinspection PhpIncompatibleReturnTypeInspection */
67	1		return $this->serializer->deserialize(FacadeOrderResponse::class, $result);
68			}
69
70	1	View Code Duplication	public function facadeOrderDetails(string $token, string $orderId): FacadeOrderDetailsResponse
			0 ignored issues – show Duplication introduced 2019-12-10 14:55 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
71			{
72	1		$result = $this->request('GET', sprintf('/api/v3/facade/order/%s/details', $orderId), null, null, $token);
73
74			/* @noinspection PhpIncompatibleReturnTypeInspection */
75	1		return $this->serializer->deserialize(FacadeOrderDetailsResponse::class, $result);
76			}
77
78	1		public function facadeDocListV2(string $token, FacadeDocListV2Query $query): FacadeDocListV2Response
79			{
80	1		$result = $this->request('GET', '/api/v3/facade/doc/listV2', null, $query->toQueryArray(), $token);
81
82			/* @noinspection PhpIncompatibleReturnTypeInspection */
83	1		return $this->serializer->deserialize(FacadeDocListV2Response::class, $result);
84			}
85
86	1	View Code Duplication	public function facadeDocBody(string $token, string $docId): FacadeDocBodyResponse
			0 ignored issues – show Duplication introduced 2019-12-10 14:55 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
87			{
88	1		$result = $this->request('GET', sprintf('/api/v3/facade/doc/%s/body', $docId), null, null, $token);
89
90			/* @noinspection PhpIncompatibleReturnTypeInspection */
91	1		return $this->serializer->deserialize(FacadeDocBodyResponse::class, $result);
92			}
93
94	1	View Code Duplication	public function facadeCisList(string $token, string $cis): FacadeCisListResponse
			0 ignored issues – show Duplication introduced 2019-12-10 14:55 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
95			{
96	1		$response = $this->request('GET', '/api/v3/facade/cis/cis_list', null, ['cis' => $cis], $token);
97
98			/* @noinspection PhpIncompatibleReturnTypeInspection */
99	1		return $this->serializer->deserialize(FacadeCisListResponse::class, $response);
100			}
101
102		View Code Duplication	public function facadeMarkedProducts(string $token, string $uit): FacadeMarkedProductsResponse
			0 ignored issues – show Duplication introduced 2020-02-11 09:31 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
103			{
104			$response = $this->request('GET', '/api/v3/facade/marked_products', null, ['uit' => $uit], $token);
105
106			/** @noinspection PhpIncompatibleReturnTypeInspection */
107			return $this->serializer->deserialize(FacadeMarkedProductsResponse::class, $response);
108			}
109
110	1		public function lkDocumentsCreate(string $token, DocumentCreateRequest $request): string
111			{
112	1		assert($request->getType() !== null, 'Document type is required for lkDocumentsCreate');
113	1		assert($request->getProductGroup() !== null, 'Product group is required for lkDocumentsCreate');
114
115	1		$body = $this->serializer->serialize($request);
			0 ignored issues – show Documentation introduced 2019-12-10 14:55 UTC by Report Bug Copy Issue Report `$request` is of type `object<Lamoda\IsmpClient...\DocumentCreateRequest>`, but the function expects a `object<Lamoda\IsmpClient\Serializer\object>`. 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...
116
117	1		return $this->request(
118	1		'POST',
119	1		'/api/v3/lk/documents/create',
120	1		$body,
121	1		['pg' => $request->getProductGroup()],
122	1		$token
123			);
124			}
125
126	1		public function lkImportSend(string $token, DocumentCreateRequest $request): string
127			{
128	1		$body = $this->serializer->serialize($request);
			0 ignored issues – show Documentation introduced 2019-12-10 14:55 UTC by Report Bug Copy Issue Report `$request` is of type `object<Lamoda\IsmpClient...\DocumentCreateRequest>`, but the function expects a `object<Lamoda\IsmpClient\Serializer\object>`. 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...
129
130	1		return $this->request('POST', '/api/v3/lk/import/send', $body, null, $token);
131			}
132
133	1		public function lkReceiptSend(string $token, DocumentCreateRequest $request): string
134			{
135	1		$body = $this->serializer->serialize($request);
			0 ignored issues – show Documentation introduced 2019-12-30 13:32 UTC by Report Bug Copy Issue Report `$request` is of type `object<Lamoda\IsmpClient...\DocumentCreateRequest>`, but the function expects a `object<Lamoda\IsmpClient\Serializer\object>`. 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...
136
137	1		return $this->request('POST', '/api/v3/lk/receipt/send', $body, null, $token);
138			}
139
140	1		public function lkDocumentsShipmentCreate(string $token, DocumentCreateRequest $request): string
141			{
142	1		$body = $this->serializer->serialize($request);
			0 ignored issues – show Documentation introduced 2019-12-10 14:55 UTC by Report Bug Copy Issue Report `$request` is of type `object<Lamoda\IsmpClient...\DocumentCreateRequest>`, but the function expects a `object<Lamoda\IsmpClient\Serializer\object>`. 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...
143
144	1		return $this->request('POST', '/api/v3/lk/documents/shipment/create', $body, null, $token);
145			}
146
147	1		public function lkDocumentsAcceptanceCreate(string $token, DocumentCreateRequest $request): string
148			{
149	1		$body = $this->serializer->serialize($request);
			0 ignored issues – show Documentation introduced 2020-02-07 12:47 UTC by Report Bug Copy Issue Report `$request` is of type `object<Lamoda\IsmpClient...\DocumentCreateRequest>`, but the function expects a `object<Lamoda\IsmpClient\Serializer\object>`. 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...
150
151	1		return $this->request('POST', '/api/v3/lk/documents/acceptance/create', $body, null, $token);
152			}
153
154	1		public function productInfo(string $token, array $gtins): ProductInfoResponse
155			{
156	1		$gtinsList = implode(',', $gtins);
157
158	1		$result = $this->request('GET', 'api/v3/product/info', null, [
159	1		'gtins' => $gtinsList,
160	1		], $token);
161
162			/* @noinspection PhpIncompatibleReturnTypeInspection */
163	1		return $this->serializer->deserialize(ProductInfoResponse::class, $result);
164			}
165
166	15		private function request(string $method, string $uri, $body = null, $query = null, string $token = null): string
167			{
168			$options = [
169	15		RequestOptions::BODY => $body,
170	15		RequestOptions::HEADERS => [
171			'Content-Type' => 'application/json',
172			],
173	15		RequestOptions::HTTP_ERRORS => true,
174	15		RequestOptions::QUERY => $query,
175			];
176
177	15		if ($token) {
			0 ignored issues – show Bug Best Practice introduced 2019-12-10 14:55 UTC by Report Bug Copy Issue Report The expression `$token` of type `null\|string` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
178	11		$options[RequestOptions::HEADERS]['Authorization'] = 'Bearer ' . $token;
179			}
180
181	15		$uri = ltrim($uri, '/');
182
183			try {
184	15		$result = $this->client->request($method, $uri, $options);
185	2		} catch (\Throwable $exception) {
186			/* @noinspection PhpUnhandledExceptionInspection */
187	2		throw $this->handleRequestException($exception);
188			}
189
190	13		return (string) $result->getBody();
191			}
192
193	2		private function handleRequestException(\Throwable $exception): \Throwable
194			{
195	2		if ($exception instanceof BadResponseException) {
196	1		$response = $exception->getResponse();
197	1		$responseBody = $response ? (string) $response->getBody() : '';
198	1		$responseCode = $response ? $response->getStatusCode() : 0;
199
200	1		return IsmpRequestErrorException::becauseOfErrorResponse($responseCode, $responseBody, $exception);
201			}
202
203	1		return IsmpGeneralErrorException::becauseOfError($exception);
204			}
205			}
206

lamoda / crpt-ismp-api-client

Pull Request — master (#10)

IsmpApi::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like