TrelloBurndown\Model\Sprint

Complexity

Total Complexity

19

Size/Duplication

Total Lines	137
Duplicated Lines	2.19 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

87.8%

Importance

Changes	2
Bugs	1	Features	1

8 Methods

Rating	Name	Duplication	Size	Complexity
A	getStart()	0	4	1
A	getDuration()	0	4	1
A	setStart()	0	4	1
A	setDuration()	0	4	1
A	getEnd()	0	10	3
A	getSprintDays()	0	18	3
A	getNextDayInSprint()	0	15	3
B	getTotalWorkDays()	3	18	6

<?php

namespace TrelloBurndown\Model;

/**
 * Class Sprint.
 */
class Sprint
{
    /**
     * @var \DateTime
     */
    private $start;

    /**
     * @var \DateInterval
     */
    private $duration;

    /**
     * @const String
     */
    const INTERVAL = 'P1D';

    /**
     * @return \DateTime
     */
    public function getStart()
    {
        return $this->start;
    }

    /**
     * @return \DateInterval
     */
    public function getDuration()
    {
        return $this->duration;
    }

    /**
     * @param \DateTime $start
     */
    public function setStart(\DateTime $start)
    {
        $this->start = $start;
    }

    /**
     * @param \DateInterval $duration
     */
    public function setDuration(\DateInterval $duration)
    {
        $this->duration = $duration;
    }

    /**
     * Calculate the end of sprint from start and duration.
     *
     * @return \DateTime|null
     */
    public function getEnd()
    {
        if ($this->start instanceof \DateTime && $this->duration instanceof \DateInterval) {
            $end = clone $this->start;

            return $end->add($this->duration);
        }

        return null;
    }

    /**
     * Get all days in the sprint from start date
     * and during duration.
     *
     * @return \DatePeriod|null
     */
    public function getSprintDays()
    {
        if ($this->start instanceof \DateTime && $this->duration instanceof \DateInterval) {
            $interval = new \DateInterval(self::INTERVAL);

            $end = $this->getEnd();

            $firstDay = clone $this->start;

            return new \DatePeriod(
                $firstDay->add($interval),
                $interval,
                $end
            );
        }

        return null;
    }

    /**
     * Calculate the next day in the sprint.
     *
     * @return \DateTime
     */
    public function getNextDayInSprint()
    {
        $today = new \DateTime();
        $today->setTime(0, 0, 0);

        if ($today->format('N') == 5) {
            return $today->modify('+3 days');
        }

        if ($today->format('N') == 6) {
            return $today->modify('+2 days');
        }

        return $today->modify('+1 days');
    }

    /**
     * Calculate the total work days in the sprint.
     * This function does not return week-end days but
     * return non-work-days such as christmas.
     *
     * @return int | null

Should the return type not be null|integer?

     */
    public function getTotalWorkDays()
    {
        $days = $this->getSprintDays();
        $total = 0;

        if(!$days instanceof \DatePeriod) {
            return null;
        }

        foreach ($days as $day) {
            if ($day instanceof \DateTime && ($day->format('N') == 6 || $day->format('N') == 7)) {

This code seems to be duplicated across your project.

                continue;
            }
            ++$total;
        }

        return $total;
    }
}