HDNET\Calendarize\Ical\VObjectEventAdapter

Complexity

Total Complexity

32

Size/Duplication

Total Lines	204
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

16 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getEvent()	0	4	1
A	getRawData()	0	4	1
A	getUid()	0	4	1
A	getTitle()	0	7	2
A	getDescription()	0	8	2
A	getLocation()	0	8	2
A	getOrganizer()	0	8	2
A	getStartDate()	0	10	2
A	getEndDate()	0	14	3
A	getEndDateTime()	0	17	3
A	getStartTime()	0	10	3
A	getEndTime()	0	9	3
A	isAllDay()	0	9	2
A	isOpenEndTime()	0	4	1
A	getState()	0	10	3

<?php

declare(strict_types=1);

namespace HDNET\Calendarize\Ical;

use HDNET\Calendarize\Domain\Model\ConfigurationInterface;
use HDNET\Calendarize\Utility\DateTimeUtility;
use Sabre\VObject\Component\VEvent;

class VObjectEventAdapter implements ICalEvent
{

    /**
     * @var VEvent
     */
    protected $event;

    /**
     * VObjectEvent constructor.
     * @param VEvent $event
     */
    public function __construct(VEvent $event)
    {
        $this->event = $event;
    }

    /**
     * @return VEvent
     */
    public function getEvent(): VEvent
    {
        return $this->event;
    }

    /**
     * @inheritDoc
     */
    public function getRawData(): array
    {
        return $this->event->children();
    }

    /**
     * @inheritDoc
     */
    public function getUid(): string
    {
        return $this->event->UID->getValue();
    }

    /**
     * @inheritDoc
     */
    public function getTitle(): ?string
    {
        if (!isset($this->event->SUMMARY)) {
            return null;
        }
        return $this->event->SUMMARY->getValue();
    }

    /**
     * @inheritDoc
     */
    public function getDescription(): ?string
    {
        if (!isset($this->event->DESCRIPTION)) {
            return null;
        }

        return $this->event->DESCRIPTION->getValue();
    }

    /**
     * @inheritDoc
     */
    public function getLocation(): ?string
    {
        if (!isset($this->event->LOCATION)) {
            return null;
        }

        return $this->event->LOCATION->getValue();
    }

    /**
     * @inheritDoc
     */
    public function getOrganizer(): ?string
    {
        if (!isset($this->event->ORGANIZER)) {
            return null;
        }

        return $this->event->ORGANIZER->getValue();
    }

    /**
     * @inheritDoc
     */
    public function getStartDate(): ?\DateTime
    {
        if (!isset($this->event->DTSTART)) {
            return null;
        }

        /** @var \Sabre\VObject\Property\ICalendar\DateTime $start */
        $start = $this->event->DTSTART;
        return DateTimeUtility::getDayStart($start->getDateTime());
    }

    /**
     * @inheritDoc
     */
    public function getEndDate(): ?\DateTime
    {
        $end = $this->getEndDateTime();
        if (empty($end)) {
            return null;
        }

        if ($this->isAllDay()) {
            // Converts the exclusive enddate to inclusive
            $end = (clone $end)->sub(new \DateInterval('P1D'));
        }

        return DateTimeUtility::getDayStart($end);

It seems like $end defined by (clone $end)->sub(new \DateInterval('P1D')) on line 125 can also be of type false; however, HDNET\Calendarize\Utility\DateTimeUtility::getDayStart() does only seem to accept integer|string|object<DateTimeInterface>|null, did you maybe forget to handle an error condition?

    }

    /**
     * Gets the end datetime, determines it with the duration or returns null.
     * @return \DateTimeImmutable|null
     */
    protected function getEndDateTime(): ?\DateTimeImmutable
    {
        if (isset($this->event->DTEND)) {
            /** @var \Sabre\VObject\Property\ICalendar\DateTime $dtEnd */
            $dtEnd = $this->event->DTEND;

            return $dtEnd->getDateTime();
        }
        if (isset($this->event->DURATION)) {
            /** @var \Sabre\VObject\Property\ICalendar\DateTime $dtStart */
            $dtStart = $this->event->DTSTART;
            $duration = $this->event->DURATION->getDateInterval();

            return (clone $dtStart)->getDateTime()->add($duration);
        }
        return null;
    }

    /**
     * @inheritDoc
     */
    public function getStartTime(): int
    {
        if ($this->isAllDay() || !isset($this->event->DTSTART)) {
            return self::ALLDAY_START_TIME;
        }

        /** @var \Sabre\VObject\Property\ICalendar\DateTime $start */
        $start = $this->event->DTSTART;
        return DateTimeUtility::getDaySecondsOfDateTime($start->getDateTime());

It seems like $start->getDateTime() can be null; however, getDaySecondsOfDateTime() does not accept null, maybe add an additional type check?

    }

    /**
     * @inheritDoc
     */
    public function getEndTime(): int
    {
        $end = $this->getEndDateTime();
        if ($this->isAllDay() || empty($end)) {
            return self::ALLDAY_END_TIME;
        }

        return DateTimeUtility::getDaySecondsOfDateTime($end);
    }

    /**
     * @inheritDoc
     */
    public function isAllDay(): bool
    {
        if (!isset($this->event->DTSTART)) {
            return true;
        }
        /** @var \Sabre\VObject\Property\ICalendar\DateTime $start */
        $start = $this->event->DTSTART;
        return !$start->hasTime();
    }

    /**
     * @inheritDoc
     */
    public function isOpenEndTime(): bool
    {
        return false;
    }

    /**
     * @inheritDoc
     */
    public function getState(): string
    {
        if (isset($this->event->STATUS)) {
            $status = $this->event->STATUS->getValue();
            if ($status === 'CANCELLED') {
                return ConfigurationInterface::STATE_CANCELED;
            }
        }
        return ConfigurationInterface::STATE_DEFAULT;
    }
}