Graze\CiffRenderer\Parser\FieldParser\AbstractFieldParserDate

Complexity

Total Complexity

5

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

87.5%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getText()	0	10	2
	getDateTime()	0	1	?
A	getDateFormat()	0	4	1
A	factory()	0	7	1

<?php

namespace Graze\CiffRenderer\Parser\FieldParser;

use Graze\CiffRenderer\Parser\FieldParser\FieldParserFixedText;
use Graze\CiffRenderer\Parser\FieldParser\FieldParserInterface;
use Graze\CiffRenderer\Parser\FieldParser\DateFormatter\DateFormatterFactory;
use Graze\CiffRenderer\Exception\UnsupportedDateFormatException;

abstract class AbstractFieldParserDate extends FieldParserFixedText implements FieldParserInterface
{
    /**
     * @var \DateTimeImmutable
     */
    protected $dateTimeNow;

    /**
     * @var DateFormatterFactory
     */
    private $dateFormatterFactory;

    /**
     * @param \DateTimeImmutable $dateTimeNow
     * @param DateFormatterFactory $dateFormatterFactory
     */
    public function __construct(\DateTimeImmutable $dateTimeNow, DateFormatterFactory $dateFormatterFactory)
    {
        $this->dateTimeNow = $dateTimeNow;
        $this->dateFormatterFactory = $dateFormatterFactory;
    }

    /**
     * @return string
     * @throws UnsupportedDateFormatException
     */
    public function getText()
    {
        try {
            $dateFormat = $this->getDateFormat();
            $dateFormatter = $this->dateFormatterFactory->getFormatter($dateFormat);
            return $dateFormatter->format($this->getDateTime(), $dateFormat);
        } catch (\Exception $e) {
            throw new UnsupportedDateFormatException($this->getDateFormat());
        }
    }

    /**
     * Fetch a DateTime object set to the correct date/time.
     *
     * @return \DateTimeInterface
     */
    abstract protected function getDateTime();

    /**
     * @return string
     */
    public function getDateFormat()
    {
        return (string) $this->xmlField->Data->Object->Default;

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

    }

    /**
     * @return FieldParserInterface
     */
    public static function factory()
    {
        return new static(
            new \DateTimeImmutable(),
            new DateFormatterFactory()
        );
    }
}