CultureKings\Afterpay\Service\InStore\Order

Complexity

Total Complexity

12

Size/Duplication

Total Lines	133
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	9

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	23	2
A	reverse()	0	26	2
B	createOrReverse()	0	47	8

<?php
namespace CultureKings\Afterpay\Service\InStore;

use CultureKings\Afterpay\Exception\ApiException;
use CultureKings\Afterpay\Model;
use DateTime;
use GuzzleHttp\Exception\BadResponseException;
use GuzzleHttp\Exception\RequestException;
use GuzzleHttp\HandlerStack;

/**
 * Class Order
 * @package CultureKings\Afterpay\Service\InStore
 */
class Order extends AbstractService
{
    const ERROR_DECLINED = 402;
    const ERROR_MINIMUM_NOT_MET = 402;
    const ERROR_EXCEED_PREAPPROVAL = 402;
    const ERROR_CONFLICT = 409;
    const ERROR_INVALID_CODE = 412;
    const ERROR_INTERNAL_ERROR = 500;
    const ERROR_MSG_PRECONDITION_FAILED = 'precondition_failed';

    /**
     * @param Model\InStore\Order $order
     * @param HandlerStack|null   $stack
     *
     * @return array|\JMS\Serializer\scalar|object
     */
    public function create(Model\InStore\Order $order, HandlerStack $stack = null)
    {
        try {
            $params = $this->generateParams($order, $stack);

            $result = $this->getClient()->post('orders', $params);

            return $this->getSerializer()->deserialize(
                (string) $result->getBody(),
                Model\InStore\Order::class,
                'json'
            );
        } catch (BadResponseException $e) {
            throw new ApiException(
                $this->getSerializer()->deserialize(
                    (string) $e->getResponse()->getBody(),
                    Model\ErrorResponse::class,
                    'json'
                ),
                $e
            );
        }
    }

    /**
     * @param Model\InStore\Reversal $orderReversal
     * @param HandlerStack|null      $stack
     * @throws ApiException
     *
     * @return Model\InStore\Reversal
     */
    public function reverse(Model\InStore\Reversal $orderReversal, HandlerStack $stack = null)
    {
        try {
            $params = $this->generateParams($orderReversal, $stack);

            $result = $this->getClient()->post('orders/reverse', $params);

            /** @var Model\InStore\Reversal $reversal */
            $reversal = $this->getSerializer()->deserialize(
                (string) $result->getBody(),
                Model\InStore\Reversal::class,
                'json'
            );

            return $reversal;
        } catch (BadResponseException $e) {
            throw new ApiException(
                $this->getSerializer()->deserialize(
                    (string) $e->getResponse()->getBody(),
                    Model\ErrorResponse::class,
                    'json'
                ),
                $e
            );
        }
    }

    /**
     * Helper method to automatically attempt to reverse an order if an error occurs.
     *
     * Order reversal model does not have to be passed in and will be automatically generated if not.
     *
     * @param Model\InStore\Order         $order
     * @param Model\InStore\Reversal|null $orderReversal
     * @param HandlerStack|null           $stack
     *
     * @return Model\InStore\Order|Model\InStore\Reversal|array|\JMS\Serializer\scalar|object
     */
    public function createOrReverse(
        Model\InStore\Order $order,
        Model\InStore\Reversal $orderReversal = null,
        HandlerStack $stack = null
    ) {
        try {
            return $this->create($order, $stack);
        } catch (ApiException $orderException) {
            // http://docs.afterpay.com.au/instore-api-v1.html#create-order
            // Should a success or error response (with exception to 409 conflict) not be received,
            // the POS should queue the request ID for reversal
            if ($orderException->getErrorResponse()->getErrorCode() === self::ERROR_CONFLICT) {

The strict comparison === seems to always evaluate to false as the types of $orderException->getErrorResponse()->getErrorCode() (string) and self::ERROR_CONFLICT (integer) can never be identical. Maybe you want to use a loose comparison == instead?

                throw $orderException;
            }
            $errorResponse = $orderException->getErrorResponse();
        } catch (RequestException $orderException) {
            // a timeout or other exception has occurred. attempt a reversal
            $errorResponse = new Model\ErrorResponse();

            $errorResponse->setHttpStatusCode(
                $orderException->getResponse() ? $orderException->getResponse()->getStatusCode() : self::ERROR_INTERNAL_ERROR
            );

            $errorResponse->setMessage($orderException->getMessage());
        }

        if ($orderReversal === null) {
            $orderReversal = new Model\InStore\Reversal();

Consider using a different name than the parameter $orderReversal. This often makes code more readable.

            $orderReversal->setReversingRequestId($order->getRequestId());
            $orderReversal->setRequestedAt(new DateTime());
        }

        try {
            $reversal = $this->reverse($orderReversal, $stack);
            $reversal->setErrorReason($errorResponse);

            return $reversal;
        } catch (ApiException $reversalException) {
            $reversalErrorResponse = $reversalException->getErrorResponse();
            if ($reversalErrorResponse->getErrorCode() === self::ERROR_MSG_PRECONDITION_FAILED) {
                // there was trouble reversing probably because the order failed
                throw $orderException;
            }

            throw $reversalException;
        }
    }
}