Recurrence\Provider\OptimizedProvider

Complexity

Total Complexity

4

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	provide()	0	25	4

<?php

namespace Recurrence\Provider;

use Recurrence\Model\Recurrence;

class OptimizedProvider extends AbstractDatetimeProvider
{
    /**
     * @return array<\DateTime>
     */
    public function provide(Recurrence $recurrence): array
    {
        $interval = $recurrence->getFrequency()->convertToDateIntervalFormat();

        $periodEndAt = ($recurrence->hasPeriodEndAt()) ? $recurrence->getPeriodEndAt() : $this->estimatePeriodEndAt($recurrence);

        // Transform interval in Datetime interval expression
        if ($recurrence->getInterval() !== 1) {
            $interval = str_replace('1', $recurrence->getInterval(), $interval);
        }

        $dateInterval = new \DateInterval($interval);

        $recurrences = iterator_to_array(new \DatePeriod(
            $recurrence->getPeriodStartAt(),
            $dateInterval,
            $periodEndAt

It seems like $periodEndAt can also be of type null; however, parameter $end of DatePeriod::__construct() does only seem to accept DateTimeInterface, maybe add an additional type check?

        ));

        // When having count option, return only amount of recurrences requested
        if ($recurrence->hasCount()) {
            return array_slice($recurrences, 0, $recurrence->getCount());
        }

        return $recurrences;
    }
}