Graze\CiffRenderer\Parser\FieldParser\FieldParserDateOffset

Complexity

Total Complexity

1

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getDateTime()	0	10	1

<?php

namespace Graze\CiffRenderer\Parser\FieldParser;

use Graze\CiffRenderer\Parser\FieldParser\AbstractFieldParserDate;
use Graze\CiffRenderer\Parser\FieldParser\FieldParserInterface;

class FieldParserDateOffset extends AbstractFieldParserDate implements FieldParserInterface
{
    /**
     * @return \DateTimeInterface
     */
    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);
    }
}