CultuurNet\UDB3\Calendar\DayOfWeekCollection

Complexity

Total Complexity

7

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	2
A	addDayOfWeek()	0	13	1
A	getDaysOfWeek()	0	9	1
A	deserialize()	0	12	2
A	serialize()	0	4	1

<?php

namespace CultuurNet\UDB3\Calendar;

use Broadway\Serializer\SerializableInterface;

class DayOfWeekCollection implements SerializableInterface
{
    /**
     * @var string[]
     */
    private $daysOfWeek = [];

    /**
     * DayOfWeekCollection constructor.
     * @param DayOfWeek|null $dayOfWeek
     */
    public function __construct(DayOfWeek $dayOfWeek = null)
    {
        if ($dayOfWeek !== null) {
            $this->addDayOfWeek($dayOfWeek);
        }
    }

    /**
     * Keeps the collection of days of week unique.
     * Makes sure that the objects are stored as strings to allow PHP serialize method.
     *
     * @param DayOfWeek $dayOfWeek
     * @return DayOfWeekCollection
     */
    public function addDayOfWeek(DayOfWeek $dayOfWeek)
    {
        $this->daysOfWeek = array_unique(

It seems like array_unique(array_merge($this->daysOfWeek, array($dayOfWeek->toNative()))) of type array<integer,string|null|boolean|integer|double,{"0":"null|boolean|integer|double|string"}> is incompatible with the declared type array<integer,string> of property $daysOfWeek.

            array_merge(
                $this->daysOfWeek,
                [
                    $dayOfWeek->toNative(),
                ]
            )
        );

        return $this;
    }

    /**
     * @return DayOfWeek[]
     */
    public function getDaysOfWeek()
    {
        return array_map(
            function ($dayOfWeek) {
                return DayOfWeek::fromNative($dayOfWeek);
            },
            $this->daysOfWeek
        );
    }

    /**
     * @inheritdoc
     */
    public static function deserialize(array $data)
    {
        $dayOfWeekCollection = new DayOfWeekCollection();

        foreach ($data as $dayOfWeek) {
            $dayOfWeekCollection->addDayOfWeek(
                DayOfWeek::fromNative($dayOfWeek)
            );
        }

        return $dayOfWeekCollection;
    }

    /**
     * @inheritdoc
     */
    public function serialize()
    {
        return $this->daysOfWeek;
    }
}