Ups\Entity\EstimatedArrival

Complexity

Total Complexity

17

Size/Duplication

Total Lines	123
Duplicated Lines	17.89 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

44.44%

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
C	__construct()	22	22	7
A	getTime()	0	4	1
A	setTime()	0	4	1
A	getPickupDate()	0	4	1
A	setPickupDate()	0	4	1
A	getPickupTime()	0	4	1
A	setPickupTime()	0	4	1
A	getDelayCount()	0	4	1
A	setDelayCount()	0	4	1
A	getDate()	0	4	1
A	setDate()	0	4	1

<?php

namespace Ups\Entity;

class EstimatedArrival extends Rate\EstimatedArrival
{
    const EA_MONDAY = 'MON';
    const EA_TUESDAY = 'TUE';
    const EA_WEDNESDAY = 'WEB';
    const EA_THUSDAY = 'THU';
    const EA_FRIDAY = 'FRI';
    const EA_SATURDAY = 'SAT';
    // Sunday is an invalid day :-)

    private $Time;
    private $PickupDate;
    private $PickupTime;
    private $DelayCount;

Consider using a different property name as you override a private property of the parent class.

    private $Date;

    /**
     * @param \stdClass|null $response
     */
    public function __construct(\stdClass $response = null)

This method seems to be duplicated in your project.

    {
        parent::__construct($response);

        if (null !== $response) {
            if (isset($response->Time)) {
                $this->Time = $response->Time;
            }
            if (isset($response->PickupDate)) {
                $this->PickupDate = $response->PickupDate;
            }
            if (isset($response->PickupTime)) {
                $this->PickupTime = $response->PickupTime;
            }
            if (isset($response->DelayCount)) {
                $this->DelayCount = $response->DelayCount;
            }
            if (isset($response->Date)) {
                $this->Date = $response->Date;
            }
        }
    }

    /**
     * @return mixed
     */
    public function getTime()
    {
        return $this->Time;
    }

    /**
     * @param mixed $Time
     */
    public function setTime($Time)
    {
        $this->Time = $Time;
    }

    /**
     * @return mixed
     */
    public function getPickupDate()
    {
        return $this->PickupDate;
    }

    /**
     * @param mixed $PickupDate
     */
    public function setPickupDate($PickupDate)
    {
        $this->PickupDate = $PickupDate;
    }

    /**
     * @return mixed
     */
    public function getPickupTime()
    {
        return $this->PickupTime;
    }

    /**
     * @param mixed $PickupTime
     */
    public function setPickupTime($PickupTime)
    {
        $this->PickupTime = $PickupTime;
    }

    /**
     * @return mixed
     */
    public function getDelayCount()
    {
        return $this->DelayCount;
    }

    /**
     * @param mixed $DelayCount
     */
    public function setDelayCount($DelayCount)
    {
        $this->DelayCount = $DelayCount;
    }

    /**
     * @return mixed
     */
    public function getDate()
    {
        return $this->Date;
    }

    /**
     * @param mixed $Date
     */
    public function setDate($Date)
    {
        $this->Date = $Date;
    }

}