CultureKings\Afterpay\Service\InStore\Order

Complexity

Total Complexity

9

Size/Duplication

Total Lines	118
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	9

Importance

Changes	2
Bugs	0	Features	0

3 Methods

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

<?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;

    /**
     * @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
    ) {
        $errorResponse = null;

$errorResponse is not used, you could remove the assignment.

        try {
            return $this->create($order, $stack);
        } catch (ApiException $e) {
            // 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 ($e->getErrorResponse()->getErrorCode() == self::ERROR_CONFLICT) {
                throw $e;
            }
            $errorResponse = $e->getErrorResponse();
        } catch (RequestException $e) {
            // a timeout or other exception has occurred. attempt a reversal
            $errorResponse = new Model\ErrorResponse();
            $errorResponse->setHttpStatusCode($e->getResponse()->getStatusCode());
            $errorResponse->setMessage($e->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());
        }

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

        return $reversal;
    }
}