Magneds\Lokalise\Project\Entity\Project

Complexity

Total Complexity

7

Size/Duplication

Total Lines	93
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	buildFromArray()	0	9	1
A	__construct()	0	7	1
A	getID()	0	3	1
A	getName()	0	3	1
A	getOwner()	0	3	1
A	getCreated()	0	3	1
A	getDescription()	0	3	1

<?php // Copyright ⓒ 2018 Magneds IP B.V. - All Rights Reserved
namespace Magneds\Lokalise\Project\Entity;

use DateTime;
use DateTimeZone;

class Project
{
    /**
     * @var string
     */
    protected $id;

    /**
     * @var string
     */
    protected $name;

    /**
     * @var string
     */
    protected $description;

    /**
     * @var DateTime
     */
    protected $created;

    /**
     * @var int
     */
    protected $owner;

    /**
     * Project constructor.
     * @param string $id
     * @param string $name
     * @param string $description
     * @param DateTime $created
     * @param int $owner
     */
    public function __construct(ProjectID $id, string $name, string $description, DateTime $created, int $owner)
    {
        $this->id          = $id;
        $this->name        = $name;
        $this->description = $description;
        $this->created     = $created;
        $this->owner       = $owner;
    }

    /**
     * @return ProjectID
     */
    public function getID(): ProjectID
    {
        return $this->id;

The expression return $this->id returns the type string which is incompatible with the type-hinted return Magneds\Lokalise\Project\Entity\ProjectID.

    }

    /**
     * @return string
     */
    public function getName(): string
    {
        return $this->name;
    }

    /**
     * @return string
     */
    public function getDescription(): string
    {
        return $this->description;
    }

    /**
     * @return DateTime
     */
    public function getCreated(): DateTime
    {
        return $this->created;
    }

    /**
     * @return int
     */
    public function getOwner(): int
    {
        return $this->owner;
    }

    public static function buildFromArray($array)
    {
        $timezone = new DateTimeZone('Europe/Amsterdam');
        $utc      = new DateTimeZone('UTC');

        $created = DateTime::createFromFormat('Y-m-d H:i:s', $array['created'], $timezone);
        $created->setTimezone($utc);

        return new Project(new ProjectID($array['id']), $array['name'], $array['desc'], $created, $array['owner']);

It seems like $created can also be of type false; however, parameter $created of Magneds\Lokalise\Project\Entity\Project::__construct() does only seem to accept DateTime, maybe add an additional type check?

    }
}