SprykerEco\Yves\Braintree\Processor\PaypalResponseProcessor

Complexity

Total Complexity

2

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	successResponse()	0	3	1
A	__construct()	0	4	1

<?php

namespace SprykerEco\Yves\Braintree\Processor;

use Generated\Shared\Transfer\PaypalExpressSuccessResponseTransfer;

The type Generated\Shared\Transfer\PaypalExpressSuccessResponseTransfer was not found. Maybe you did not declare it correctly or list all dependencies?

use SprykerEco\Yves\Braintree\Mapper\PaypalResponse\PaypalResponseMapperInterface;

class PaypalResponseProcessor implements PaypalResponseProcessorInterface
{
    /**
     * @var PaypalResponseMapperInterface
     */
    protected $paypalResponseMapper;

    /**
     * @param PaypalResponseMapperInterface $paypalResponseMapper
     */
    public function __construct(
        PaypalResponseMapperInterface $paypalResponseMapper
    ) {
        $this->paypalResponseMapper = $paypalResponseMapper;
    }

    /**
     * @param array $payload
     *
     * @return PaypalExpressSuccessResponseTransfer
     */
    public function successResponse(array $payload): PaypalExpressSuccessResponseTransfer
    {
        $responseTransfer = $this->paypalResponseMapper->mapSuccessResponse($payload);

The assignment to $responseTransfer is dead and can be removed.

In this branch, the function will implicitly return null which is incompatible with the type-hinted return Generated\Shared\Transfer\PaypalExpressSuccessResponseTransfer. Consider adding a return statement or allowing null as return value.

    }
}