smtech\CanvasICSSync\SyncIntoCanvas\Schedule

Complexity

Total Complexity

12

Size/Duplication

Total Lines	122
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	3
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	3
A	getId()	0	4	1
A	getCalendar()	0	4	1
A	getSchedule()	0	4	1
A	setSchedule()	0	4	1
B	save()	0	41	2
A	delete()	0	9	1
A	load()	0	12	2

<?php

namespace smtech\CanvasICSSync\SyncIntoCanvas;

class Schedule extends Syncable
{
    const ONCE = 'once';
    const HOURLY = 'hourly';
    const DAILY = 'daily';
    const WEEKLY = 'weekly';

    /**
     * Unique identifier for this schedule
     * @var int
     */
    protected $id;

    /**
     * Calendar associated with this schedule
     * @var Calendar
     */
    protected $calendar;

    /**
     * Schedule identifier
     * @var string
     */
    protected $schedule;

    public function __construct(Calendar $calendar, $schedule, $id = false)
    {
        if (empty($calendar)) {
            throw new Exception("Valid Calendar required");
        }
        $this->calendar = $calendar;
        $this->schedule = (string) $schedule;
        if ($id !== false) {
            $this->id = intval($id);
        }
    }

    public function getId()
    {
        return $this->id;
    }

    public function getCalendar()
    {
        return $this->calendar;
    }

    public function getSchedule()
    {
        return $this->schedule;
    }

    public function setSchedule($schedule)
    {
        $this->schedule = (string) $schedule;
    }

    public function save()
    {
        $db = Syncable::getDatabase();
        $select = $db->prepare(
            "SELECT * FROM `schedules` WHERE `id` = :id"
        );
        $update = $db->prepare(
            "UPDATE `schedules`
                SET
                    `synced` = :synced,
                    `schedule` = :schedule
                WHERE
                    `id` = :id"
        );
        $insert = $db->prepare(
            "INSERT INTO `schedules`
                (
                    `calendar`,
                    `schedule`,
                    `synced`
                ) VALUES (
                    :calendar,
                    :schedule,
                    :synced
                )"
        );

        $params = [
            'id' => $this->getId(),
            'calendar' => $this->getCalendar()->getId(),
            'schedule' => $this->getSchedule(),
            'synced' => Syncable::getTimestamp()
        ];

        $select->execute($params);
        if (($select->fetch()) === false) {
            $insert->execute($params);
        } else {
            $update->execute($params);
        }
    }

    public function delete()
    {
        $delete = Syncable::getDatabase()->prepare(
            "DELETE FROM `schedules`
                WHERE `id` = :id"
        );

        $delete->execute($this->getId());
    }

    public static function load($id)
    {
        $select = Syncable::getDatabase()->prepare(
            "SELECT * FROM `schedules` WHERE `id` = :id"
        );

        $select->execute($id);
        if (($_schedule = $select->fetch()) !== false) {
            return new Schedule(Calendar::load($_schedule['calendar']), $_schedule['schedule'], $_schedule['id']);

It seems like \smtech\CanvasICSSync\SyncIntoCanvas\Calendar::load($_schedule['calendar']) can be null; however, __construct() does not accept null, maybe add an additional type check?

        }
        return null;
    }
}