SprykerEco\Yves\Braintree\Controller\PaypalExpressController

Complexity

Total Complexity

1

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	successAction()	0	9	1

<?php

/**
 * MIT License
 * For full license information, please view the LICENSE file that was distributed with this source code.
 */

namespace SprykerEco\Yves\Braintree\Controller;

use Spryker\Yves\Kernel\Controller\AbstractController;
use Symfony\Component\HttpFoundation\Request;

/**
 * @method \SprykerEco\Yves\Braintree\BraintreeFactory getFactory()
 */
class PaypalExpressController extends AbstractController
{
    /**
     * @param \Symfony\Component\HttpFoundation\Request $request
     *
     * @return \Symfony\Component\HttpFoundation\JsonResponse
     */
    public function successAction(Request $request)
    {
        $payload = $this->getFactory()->getUtilEncodingService()->decodeJson($request->getContent(), true);

        $this->getFactory()->createResponseProcessor()->processSuccessResponse($payload);

It seems like $payload can also be of type null; however, parameter $payload of SprykerEco\Yves\Braintree\Model\Processor\PaypalResponseProcessor::processSuccessResponse() does only seem to accept array, maybe add an additional type check?

        //TODO: Update route
        return $this->jsonResponse([
            'redirectUrl' => 'http://www.de.suite-nonsplit.local/checkout/summary',
        ]);
    }
}