Recurrence\RruleTransformer\DtStartTransformer::transform()

Complexity

Conditions	7
Paths	7

Size

Total Lines	28
Code Lines	15

Duplication

Lines	9
Ratio	32.14 %

Importance

Changes

0

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