ObtainProfileInformationConverter::convert() - Code Metrics - Inspection of "Bugfix/eco 1031 recent review" - spryker-eco/amazon-pay - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — dev (#9)

by Andrey

created 2018-01-29 15:10 UTC

ObtainProfileInformationConverter::convert() A

↳ Parent: ObtainProfileInformationConverter

Complexity

Conditions	1
Paths	1

Size

Total Lines	11
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	6
nc	1
nop	1
dl	0
loc	11
rs	9.4285
c	0
b	0
f	0

<?php

/**
 * Apache OSL-2
 * Use of this software requires acceptance of the Evaluation License Agreement. See LICENSE file.
 */

namespace SprykerEco\Zed\AmazonPay\Business\Api\Converter;

use Generated\Shared\Transfer\AmazonpayResponseTransfer;
filter:
    dependency_paths: ["lib/*"]
use Generated\Shared\Transfer\CustomerTransfer;
filter:
    dependency_paths: ["lib/*"]

class ObtainProfileInformationConverter extends AbstractArrayConverter
{
    const NAME = 'name';
    const EMAIL = 'email';

    /**
     * @param array $response
     *
     * @return \Generated\Shared\Transfer\AmazonpayResponseTransfer
     */
    public function convert(array $response)
    {
        $responseTransfer = new AmazonpayResponseTransfer();
        $customerTransfer = new CustomerTransfer();

        $this->extractName($response, $customerTransfer);
        $this->extractEmail($response, $customerTransfer);

        $responseTransfer->setCustomer($customerTransfer);

        return $responseTransfer;
    }

    /**
     * @param array $response
     * @param \Generated\Shared\Transfer\CustomerTransfer $responseTransfer
     *
     * @return void
     */
    protected function extractName(array $response, CustomerTransfer $responseTransfer)
    {
        if (!empty($response[self::NAME])) {
            $this->updateNameData($responseTransfer, $response[self::NAME]);
        }
    }

    /**
     * @param array $response
     * @param \Generated\Shared\Transfer\CustomerTransfer $responseTransfer
     *
     * @return void
     */
    protected function extractEmail(array $response, CustomerTransfer $responseTransfer)
    {
        if (!empty($response[self::EMAIL])) {
            $responseTransfer->setEmail($response[self::EMAIL]);
        }
    }
}


1			<?php
2
3			/**
4			* Apache OSL-2
5			* Use of this software requires acceptance of the Evaluation License Agreement. See LICENSE file.
6			*/
7
8			namespace SprykerEco\Zed\AmazonPay\Business\Api\Converter;
9
10			use Generated\Shared\Transfer\AmazonpayResponseTransfer;
			1 ignored issue – show Bug introduced 2017-12-20 14:20 UTC by Report Bug Copy Issue Report The type `Generated\Shared\Transfe...azonpayResponseTransfer` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
11			use Generated\Shared\Transfer\CustomerTransfer;
			1 ignored issue – show Bug introduced 2017-09-29 10:26 UTC by Report Bug Copy Issue Report The type `Generated\Shared\Transfer\CustomerTransfer` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
12
13			class ObtainProfileInformationConverter extends AbstractArrayConverter
14			{
15			const NAME = 'name';
16			const EMAIL = 'email';
17
18			/**
19			* @param array $response
20			*
21			* @return \Generated\Shared\Transfer\AmazonpayResponseTransfer
22			*/
23			public function convert(array $response)
24			{
25			$responseTransfer = new AmazonpayResponseTransfer();
26			$customerTransfer = new CustomerTransfer();
27
28			$this->extractName($response, $customerTransfer);
29			$this->extractEmail($response, $customerTransfer);
30
31			$responseTransfer->setCustomer($customerTransfer);
32
33			return $responseTransfer;
34			}
35
36			/**
37			* @param array $response
38			* @param \Generated\Shared\Transfer\CustomerTransfer $responseTransfer
39			*
40			* @return void
41			*/
42			protected function extractName(array $response, CustomerTransfer $responseTransfer)
43			{
44			if (!empty($response[self::NAME])) {
45			$this->updateNameData($responseTransfer, $response[self::NAME]);
46			}
47			}
48
49			/**
50			* @param array $response
51			* @param \Generated\Shared\Transfer\CustomerTransfer $responseTransfer
52			*
53			* @return void
54			*/
55			protected function extractEmail(array $response, CustomerTransfer $responseTransfer)
56			{
57			if (!empty($response[self::EMAIL])) {
58			$responseTransfer->setEmail($response[self::EMAIL]);
59			}
60			}
61			}
62

spryker-eco / amazon-pay

Pull Request — dev (#9)

ObtainProfileInformationConverter::convert() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like