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

Accept::execute() B
last analyzed 2023-04-13 13:06 UTC

↳ Parent: Accept
Complexity

Conditions	6
Paths	8
Size

Total Lines	73
Code Lines	49
Duplication

Lines	0
Ratio	0 %
Importance

Changes	1
Bugs	0	Features	0
Metric	Value
cc	6
eloc	49
c	1
b	0
f	0
nc	8
nop	0
dl	0
loc	73
rs	8.4905
How to fix Long Method

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

namespace Moip\Magento2\Controller\Webhooks;

use Magento\Framework\App\Action\Action;
filter:
    dependency_paths: ["lib/*"]
use Magento\Framework\App\Action\Context;
filter:
    dependency_paths: ["lib/*"]
use Magento\Framework\App\CsrfAwareActionInterface as Csrf;
filter:
    dependency_paths: ["lib/*"]
use Magento\Framework\App\Request\InvalidRequestException;
filter:
    dependency_paths: ["lib/*"]
use Magento\Framework\App\RequestInterface;
filter:
    dependency_paths: ["lib/*"]
use Magento\Framework\Controller\Result\JsonFactory;
filter:
    dependency_paths: ["lib/*"]
use Magento\Framework\Serialize\Serializer\Json;
filter:
    dependency_paths: ["lib/*"]
use Magento\Payment\Model\Method\Logger;
filter:
    dependency_paths: ["lib/*"]
use Magento\Sales\Api\Data\OrderInterfaceFactory;
filter:
    dependency_paths: ["lib/*"]
use Magento\Sales\Model\Order\CreditmemoFactory;
filter:
    dependency_paths: ["lib/*"]
use Magento\Sales\Model\Order\Invoice;
filter:
    dependency_paths: ["lib/*"]
use Magento\Sales\Model\Service\CreditmemoService;
filter:
    dependency_paths: ["lib/*"]
use Magento\Store\Model\StoreManagerInterface;
filter:
    dependency_paths: ["lib/*"]
use Moip\Magento2\Gateway\Config\Config;

/**
 * Class Accept - Receives communication for accept payment.
 */
class Accept extends Action implements Csrf
{
    /**
     * createCsrfValidationException.
     *
     * @param RequestInterface $request
     *
     * @return null
     */
    public function createCsrfValidationException(RequestInterface $request): InvalidRequestException
    {
        if ($request) {

            return null;

        }
interface ReturnsInt {
    public function returnsIntHinted(): int;
}

class MyClass implements ReturnsInt {
    public function returnsIntHinted(): int
    {
        if (foo()) {
            return 123;
        }
        // here: null is implicitly returned
    }
}
    }

    /**
     * validateForCsrf.
     *
     * @param RequestInterface $request
     *
     * @return bool true
     */
    public function validateForCsrf(RequestInterface $request): bool
    {
        if ($request) {

            return true;
        }
interface ReturnsInt {
    public function returnsIntHinted(): int;
}

class MyClass implements ReturnsInt {
    public function returnsIntHinted(): int
    {
        if (foo()) {
            return 123;
        }
        // here: null is implicitly returned
    }
}
    }

    /**
     * @var logger
     */
    protected $logger;

    /**
     * @var orderFactory
filter:
    dependency_paths: ["lib/*"]
     */
    protected $orderFactory;

    /**
     * @var resultJsonFactory
filter:
    dependency_paths: ["lib/*"]
     */
    protected $resultJsonFactory;

    /**
     * @var storeManager
filter:
    dependency_paths: ["lib/*"]
     */
    protected $storeManager;

    /**
     * @var Json
     */
    protected $json;

    /**
     * @param Context               $context
     * @param logger                $logger
     * @param Config                $config
     * @param OrderInterfaceFactory $orderFactory
     * @param JsonFactory           $resultJsonFactory
     * @param Json                  $json
     */
    public function __construct(
        Context $context,
        Config $config,
        Logger $logger,
        OrderInterfaceFactory $orderFactory,
        CreditmemoFactory $creditmemoFactory,
        CreditmemoService $creditmemoService,
        Invoice $invoice,
        StoreManagerInterface $storeManager,
        JsonFactory $resultJsonFactory,
        Json $json
    ) {
        parent::__construct($context);
        $this->config = $config;

        $this->logger = $logger;
        $this->orderFactory = $orderFactory;
        $this->creditmemoFactory = $creditmemoFactory;

        $this->creditmemoService = $creditmemoService;

        $this->invoice = $invoice;

        $this->storeManager = $storeManager;
        $this->resultJsonFactory = $resultJsonFactory;
        $this->json = $json;
    }

    /**
     * Command Accept.
     *
     * @return json
     */
    public function execute()
    {
        if (!$this->getRequest()->isPost()) {
            $resultPage = $this->resultJsonFactory->create();
            $resultPage->setHttpResponseCode(404);

            return $resultPage;
        }

        $resultPage = $this->resultJsonFactory->create();
        $response = $this->getRequest()->getContent();
        $originalNotification = $this->json->unserialize($response);
        $authorization = $this->getRequest()->getHeader('Authorization');
        $storeId = $this->storeManager->getStore()->getId();
        $storeCaptureToken = $this->config->getMerchantGatewayCaptureToken($storeId);
        if ($storeCaptureToken === $authorization) {
            $data = $originalNotification['resource']['order'];
            $order = $this->orderFactory->create()->load($data['id'], 'ext_order_id');

            if(!$order->getId()) {
                $resultPage->setHttpResponseCode(406);
                return $resultPage->setJsonData(
                    $this->json->serialize([
                        'error' => 400,
                        'message' => __('Can not find this order'),
                        'message' => /** @scrutinizer ignore-call */ __('Can not find this order'),
                    ])
                );
            }

            $this->logger->debug([
                'webhook'            => 'accept',
                'ext_order_id'       => $data['id'],
                'increment_order_id' => $order->getIncrementId(),
                'webhook_data'       => $response,
            ]);
            $payment = $order->getPayment();
            if (!$order->getInvoiceCollection()->count()) {
                try {
                    $isOnline = true;
                    $payment->accept($isOnline);
                    $payment->save();
                    $order->save();
                } catch (\Exception $exc) {
                    $resultPage->setHttpResponseCode(500);
                    return $resultPage->setJsonData(
                        $this->json->serialize([
                            'error'   => 400,
                            'message' => $exc->getMessage(),
                        ])
                    );
                }

                return $resultPage->setJsonData(
                    $this->json->serialize([
                        'success'   => 1,
                        'status'    => $order->getStatus(),
                        'state'     => $order->getState(),
                    ])
                );
            }

            $resultPage->setHttpResponseCode(400);

            return $resultPage->setJsonData(
                $this->json->serialize([
                    'error'   => 400,
                    'message' => 'The transaction could not be refund',
                ])
            );
        }
        $resultPage->setHttpResponseCode(401);

        return $resultPage;
    }
}

wirecardBrasil / magento2

Accept::execute() 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

Accept::execute() B
last analyzed 2023-04-13 13:06 UTC
