elisei/pagbank-payment-magento

View on GitHub
Controller/Notification/All.php

Summary

Maintainability
A
1 hr
Test Coverage

Avoid too many return statements within this method.
Open

            return $this->createResult(
                200,
                [
                    'error'   => 200,
                    'message' => __('Not apply.'),
Severity: Minor
Found in Controller/Notification/All.php - About 30 mins to fix

    Avoid too many return statements within this method.
    Open

            return $this->initProcess($psPaymentId);
    Severity: Minor
    Found in Controller/Notification/All.php - About 30 mins to fix

      Avoid too many return statements within this method.
      Open

              return $result;
      Severity: Minor
      Found in Controller/Notification/All.php - About 30 mins to fix

        Argument 1 (order) is \Magento\Framework\Controller\ResultInterface|\Magento\Sales\Model\OrderRepository but \PagBank\PaymentMagento\Controller\Notification\All::processNotification() takes \PagBank\PaymentMagento\Controller\Notification\OrderRepository defined at /code/Controller/Notification/All.php:161
        Open

                    $process = $this->processNotification($order);
        Severity: Minor
        Found in Controller/Notification/All.php by phan

        Argument 1 (order) is \PagBank\PaymentMagento\Controller\Notification\OrderRepository but \PagBank\PaymentMagento\Controller\Notification\All::filterInvalidNotification() takes \Magento\Sales\Model\OrderRepository defined at /code/Controller/AbstractNotification.php:195
        Open

                $isNotApplicable = $this->filterInvalidNotification($order);
        Severity: Minor
        Found in Controller/Notification/All.php by phan

        There are no issues that match your filters.

        Category
        Status