Core23\LastFm\Model\Chart

Complexity

Total Complexity

4

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getFrom()	0	4	1
A	getTo()	0	4	1
A	fromApi()	0	7	1

<?php

declare(strict_types=1);

/*
 * (c) Christian Gripp <mail@core23.de>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Core23\LastFm\Model;

final class Chart
{
    /**
     * @var \DateTime
     */
    private $from;

    /**
     * @var \DateTime
     */
    private $to;

    /**
     * Chart constructor.
     *
     * @param \DateTime $from
     * @param \DateTime $to
     */
    public function __construct(\DateTime $from, \DateTime $to)
    {
        $this->from = $from;
        $this->to   = $to;
    }

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

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

    /**
     * @param array $data
     *
     * @return Chart
     */
    public static function fromApi(array $data): self
    {
        return new self(
            \DateTime::createFromFormat('U', $data['from']),

It seems like \DateTime::createFromFormat('U', $data['from']) targeting DateTime::createFromFormat() can also be of type false; however, Core23\LastFm\Model\Chart::__construct() does only seem to accept object<DateTime>, did you maybe forget to handle an error condition?

            \DateTime::createFromFormat('U', $data['to'])

It seems like \DateTime::createFromFormat('U', $data['to']) targeting DateTime::createFromFormat() can also be of type false; however, Core23\LastFm\Model\Chart::__construct() does only seem to accept object<DateTime>, did you maybe forget to handle an error condition?

        );
    }
}