Scheduler\Job\RRule

Complexity

Total Complexity

4

Size/Duplication

Total Lines	45
Duplicated Lines	60 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getRecurrences()	13	13	2
A	getNextRecurrence()	14	14	2

<?php

namespace Scheduler\Job;

use DateTimeInterface;
use Recurr\Rule as RecurrRule;
use Recurr\Recurrence;
use Recurr\Transformer\ArrayTransformer;
use Recurr\Transformer\Constraint\AfterConstraint;
use Recurr\Transformer\Constraint\BetweenConstraint;

/**
 * Class RRule
 * @package Scheduler\Job
 * @author Aleh Hutnikau, <goodnickoff@gmail.com>
 */
class RRule extends AbstractRule
{

    /**
     * @param DateTimeInterface $from
     * @param DateTimeInterface $to
     * @param boolean $inc
     * @throws
     * @return DateTimeInterface[]
     */
    public function getRecurrences(DateTimeInterface $from, DateTimeInterface $to, $inc = true)

This method seems to be duplicated in your project.

    {
        $rRule = new RecurrRule($this->getRrule(), $this->getStartDate());
        $rRuleTransformer = new ArrayTransformer();
        $constraint = new BetweenConstraint($from, $to, $inc);
        $recurrenceCollection = $rRuleTransformer->transform($rRule, $constraint);
        $result = [];
        /** @var Recurrence $recurrence */
        foreach ($recurrenceCollection as $recurrence) {
            $result[] = $recurrence->getStart();
        }
        return $result;
    }

    /**
     * @param DateTimeInterface $from
     * @param boolean $inc including $from and $to dates
     * @return DateTimeInterface|null date of the next recurrence or null of no more recurrences scheduled.
     * @throws
     */
    public function getNextRecurrence(DateTimeInterface $from, $inc = true)

This method seems to be duplicated in your project.

    {
        $rRule = new RecurrRule($this->getRrule(), $this->getStartDate());
        $rRuleTransformer = new ArrayTransformer();
        $constraint = new AfterConstraint($from, $inc);
        $recurrenceCollection = $rRuleTransformer->transform($rRule, $constraint);
        $result = null;
        /** @var Recurrence $recurrence */
        foreach ($recurrenceCollection as $recurrence) {
            $result = $recurrence->getStart();
            break;
        }
        return $result;
    }
}