Bpost\BpostApiClient\Bpost\ProductConfiguration

Complexity

Total Complexity

5

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getDeliveryMethods()	0	3	1
A	createFromXML()	0	13	3
A	addDeliveryMethod()	0	3	1

<?php
declare(strict_types=1);

namespace Bpost\BpostApiClient\Bpost;

use Bpost\BpostApiClient\Bpost\ProductConfiguration\DeliveryMethod;
use SimpleXMLElement;

/**
 * Class ProductConfiguration
 */
class ProductConfiguration
{
    private array $deliveryMethods = [];

    public function getDeliveryMethods(): array
    {
        return $this->deliveryMethods;
    }

    public function addDeliveryMethod(DeliveryMethod $deliveryMethod): void
    {
        $this->deliveryMethods[] = $deliveryMethod;
    }

    public static function createFromXML(SimpleXMLElement $xml): self
    {
        $productConfiguration = new self();

        if (isset($xml->deliveryMethod)) {
            foreach ($xml->deliveryMethod as $deliveryMethodXml) {
                $productConfiguration->addDeliveryMethod(
                    DeliveryMethod::createFromXML($deliveryMethodXml)

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

                );
            }
        }

        return $productConfiguration;
    }
}