Graze\CiffRenderer\Parser\FieldParser\FieldParserDateOffset::getDateTime()

Complexity

Conditions	1
Paths	1

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

0

<?php
    protected function getDateTime()
    {
        $offsetFieldName = (string) $this->xmlField->Data->Object->OffsetDate['SrcOffset'];

Accessing Data on the interface Graze\CiffRenderer\SimpleXmlElement\SimpleXmlElementInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        $offsetField = $this->xmlHeader->xpath('//DateOffset[@Name="'. $offsetFieldName .'"]')[0];

The method xpath() does not exist on Graze\CiffRenderer\SimpleXmlElement\SimpleXmlElementInterface. It seems like you code against a sub-type of Graze\CiffRenderer\SimpleXmlElement\SimpleXmlElementInterface such as Graze\CiffRenderer\SimpleXmlElement\SimpleXmlElement.

        // currenty only supports offset days
        $offsetDays = (string) $offsetField->DefaultOffset->Day; // @codingStandardsIgnoreLine
        $dateInterval = new \DateInterval('P' . $offsetDays . 'D');

        return $this->dateTimeNow->add($dateInterval);
    }