Bpost\BpostApiClient\Bpost\ProductConfiguration\DeliveryMethod

Complexity

Total Complexity

13

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	createFromXML()	0	23	6
A	getVisibility()	0	3	1
A	setName()	0	3	1
A	getName()	0	3	1
A	getProducts()	0	3	1
A	isVisibleAndActive()	0	3	1
A	setVisibility()	0	3	1
A	addProduct()	0	3	1

<?php
declare(strict_types=1);

namespace Bpost\BpostApiClient\Bpost\ProductConfiguration;

use SimpleXMLElement;

/**
 * Class DeliveryMethod
 */
class DeliveryMethod
{
    public const DELIVERY_METHOD_NAME_HOME_OR_OFFICE  = 'home or office';
    public const DELIVERY_METHOD_NAME_PICKUP_POINT    = 'pick-up point';
    public const DELIVERY_METHOD_NAME_PARCEL_LOCKER   = 'parcel locker';
    public const DELIVERY_METHOD_NAME_CLICK_AND_COLLECT = 'Click & Collect';

    private string $name;
    private string $visibility;
    /** @var Product[] */
    private array $products = [];

    public static function createFromXML(SimpleXMLElement $xml): self
    {
        $attributes = $xml->attributes();
        $children   = $xml->children();

        $instance = new self();
        if (isset($attributes['name'])) {
            $instance->setName((string) $attributes['name']);
        }
        // Correction de l'attribut: visibility (et fallback si l’API renvoie l’ancienne coquille)
        if (isset($attributes['visibility'])) {
            $instance->setVisibility((string) $attributes['visibility']);
        } elseif (isset($attributes['visiblity'])) {
            $instance->setVisibility((string) $attributes['visiblity']);
        }

        if (isset($children->product)) {
            foreach ($children->product as $productXml) {
                $instance->addProduct(Product::createFromXML($productXml));

It seems like $productXml can also be of type null; however, parameter $xml of Bpost\BpostApiClient\Bpost\ProductConfiguration\Product::createFromXML() does only seem to accept SimpleXMLElement, maybe add an additional type check?

            }
        }

        return $instance;
    }

    public function getName(): string
    {
        return $this->name;
    }
    public function setName(string $name): void
    {
        $this->name = $name;
    }

    public function getVisibility(): string
    {
        return $this->visibility;
    }

    public function setVisibility(string $visibility): void
    {
        $this->visibility = $visibility;
    }

    public function isVisibleAndActive(): bool
    {
        return $this->getVisibility() === Visibility::DELIVERY_METHOD_VISIBILITY_VISIBLE;
    }

    public function getProducts(): array
    {
        return $this->products;
    }
    public function addProduct(Product $product): void
    {
        $this->products[] = $product;
    }
}