Recurrence\RruleTransformer\DtStartTransformer

Complexity

Total Complexity

7

Size/Duplication

Total Lines	37
Duplicated Lines	24.32 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
C	transform()	9	28	7

<?php

namespace Recurrence\RruleTransformer;

class DtStartTransformer implements TransformerInterface
{
    const RRULE_PARAMETER = 'DTSTART';

    /**
     * @param string $rRule
     * @return \DateTime
     */
    public function transform($rRule)
    {
        if(preg_match('/' . $this::RRULE_PARAMETER . ';TZID=([a-zA-Z_-]+[\/[a-zA-Z_+\-0-9]+]?):([0-9]{8}T[0-9]{6})/', $rRule, $matches)) {
            try {
                return \DateTime::createFromFormat('Ymd\THis', $matches[2], new \DateTimeZone($matches[1]));

The expression \DateTime::createFromFormat('Ymd\\THis', $matches[2], new \DateTimeZone($matches[1])); of type DateTime|false adds false to the return on line 17 which is incompatible with the return type documented by Recurrence\RruleTransformer\DtStartTransformer::transform of type DateTime|null. It seems like you forgot to handle an error condition.

            } catch (\Exception $e) {
                throw new \InvalidArgumentException(sprintf('Invalid RRULE [%s] option : [%s] with timezone [%s]', $this::RRULE_PARAMETER, $matches[2], $matches[1]));
            }
        }

        if(preg_match('/' . $this::RRULE_PARAMETER . '=([0-9]{8}T[0-9]{6}Z)/', $rRule, $matches)) {

This code seems to be duplicated across your project.

            return \DateTime::createFromFormat('Ymd\THisZ', $matches[1], new \DateTimeZone('UTC'));

The expression \DateTime::createFromFormat('Ymd\\THisZ', $matches[1], new \DateTimeZone('UTC')); of type DateTime|false adds false to the return on line 24 which is incompatible with the return type documented by Recurrence\RruleTransformer\DtStartTransformer::transform of type DateTime|null. It seems like you forgot to handle an error condition.

        }

        if(preg_match('/' . $this::RRULE_PARAMETER . '=([0-9]{8}T[0-9]{6})/', $rRule, $matches)) {

This code seems to be duplicated across your project.

            return \DateTime::createFromFormat('Ymd\THis', $matches[1]);

The expression \DateTime::createFromFormat('Ymd\\THis', $matches[1]); of type DateTime|false adds false to the return on line 28 which is incompatible with the return type documented by Recurrence\RruleTransformer\DtStartTransformer::transform of type DateTime|null. It seems like you forgot to handle an error condition.

        }

        if(preg_match('/' . $this::RRULE_PARAMETER . '=([0-9]{8})/', $rRule, $matches)) {

This code seems to be duplicated across your project.

            return \DateTime::createFromFormat('Ymd', $matches[1]);

The expression \DateTime::createFromFormat('Ymd', $matches[1]); of type DateTime|false adds false to the return on line 32 which is incompatible with the return type documented by Recurrence\RruleTransformer\DtStartTransformer::transform of type DateTime|null. It seems like you forgot to handle an error condition.

        }

        if(preg_match('/' . $this::RRULE_PARAMETER . '/', $rRule, $matches)) {
            throw new \InvalidArgumentException(sprintf('Invalid RRULE [%s]', $this::RRULE_PARAMETER));
        }

        return null;
    }
}