Spiral\Migrations\Migration\State

Complexity

Total Complexity

6

Size/Duplication

Total Lines	100
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
A	getName()	0	4	1
A	getStatus()	0	4	1
A	getTimeCreated()	0	4	1
A	getTimeExecuted()	0	4	1
A	withStatus()	0	8	1

<?php
/**
 * Spiral Framework.
 *
 * @license   MIT
 * @author    Anton Titov (Wolfy-J)
 */
namespace Spiral\Migrations\Migration;

/**
 * Migration meta information specific to current environment
 */
final class State
{
    /**
     * Migration statues.
     */
    const STATUS_UNDEFINED = -1;
    const STATUS_PENDING   = 0;
    const STATUS_EXECUTED  = 1;

    /**
     * @var string
     */
    private $name = '';

    /**
     * @var bool
     */
    private $status = self::STATUS_PENDING;

    /**
     * @var \DateTime
     */
    private $timeCreated = null;

    /**
     * @var \DateTime|null
     */
    private $timeExecuted = null;

    /**
     * @param string    $name
     * @param \DateTime $timeCreated
     * @param int       $status
     * @param \DateTime $timeExecuted
     */
    public function __construct(
        string $name,
        \DateTime $timeCreated,
        int $status = self::STATUS_UNDEFINED,
        \DateTime $timeExecuted = null
    ) {
        $this->name = $name;
        $this->status = $status;

The property $status was declared of type boolean, but $status is of type integer. Maybe add a type cast?

        $this->timeCreated = $timeCreated;
        $this->timeExecuted = $timeExecuted;
    }

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

    /**
     * Migration status.
     *
     * @return int
     */
    public function getStatus(): int
    {
        return $this->status;
    }

    /**
     * Get migration creation time.
     *
     * @return \DateTime
     */
    public function getTimeCreated(): \DateTime
    {
        return $this->timeCreated;
    }

    /**
     * Get migration execution time if any.
     *
     * @return \DateTime|null
     */
    public function getTimeExecuted()
    {
        return $this->timeExecuted;
    }

    /**
     * @param int            $status
     * @param \DateTime|null $timeExecuted
     *
     * @return State
     */
    public function withStatus(int $status, \DateTime $timeExecuted = null): State
    {
        $state = clone $this;
        $state->status = $status;

The property $status was declared of type boolean, but $status is of type integer. Maybe add a type cast?

        $state->timeExecuted = $timeExecuted;

        return $state;
    }
}