Ogone\Subscription\SubscriptionPeriod

Complexity

Total Complexity

22

Size/Duplication

Total Lines	140
Duplicated Lines	11.43 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	2
Bugs	1	Features	1

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getUnit()	0	4	1
B	setUnit()	7	15	6
A	getInterval()	0	4	1
A	setInterval()	0	13	4
A	getMoment()	0	4	1
C	setMoment()	9	24	8

<?php

namespace Ogone\Subscription;

use InvalidArgumentException;

class SubscriptionPeriod
{

    /** @var string */
    const UNIT_DAILY = 'd';

    /** @var string */
    const UNIT_WEEKLY = 'ww';

    /** @var string */
    const UNIT_MONTHLY = 'm';

    /**
     * @var string
     * ‘d’ = daily, ‘ww’ = weekly, ‘m’ = monthly
     */
    protected $unit;

    /**
     * @var int
     * Interval between each occurrence of the subscription payments
     */
    protected $interval;

    /**
     * @var int
     * Depending on sub_period_unit
     * Daily (d):
     *      interval in days
     * Weekly (ww):
     *      1=Sunday, … 7=Saturday
     * Monthly (m):
     *      day of the month
     */
    protected $moment;

    /**
     * @param string $unit
     * @param int $interval
     * @param int $moment
     * @throws InvalidArgumentException
     */
    public function __construct($unit, $interval, $moment)
    {
        $this->setUnit($unit);
        $this->setInterval($interval);
        $this->setMoment($moment);
    }

    /**
     * @return string
     */
    public function getUnit()
    {
        return $this->unit;
    }

    /**
     * @param string $unit
     * @throws InvalidArgumentException
     */
    public function setUnit($unit)
    {
        if (!in_array($unit, array(self::UNIT_DAILY, self::UNIT_WEEKLY, self::UNIT_MONTHLY))) {
            throw new InvalidArgumentException("Subscription period unit should be '".self::UNIT_DAILY."' (daily), '".self::UNIT_WEEKLY."' (weekly) or '".self::UNIT_MONTHLY."' (monthly)");
        }

        if (self::UNIT_WEEKLY === $unit) {

This code seems to be duplicated across your project.

            if ($this->moment > 7) {
                throw new InvalidArgumentException('The unit cannot be set to weekly while the moment > 7');
            }
        } elseif (self::UNIT_MONTHLY === $unit && $this->moment > 28) {
            throw new InvalidArgumentException('The unit cannot be set to monthly while the moment > 28');
        }
        $this->unit = $unit;
    }

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

    /**
     * @param int $interval
     * @throws InvalidArgumentException
     */
    public function setInterval($interval)
    {
        if (!is_int($interval)) {
            throw new InvalidArgumentException("Integer expected for interval");
        }
        if ($interval < 0) {
            throw new InvalidArgumentException("Interval must be a positive number > 0");
        }
        if ($interval >= 1.0E+15) {
            throw new InvalidArgumentException("Interval is too high");
        }
        $this->interval = $interval;
    }

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

    /**
     * @param int $moment
     * @throws InvalidArgumentException
     */
    public function setMoment($moment)
    {
        if (!is_int($moment)) {
            throw new InvalidArgumentException("Integer expected for moment");
        }
        if ($moment <= 0) {
            throw new InvalidArgumentException("Moment must be a positive number");
        }
        if ($moment >= 1.0E+15) {
            throw new InvalidArgumentException("Interval is too high");
        }

        if (self::UNIT_WEEKLY == $this->unit) {

This code seems to be duplicated across your project.

            // Valid values are 1 to 7
            if ($moment > 7) {
                throw new InvalidArgumentException("Moment should be 1 (Sunday), 2, 3 .. 7 (Saturday)");
            }
        } elseif (self::UNIT_MONTHLY == $this->unit && $moment > 28) {
            // We will not allow a day of month > 28
            throw new InvalidArgumentException("Moment can't be larger than 29. Last day for month allowed is 28.");
        }

        $this->moment = $moment;
    }
}