wirecardBrasil / magento2

Controller/Webhooks/Accept.php

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

namespace Moip\Magento2\Controller\Webhooks;

use Magento\Framework\App\Action\Action;

The type Magento\Framework\App\Action\Action was not found. Maybe you did not declare it correctly or list all dependencies?

use Magento\Framework\App\Action\Context;

The type Magento\Framework\App\Action\Context was not found. Maybe you did not declare it correctly or list all dependencies?

use Magento\Framework\App\CsrfAwareActionInterface as Csrf;

The type Magento\Framework\App\CsrfAwareActionInterface was not found. Maybe you did not declare it correctly or list all dependencies?

use Magento\Framework\App\Request\InvalidRequestException;

The type Magento\Framework\App\Request\InvalidRequestException was not found. Maybe you did not declare it correctly or list all dependencies?

use Magento\Framework\App\RequestInterface;

The type Magento\Framework\App\RequestInterface was not found. Maybe you did not declare it correctly or list all dependencies?

use Magento\Framework\Controller\Result\JsonFactory;

The type Magento\Framework\Controller\Result\JsonFactory was not found. Maybe you did not declare it correctly or list all dependencies?

use Magento\Framework\Serialize\Serializer\Json;

The type Magento\Framework\Serialize\Serializer\Json was not found. Maybe you did not declare it correctly or list all dependencies?

use Magento\Payment\Model\Method\Logger;

The type Magento\Payment\Model\Method\Logger was not found. Maybe you did not declare it correctly or list all dependencies?

use Magento\Sales\Api\Data\OrderInterfaceFactory;

The type Magento\Sales\Api\Data\OrderInterfaceFactory was not found. Maybe you did not declare it correctly or list all dependencies?

use Magento\Sales\Model\Order\CreditmemoFactory;

The type Magento\Sales\Model\Order\CreditmemoFactory was not found. Maybe you did not declare it correctly or list all dependencies?

use Magento\Sales\Model\Order\Invoice;

The type Magento\Sales\Model\Order\Invoice was not found. Maybe you did not declare it correctly or list all dependencies?

use Magento\Sales\Model\Service\CreditmemoService;

The type Magento\Sales\Model\Service\CreditmemoService was not found. Maybe you did not declare it correctly or list all dependencies?

use Magento\Store\Model\StoreManagerInterface;

The type Magento\Store\Model\StoreManagerInterface was not found. Maybe you did not declare it correctly or list all dependencies?

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) {

$request is of type Magento\Framework\App\RequestInterface, thus it always evaluated to true.

            return null;

The expression return null returns the type null which is incompatible with the type-hinted return Magento\Framework\App\Request\InvalidRequestException.

        }

The function implicitly returns null when the if condition on line 41 is false. This is incompatible with the type-hinted return Magento\Framework\App\Request\InvalidRequestException. Consider adding a return statement or allowing null as return value.

    }

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

$request is of type Magento\Framework\App\RequestInterface, thus it always evaluated to true.

            return true;
        }

The function implicitly returns null when the if condition on line 55 is false. This is incompatible with the type-hinted return boolean. Consider adding a return statement or allowing null as return value.

    }

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

    /**
     * @var orderFactory

The type Moip\Magento2\Controller\Webhooks\orderFactory was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    protected $orderFactory;

    /**
     * @var resultJsonFactory

The type Moip\Magento2\Controller\Webhooks\resultJsonFactory was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    protected $resultJsonFactory;

    /**
     * @var storeManager

The type Moip\Magento2\Controller\Webhooks\storeManager was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    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;

The property config does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The property creditmemoFactory does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->creditmemoService = $creditmemoService;

The property creditmemoService does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->invoice = $invoice;

The property invoice does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $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'),

The function __ was not found. Maybe you did not declare it correctly or list all dependencies?

                    ])
                );
            }

            $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;
    }
}