Recurrence\Provider\OptimizedProvider::provide()

Complexity

Conditions	4
Paths	8

Size

Total Lines	25
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	4

Importance

Changes	1
Bugs	0	Features	0

<?php
    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;
    }