BitBag\SyliusMolliePlugin\DTO\PartialRefundItem

Complexity

Total Complexity

14

Size/Duplication

Total Lines	93
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	setAmountRefunded()	0	3	1
A	getAmountTotal()	0	3	1
A	setAmountTotal()	0	3	1
A	getType()	0	3	1
A	getQuantity()	0	3	1
A	setType()	0	3	1
A	getId()	0	3	1
A	setId()	0	3	1
A	setQuantity()	0	3	1
A	getAmountRefunded()	0	3	1
A	getAvailableAmountToRefund()	0	3	1
A	getAmountToRefund()	0	3	1
A	setAmountToRefund()	0	13	2

<?php

/*
 * This file has been created by developers from BitBag.
 * Feel free to contact us once you face any issues or want to start
 * another great project.
 * You can find more information about us on https://bitbag.shop and write us
 * an email on mikolaj.krol@bitbag.pl.
 */

declare(strict_types=1);

namespace BitBag\SyliusMolliePlugin\DTO;

final class PartialRefundItem
{
    /** @var int */
    private $id;

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

    /** @var int */
    private $amountTotal = 0;

    /** @var int */
    private $amountRefunded = 0;

    /** @var int */
    private $quantity = 1;

    /** @var int */
    private $amountToRefund = 0;

    public function getId(): int
    {
        return $this->id;
    }

    public function setId(int $id): void
    {
        $this->id = $id;
    }

    public function getType(): string
    {
        return $this->type;
    }

    public function setType(string $type): void
    {
        $this->type = $type;
    }

    public function getAmountTotal(): int
    {
        return $this->amountTotal;
    }

    public function setAmountTotal(int $amountTotal): void
    {
        $this->amountTotal = $amountTotal;
    }

    public function getAmountRefunded(): int
    {
        return $this->amountRefunded;
    }

    public function setAmountRefunded(int $amountRefunded): void
    {
        $this->amountRefunded += $amountRefunded;
    }

    public function getQuantity(): int
    {
        return $this->quantity;
    }

    public function setQuantity(int $quantity): void
    {
        $this->quantity = $quantity;
    }

    public function getAvailableAmountToRefund(): int
    {
        return $this->getAmountTotal() - $this->getAmountRefunded() - $this->getAmountToRefund();
    }

    public function setAmountToRefund(int $amount): int
    {
        $value = $this->getAvailableAmountToRefund() - $amount;

        if ($value < 0) {
            $this->amountToRefund = $this->getAvailableAmountToRefund();

            return abs($value);

The expression return abs($value) could return the type double which is incompatible with the type-hinted return integer. Consider adding an additional type-check to rule them out.

        }

        $this->amountToRefund = $amount;

        return 0;
    }

    public function getAmountToRefund()
    {
        return $this->amountToRefund;
    }
}