RefundRequest::build() - Code Metrics - wirecardBrasil/magento2 - Measure and Improve Code Quality continuously with Scrutinizer

RefundRequest::build() B
last analyzed 2023-04-13 13:06 UTC

↳ Parent: RefundRequest

Complexity

Conditions	5
Paths	4

Size

Total Lines	70
Code Lines	44

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	5
eloc	44
c	1
b	0
f	0
nc	4
nop	1
dl	0
loc	70
rs	8.9048

How to fix Long Method

<?php
/**
 * Copyright © Wirecard Brasil. All rights reserved.
 *
 * @author    Bruno Elisei <[email protected]>
 * See COPYING.txt for license details.
 */

namespace Moip\Magento2\Gateway\Request;

use Magento\Payment\Gateway\ConfigInterface;
filter:
    dependency_paths: ["lib/*"]
use Magento\Payment\Gateway\Data\PaymentDataObjectInterface;
filter:
    dependency_paths: ["lib/*"]
use Magento\Payment\Gateway\Request\BuilderInterface;
filter:
    dependency_paths: ["lib/*"]
use Moip\Magento2\Gateway\Config\Config;
use Moip\Magento2\Model\Ui\ConfigProviderBoleto;

/**
 * Class RefundRequest - Refund data structure.
 */
class RefundRequest implements BuilderInterface
{
    /**
     * @var Moip Order Id
     */
    const MOIP_ORDER_ID = 'moip_order_id';

    const BANK_NUMBER = 'moip_magento2_boleto_bank_number';

    const AGENCY_NUMBER = 'moip_magento2_boleto_agency_number';

    const AGENCY_CHECK_NUMBER = 'moip_magento2_boleto_agency_check_number';

    const ACCOUNT_NUMBER = 'moip_magento2_boleto_account_number';

    const ACCOUNT_CHECK_NUMBER = 'moip_magento2_boleto_account_check_number';

    const HOLDER_FULLNAME = 'moip_magento2_boleto_account_holder_fullname';

    const HOLDER_DOCUMENT_NUMBER = 'moip_magento2_boleto_account_holder_document_number';

    /**
     * @var ConfigInterface
     */
    private $config;

    /**
     * @var Config
     */
    private $configPayment;

    /**
     * @param ConfigInterface $config
     */
    public function __construct(
        ConfigInterface $config,
        Config $configPayment
    ) {
        $this->config = $config;
        $this->configPayment = $configPayment;
    }

    /**
     * {@inheritdoc}
     */
    public function build(array $buildSubject)
    {
        if (!isset($buildSubject['payment'])
            || !$buildSubject['payment'] instanceof PaymentDataObjectInterface
        ) {
            throw new \InvalidArgumentException('Payment data object should be provided');
        }

        $result = [];


        $paymentDO = $buildSubject['payment'];

        $payment = $paymentDO->getPayment();

        $order = $payment->getOrder();

        $creditmemo = $payment->getCreditMemo();

        $total = $creditmemo->getGrandTotal();

        $result = [
            self::MOIP_ORDER_ID => $order->getExtOrderId(),
            'send'              => [
                'amount' => $this->configPayment->formatPrice($total),
            ],
        ];

        if ($order->getPayment()->getMethodInstance()->getCode() === ConfigProviderBoleto::CODE) {
            $bankNumber = $creditmemo->getData(self::BANK_NUMBER);
            $agencyNumber = $creditmemo->getData(self::AGENCY_NUMBER);
            $agencyCheckNumber = $creditmemo->getData(self::AGENCY_CHECK_NUMBER);
            $accountNumber = $creditmemo->getData(self::ACCOUNT_NUMBER);
            $accountCheckNumber = $creditmemo->getData(self::ACCOUNT_CHECK_NUMBER);
            $holderFullname = $creditmemo->getData(self::HOLDER_FULLNAME);
            $holderDocumment = $creditmemo->getData(self::HOLDER_DOCUMENT_NUMBER);

            $typeDocument = 'CPF';
            $taxDocument = preg_replace('/[^0-9]/', '', $holderDocumment);
            if (strlen($taxDocument) === 14) {
                $typeDocument = 'CNPJ';
            }

            $resultBoleto = [
                'send' => [
                    'amount'              => $this->configPayment->formatPrice($total),
                    'refundingInstrument' => [
                        'method'      => 'BANK_ACCOUNT',
                        'bankAccount' => [
                            'type'               => 'CHECKING',
                            'bankNumber'         => $bankNumber,
                            'agencyNumber'       => $agencyNumber,
                            'agencyCheckNumber'  => $agencyCheckNumber,
                            'accountNumber'      => $accountNumber,
                            'accountCheckNumber' => $accountCheckNumber,
                            'holder'             => [
                                'fullname'    => $holderFullname,
                                'taxDocument' => [
                                    'type'   => $typeDocument,
                                    'number' => $taxDocument,
                                ],
                            ],
                        ],
                    ],
                ],
            ];

            $result = array_merge($result, $resultBoleto);
        }

        return $result;
    }
}


1			<?php
2			/**
3			* Copyright © Wirecard Brasil. All rights reserved.
4			*
5			* @author Bruno Elisei <[email protected]>
6			* See COPYING.txt for license details.
7			*/
8
9			namespace Moip\Magento2\Gateway\Request;
10
11			use Magento\Payment\Gateway\ConfigInterface;
			0 ignored issues – show Bug introduced 2020-12-09 14:45 UTC by Report Bug Copy Issue Report The type `Magento\Payment\Gateway\ConfigInterface` 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			use Magento\Payment\Gateway\Data\PaymentDataObjectInterface;
			0 ignored issues – show Bug introduced 2020-12-09 14:45 UTC by Report Bug Copy Issue Report The type `Magento\Payment\Gateway\...mentDataObjectInterface` 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...
13			use Magento\Payment\Gateway\Request\BuilderInterface;
			0 ignored issues – show Bug introduced 2020-12-09 14:45 UTC by Report Bug Copy Issue Report The type `Magento\Payment\Gateway\Request\BuilderInterface` 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...
14			use Moip\Magento2\Gateway\Config\Config;
15			use Moip\Magento2\Model\Ui\ConfigProviderBoleto;
16
17			/**
18			* Class RefundRequest - Refund data structure.
19			*/
20			class RefundRequest implements BuilderInterface
21			{
22			/**
23			* @var Moip Order Id
24			*/
25			const MOIP_ORDER_ID = 'moip_order_id';
26
27			const BANK_NUMBER = 'moip_magento2_boleto_bank_number';
28
29			const AGENCY_NUMBER = 'moip_magento2_boleto_agency_number';
30
31			const AGENCY_CHECK_NUMBER = 'moip_magento2_boleto_agency_check_number';
32
33			const ACCOUNT_NUMBER = 'moip_magento2_boleto_account_number';
34
35			const ACCOUNT_CHECK_NUMBER = 'moip_magento2_boleto_account_check_number';
36
37			const HOLDER_FULLNAME = 'moip_magento2_boleto_account_holder_fullname';
38
39			const HOLDER_DOCUMENT_NUMBER = 'moip_magento2_boleto_account_holder_document_number';
40
41			/**
42			* @var ConfigInterface
43			*/
44			private $config;
45
46			/**
47			* @var Config
48			*/
49			private $configPayment;
50
51			/**
52			* @param ConfigInterface $config
53			*/
54			public function __construct(
55			ConfigInterface $config,
56			Config $configPayment
57			) {
58			$this->config = $config;
59			$this->configPayment = $configPayment;
60			}
61
62			/**
63			* {@inheritdoc}
64			*/
65			public function build(array $buildSubject)
66			{
67			if (!isset($buildSubject['payment'])
68			\|\| !$buildSubject['payment'] instanceof PaymentDataObjectInterface
69			) {
70			throw new \InvalidArgumentException('Payment data object should be provided');
71			}
72
73			$result = [];
			0 ignored issues – show Unused Code introduced 2021-07-08 12:25 UTC by Report Bug Copy Issue Report The assignment to `$result` is dead and can be removed. Loading history...
74
75			$paymentDO = $buildSubject['payment'];
76
77			$payment = $paymentDO->getPayment();
78
79			$order = $payment->getOrder();
80
81			$creditmemo = $payment->getCreditMemo();
82
83			$total = $creditmemo->getGrandTotal();
84
85			$result = [
86			self::MOIP_ORDER_ID => $order->getExtOrderId(),
87			'send' => [
88			'amount' => $this->configPayment->formatPrice($total),
89			],
90			];
91
92			if ($order->getPayment()->getMethodInstance()->getCode() === ConfigProviderBoleto::CODE) {
93			$bankNumber = $creditmemo->getData(self::BANK_NUMBER);
94			$agencyNumber = $creditmemo->getData(self::AGENCY_NUMBER);
95			$agencyCheckNumber = $creditmemo->getData(self::AGENCY_CHECK_NUMBER);
96			$accountNumber = $creditmemo->getData(self::ACCOUNT_NUMBER);
97			$accountCheckNumber = $creditmemo->getData(self::ACCOUNT_CHECK_NUMBER);
98			$holderFullname = $creditmemo->getData(self::HOLDER_FULLNAME);
99			$holderDocumment = $creditmemo->getData(self::HOLDER_DOCUMENT_NUMBER);
100
101			$typeDocument = 'CPF';
102			$taxDocument = preg_replace('/[^0-9]/', '', $holderDocumment);
103			if (strlen($taxDocument) === 14) {
104			$typeDocument = 'CNPJ';
105			}
106
107			$resultBoleto = [
108			'send' => [
109			'amount' => $this->configPayment->formatPrice($total),
110			'refundingInstrument' => [
111			'method' => 'BANK_ACCOUNT',
112			'bankAccount' => [
113			'type' => 'CHECKING',
114			'bankNumber' => $bankNumber,
115			'agencyNumber' => $agencyNumber,
116			'agencyCheckNumber' => $agencyCheckNumber,
117			'accountNumber' => $accountNumber,
118			'accountCheckNumber' => $accountCheckNumber,
119			'holder' => [
120			'fullname' => $holderFullname,
121			'taxDocument' => [
122			'type' => $typeDocument,
123			'number' => $taxDocument,
124			],
125			],
126			],
127			],
128			],
129			];
130
131			$result = array_merge($result, $resultBoleto);
132			}
133
134			return $result;
135			}
136			}
137

wirecardBrasil / magento2

RefundRequest::build() B last analyzed 2023-04-13 13:06 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like

RefundRequest::build() B
last analyzed 2023-04-13 13:06 UTC