DalliSDK\Requests\CancelOrderRequest

Complexity

Total Complexity

5

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	addBarcode()	0	10	3
A	setBarcodes()	0	4	1
A	getBarcodes()	0	3	1

<?php

declare(strict_types=1);

namespace DalliSDK\Requests;

use DalliSDK\Responses\CancelOrderResponse;
use JMS\Serializer\Annotation as JMS;

/**
 * Запрос на отмену заявки
 *
 * @see https://api.dalli-service.com/v1/doc/cancelOrder
 * @JMS\XmlRoot("cancelorder")
 */
class CancelOrderRequest extends AbstractRequest implements RequestInterface
{
    public const RESPONSE_CLASS = CancelOrderResponse::class;

    /**
     * Контейнер, который содержит штрих-коды заказов.
     *
     * @JMS\XmlList(inline = true, entry = "barcode")
     */
    private ?array $barcodes = null;

    /**
     * @return array|null
     */
    public function getBarcodes(): ?array
    {
        return $this->barcodes;
    }

    /**
     * @param array|null $barcodes
     *
     * @return CancelOrderRequest
     */
    public function setBarcodes(?array $barcodes): CancelOrderRequest
    {
        $this->barcodes = $barcodes;
        return $this;
    }

    /**
     * @param string $barcode
     *
     * @return $this
     */
    public function addBarcode(string $barcode): CancelOrderRequest
    {
        if ($this->barcodes === null) {
            $this->barcodes = [];
        }
        if (in_array($barcode, $this->barcodes)) {

It seems like $this->barcodes can also be of type null; however, parameter $haystack of in_array() does only seem to accept array, maybe add an additional type check?

            throw new \InvalidArgumentException("Barcode: $barcode already exists.");
        }
        $this->barcodes[] = $barcode;
        return $this;
    }
}