Jeancsil\FlightSpy\Api\DataTransfer\Period

Complexity

Total Complexity

8

Size/Duplication

Total Lines	86
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

55.56%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	generateDateCombinations()	0	22	3
A	getDurationInDays()	0	4	1
A	getDateFrom()	0	4	1
A	getDateTo()	0	4	1
A	getOneDayMore()	0	3	1

<?php
/**
 * @author Jean Silva <me@jeancsil.com>
 * @license MIT
 */
namespace Jeancsil\FlightSpy\Api\DataTransfer;

use Doctrine\Common\Collections\ArrayCollection;

class Period
{
    /**
     * @var int
     */
    private $durationInDays;

    /**
     * @var \Datetime
     */
    private $dateFrom;

    /**
     * @var \Datetime
     */
    private $dateTo;

    /**
     * @param int $durationInDays
     * @param \Datetime $dateFrom
     * @param \Datetime $dateTo
     */
    public function __construct($durationInDays, \Datetime $dateFrom, \Datetime $dateTo)
    {
        $this->durationInDays = $durationInDays;
        $this->dateFrom = $dateFrom;

Equals sign not aligned with surrounding assignments; expected 7 spaces but found 1 space

        $this->dateTo = $dateTo;

Equals sign not aligned with surrounding assignments; expected 9 spaces but found 1 space

    }

    /**
     * @return ArrayCollection

Should the return type not be ArrayCollection|null?

     */
    public function generateDateCombinations()
    {
        $combinations = new ArrayCollection();
        $possibleDays = $this->dateFrom->diff($this->dateTo)->days;

        $initialDate = clone $this->dateFrom;
        $endDate = clone $this->dateFrom->add(new \DateInterval(sprintf('P%dD', $this->durationInDays)));

Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space

        for ($i = 0; $i < $possibleDays; $i++) {
            $combinations->add([
                'from' => clone $initialDate,
                'to' => clone $endDate
            ]);

            if ($endDate->diff($this->dateTo)->days == 0) {
                return $combinations;
            }

            $initialDate->add($this->getOneDayMore());
            $endDate->add($this->getOneDayMore());
        }
    }

    /**
     * @return int
     */
    public function getDurationInDays()
    {
        return $this->durationInDays;
    }

    /**
     * @return \Datetime
     */
    public function getDateFrom()
    {
        return $this->dateFrom;
    }

    /**
     * @return \Datetime
     */
    public function getDateTo()
    {
        return $this->dateTo;
    }

    /**
     * @return \DateInterval
     */
    private function getOneDayMore() {
        return new \DateInterval('P1D');
    }
}