Loevgaard\AltaPay\Payload\RefundCapturedReservation

Complexity

Total Complexity

14

Size/Duplication

Total Lines	161
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

100%

Importance

Changes

0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getPayload()	0	15	2
A	validate()	0	9	1
A	getTransactionId()	0	4	1
A	setTransactionId()	0	5	1
A	getAmount()	0	4	1
A	setAmount()	0	7	1
A	getReconciliationIdentifier()	0	4	1
A	setReconciliationIdentifier()	0	5	1
A	isAllowOverRefund()	0	4	1
A	setAllowOverRefund()	0	5	1
A	getInvoiceNumber()	0	4	1
A	setInvoiceNumber()	0	5	1

<?php
namespace Loevgaard\AltaPay\Payload;

use Assert\Assert;
use Loevgaard\AltaPay;
use Money\Money;

class RefundCapturedReservation extends Payload implements RefundCapturedReservationInterface
{
    use OrderLineArrayTrait;

    /**
     * @var string
     */
    private $transactionId;

    /**
     * @var int
     */
    private $amountValue;

    /**
     * @var string
     */
    private $amountCurrency;

    /**
     * @var string
     */
    private $reconciliationIdentifier;

    /**
     * @var boolean
     */
    private $allowOverRefund;

    /**
     * @var string
     */
    private $invoiceNumber;

    public function __construct(string $transactionId)
    {
        $this->transactionId = $transactionId;
        $this->orderLines = [];
    }

    /**
     * @return array
     */
    public function getPayload() : array
    {
        $this->validate();

        $payload = [
            'transaction_id' => $this->transactionId,
            'amount' => AltaPay\floatFromMoney($this->getAmount()),
            'reconciliation_identifier' => $this->reconciliationIdentifier,
            'allow_over_refund' => is_bool($this->allowOverRefund) ? intval($this->allowOverRefund) : null,
            'invoice_number' => $this->invoiceNumber,
            'orderLines' => $this->orderLines,
        ];

        return static::simplePayload($payload);
    }

    public function validate()
    {
        Assert::that($this->transactionId)->string();
        Assert::thatNullOr($this->getAmount())->isInstanceOf(Money::class);
        Assert::thatNullOr($this->reconciliationIdentifier)->string();
        Assert::thatNullOr($this->allowOverRefund)->boolean();
        Assert::thatNullOr($this->invoiceNumber)->string();
        Assert::thatNullOr($this->orderLines)->isArray();
    }

    /**
     * @return string
     */
    public function getTransactionId(): string
    {
        return $this->transactionId;
    }

    /**
     * @param string $transactionId
     * @return RefundCapturedReservation
     */
    public function setTransactionId(string $transactionId) : self
    {
        $this->transactionId = $transactionId;
        return $this;
    }

    /**
     * @return Money
     */
    public function getAmount(): ?Money
    {
        return AltaPay\createMoney((string)$this->amountCurrency, (int)$this->amountValue);
    }

    /**
     * @param Money $amount
     * @return RefundCapturedReservation
     */
    public function setAmount(Money $amount) : self
    {
        $this->amountCurrency = $amount->getCurrency()->getCode();
        $this->amountValue = $amount->getAmount();

The property $amountValue was declared of type integer, but $amount->getAmount() is of type string. Maybe add a type cast?

        return $this;
    }

    /**
     * @return string
     */
    public function getReconciliationIdentifier(): ?string
    {
        return $this->reconciliationIdentifier;
    }

    /**
     * @param string $reconciliationIdentifier
     * @return RefundCapturedReservation
     */
    public function setReconciliationIdentifier(string $reconciliationIdentifier) : self
    {
        $this->reconciliationIdentifier = $reconciliationIdentifier;
        return $this;
    }

    /**
     * @return bool
     */
    public function isAllowOverRefund(): ?bool
    {
        return $this->allowOverRefund;
    }

    /**
     * @param bool $allowOverRefund
     * @return RefundCapturedReservation
     */
    public function setAllowOverRefund(bool $allowOverRefund) : self
    {
        $this->allowOverRefund = $allowOverRefund;
        return $this;
    }

    /**
     * @return string
     */
    public function getInvoiceNumber(): ?string
    {
        return $this->invoiceNumber;
    }

    /**
     * @param string $invoiceNumber
     * @return RefundCapturedReservation
     */
    public function setInvoiceNumber(string $invoiceNumber) : self
    {
        $this->invoiceNumber = $invoiceNumber;
        return $this;
    }
}