PtrTn\Battlerite\Dto\Status

Complexity

Total Complexity

2

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	12	1
A	createFromArray()	0	14	1

<?php

namespace PtrTn\Battlerite\Dto;

use DateTime;
use Webmozart\Assert\Assert;

class Status
{
    /**
     * @var string
     */
    public $type;

    /**
     * @var string
     */
    public $id;

    /**
     * @var DateTime
     */
    public $releasedAt;

    /**
     * @var string
     */
    public $version;

    public function __construct(
        string $type,
        string $id,
        DateTime $releasedAt,
        string $version
    ) {
        $this->type = $type;
        $this->type = $type;
        $this->id = $id;
        $this->releasedAt = $releasedAt;
        $this->version = $version;
    }

    public static function createFromArray($status): self
    {
        Assert::string($status['type']);
        Assert::string($status['id']);
        Assert::string($status['attributes']['releasedAt']);
        Assert::string($status['attributes']['version']);

        return new self(
            $status['type'],
            $status['id'],
            DateTime::createFromFormat(DateTime::ISO8601, $status['attributes']['releasedAt']),

It seems like \DateTime::createFromFormat(\DateTime::ISO8601, $status['attributes']['releasedAt']) targeting DateTime::createFromFormat() can also be of type false; however, PtrTn\Battlerite\Dto\Status::__construct() does only seem to accept object<DateTime>, did you maybe forget to handle an error condition?

            $status['attributes']['version']
        );
    }
}