Debril\RssAtomBundle\Request\ModifiedSince

Complexity

Total Complexity

4

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getValue()	0	4	1
A	getModifiedSince()	0	9	2

<?php declare(strict_types=1);

namespace Debril\RssAtomBundle\Request;


use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\RequestStack;

class ModifiedSince
{

    const HTTP_HEADER_NAME = 'If-Modified-Since';

    private $value;

    public function __construct(RequestStack $requestStack)
    {
        $this->value = $this->getModifiedSince($requestStack->getCurrentRequest());

It seems like $requestStack->getCurrentRequest() can be null; however, getModifiedSince() does not accept null, maybe add an additional type check?

    }

    /**
     * @return \DateTime
     */
    public function getValue(): \DateTime
    {
        return $this->value;
    }

    private function getModifiedSince(Request $request):\DateTime
    {
        if ($request->headers->has(self::HTTP_HEADER_NAME)) {
            $string = $request->headers->get(self::HTTP_HEADER_NAME);
            return \DateTime::createFromFormat(\DateTime::RSS, $string);
        }

        return new \DateTime('@1');
    }
}