Dolibarr / dolibarr

htdocs/includes/sabre/sabre/vobject/lib/Recur/EventIterator.php

Indentation +434 added lines, -434 removed lines

@@ -60,438 +60,438 @@
  */
 class EventIterator implements \Iterator
 {
-    /**
-     * Reference timeZone for floating dates and times.
-     *
-     * @var DateTimeZone
-     */
-    protected $timeZone;
-
-    /**
-     * True if we're iterating an all-day event.
-     *
-     * @var bool
-     */
-    protected $allDay = false;
-
-    /**
-     * Creates the iterator.
-     *
-     * There's three ways to set up the iterator.
-     *
-     * 1. You can pass a VCALENDAR component and a UID.
-     * 2. You can pass an array of VEVENTs (all UIDS should match).
-     * 3. You can pass a single VEVENT component.
-     *
-     * Only the second method is recommended. The other 1 and 3 will be removed
-     * at some point in the future.
-     *
-     * The $uid parameter is only required for the first method.
-     *
-     * @param Component|array $input
-     * @param string|null     $uid
-     * @param DateTimeZone    $timeZone reference timezone for floating dates and
-     *                                  times
-     */
-    public function __construct($input, $uid = null, DateTimeZone $timeZone = null)
-    {
-        if (is_null($timeZone)) {
-            $timeZone = new DateTimeZone('UTC');
-        }
-        $this->timeZone = $timeZone;
-
-        if (is_array($input)) {
-            $events = $input;
-        } elseif ($input instanceof VEvent) {
-            // Single instance mode.
-            $events = [$input];
-        } else {
-            // Calendar + UID mode.
-            $uid = (string) $uid;
-            if (!$uid) {
-                throw new InvalidArgumentException('The UID argument is required when a VCALENDAR is passed to this constructor');
-            }
-            if (!isset($input->VEVENT)) {
-                throw new InvalidArgumentException('No events found in this calendar');
-            }
-            $events = $input->getByUID($uid);
-        }
-
-        foreach ($events as $vevent) {
-            if (!isset($vevent->{'RECURRENCE-ID'})) {
-                $this->masterEvent = $vevent;
-            } else {
-                $this->exceptions[
-                    $vevent->{'RECURRENCE-ID'}->getDateTime($this->timeZone)->getTimeStamp()
-                ] = true;
-                $this->overriddenEvents[] = $vevent;
-            }
-        }
-
-        if (!$this->masterEvent) {
-            // No base event was found. CalDAV does allow cases where only
-            // overridden instances are stored.
-            //
-            // In this particular case, we're just going to grab the first
-            // event and use that instead. This may not always give the
-            // desired result.
-            if (!count($this->overriddenEvents)) {
-                throw new InvalidArgumentException('This VCALENDAR did not have an event with UID: '.$uid);
-            }
-            $this->masterEvent = array_shift($this->overriddenEvents);
-        }
-
-        $this->startDate = $this->masterEvent->DTSTART->getDateTime($this->timeZone);
-        $this->allDay = !$this->masterEvent->DTSTART->hasTime();
-
-        if (isset($this->masterEvent->EXDATE)) {
-            foreach ($this->masterEvent->EXDATE as $exDate) {
-                foreach ($exDate->getDateTimes($this->timeZone) as $dt) {
-                    $this->exceptions[$dt->getTimeStamp()] = true;
-                }
-            }
-        }
-
-        if (isset($this->masterEvent->DTEND)) {
-            $this->eventDuration =
-                $this->masterEvent->DTEND->getDateTime($this->timeZone)->getTimeStamp() -
-                $this->startDate->getTimeStamp();
-        } elseif (isset($this->masterEvent->DURATION)) {
-            $duration = $this->masterEvent->DURATION->getDateInterval();
-            $end = clone $this->startDate;
-            $end = $end->add($duration);
-            $this->eventDuration = $end->getTimeStamp() - $this->startDate->getTimeStamp();
-        } elseif ($this->allDay) {
-            $this->eventDuration = 3600 * 24;
-        } else {
-            $this->eventDuration = 0;
-        }
-
-        if (isset($this->masterEvent->RDATE)) {
-            $this->recurIterator = new RDateIterator(
-                $this->masterEvent->RDATE->getParts(),
-                $this->startDate
-            );
-        } elseif (isset($this->masterEvent->RRULE)) {
-            $this->recurIterator = new RRuleIterator(
-                $this->masterEvent->RRULE->getParts(),
-                $this->startDate
-            );
-        } else {
-            $this->recurIterator = new RRuleIterator(
-                [
-                    'FREQ' => 'DAILY',
-                    'COUNT' => 1,
-                ],
-                $this->startDate
-            );
-        }
-
-        $this->rewind();
-        if (!$this->valid()) {
-            throw new NoInstancesException('This recurrence rule does not generate any valid instances');
-        }
-    }
-
-    /**
-     * Returns the date for the current position of the iterator.
-     *
-     * @return DateTimeImmutable
-     */
-    #[\ReturnTypeWillChange]
-    public function current()
-    {
-        if ($this->currentDate) {
-            return clone $this->currentDate;
-        }
-    }
-
-    /**
-     * This method returns the start date for the current iteration of the
-     * event.
-     *
-     * @return DateTimeImmutable
-     */
-    public function getDtStart()
-    {
-        if ($this->currentDate) {
-            return clone $this->currentDate;
-        }
-    }
-
-    /**
-     * This method returns the end date for the current iteration of the
-     * event.
-     *
-     * @return DateTimeImmutable
-     */
-    public function getDtEnd()
-    {
-        if (!$this->valid()) {
-            return;
-        }
-        if ($this->currentOverriddenEvent && $this->currentOverriddenEvent->DTEND) {
-            return $this->currentOverriddenEvent->DTEND->getDateTime($this->timeZone);
-        } else {
-            $end = clone $this->currentDate;
-
-            return $end->modify('+'.$this->eventDuration.' seconds');
-        }
-    }
-
-    /**
-     * Returns a VEVENT for the current iterations of the event.
-     *
-     * This VEVENT will have a recurrence id, and its DTSTART and DTEND
-     * altered.
-     *
-     * @return VEvent
-     */
-    public function getEventObject()
-    {
-        if ($this->currentOverriddenEvent) {
-            return $this->currentOverriddenEvent;
-        }
-
-        $event = clone $this->masterEvent;
-
-        // Ignoring the following block, because PHPUnit's code coverage
-        // ignores most of these lines, and this messes with our stats.
-        //
-        // @codeCoverageIgnoreStart
-        unset(
-            $event->RRULE,
-            $event->EXDATE,
-            $event->RDATE,
-            $event->EXRULE,
-            $event->{'RECURRENCE-ID'}
-        );
-        // @codeCoverageIgnoreEnd
-
-        $event->DTSTART->setDateTime($this->getDtStart(), $event->DTSTART->isFloating());
-        if (isset($event->DTEND)) {
-            $event->DTEND->setDateTime($this->getDtEnd(), $event->DTEND->isFloating());
-        }
-        $recurid = clone $event->DTSTART;
-        $recurid->name = 'RECURRENCE-ID';
-        $event->add($recurid);
-
-        return $event;
-    }
-
-    /**
-     * Returns the current position of the iterator.
-     *
-     * This is for us simply a 0-based index.
-     *
-     * @return int
-     */
-    #[\ReturnTypeWillChange]
-    public function key()
-    {
-        // The counter is always 1 ahead.
-        return $this->counter - 1;
-    }
-
-    /**
-     * This is called after next, to see if the iterator is still at a valid
-     * position, or if it's at the end.
-     *
-     * @return bool
-     */
-    #[\ReturnTypeWillChange]
-    public function valid()
-    {
-        if ($this->counter > Settings::$maxRecurrences && -1 !== Settings::$maxRecurrences) {
-            throw new MaxInstancesExceededException('Recurring events are only allowed to generate '.Settings::$maxRecurrences);
-        }
-
-        return (bool) $this->currentDate;
-    }
-
-    /**
-     * Sets the iterator back to the starting point.
-     *
-     * @return void
-     */
-    #[\ReturnTypeWillChange]
-    public function rewind()
-    {
-        $this->recurIterator->rewind();
-        // re-creating overridden event index.
-        $index = [];
-        foreach ($this->overriddenEvents as $key => $event) {
-            $stamp = $event->DTSTART->getDateTime($this->timeZone)->getTimeStamp();
-            $index[$stamp][] = $key;
-        }
-        krsort($index);
-        $this->counter = 0;
-        $this->overriddenEventsIndex = $index;
-        $this->currentOverriddenEvent = null;
-
-        $this->nextDate = null;
-        $this->currentDate = clone $this->startDate;
-
-        $this->next();
-    }
-
-    /**
-     * Advances the iterator with one step.
-     *
-     * @return void
-     */
-    #[\ReturnTypeWillChange]
-    public function next()
-    {
-        $this->currentOverriddenEvent = null;
-        ++$this->counter;
-        if ($this->nextDate) {
-            // We had a stored value.
-            $nextDate = $this->nextDate;
-            $this->nextDate = null;
-        } else {
-            // We need to ask rruleparser for the next date.
-            // We need to do this until we find a date that's not in the
-            // exception list.
-            do {
-                if (!$this->recurIterator->valid()) {
-                    $nextDate = null;
-                    break;
-                }
-                $nextDate = $this->recurIterator->current();
-                $this->recurIterator->next();
-            } while (isset($this->exceptions[$nextDate->getTimeStamp()]));
-        }
-
-        // $nextDate now contains what rrule thinks is the next one, but an
-        // overridden event may cut ahead.
-        if ($this->overriddenEventsIndex) {
-            $offsets = end($this->overriddenEventsIndex);
-            $timestamp = key($this->overriddenEventsIndex);
-            $offset = end($offsets);
-            if (!$nextDate || $timestamp < $nextDate->getTimeStamp()) {
-                // Overridden event comes first.
-                $this->currentOverriddenEvent = $this->overriddenEvents[$offset];
-
-                // Putting the rrule next date aside.
-                $this->nextDate = $nextDate;
-                $this->currentDate = $this->currentOverriddenEvent->DTSTART->getDateTime($this->timeZone);
-
-                // Ensuring that this item will only be used once.
-                array_pop($this->overriddenEventsIndex[$timestamp]);
-                if (!$this->overriddenEventsIndex[$timestamp]) {
-                    array_pop($this->overriddenEventsIndex);
-                }
-
-                // Exit point!
-                return;
-            }
-        }
-
-        $this->currentDate = $nextDate;
-    }
-
-    /**
-     * Quickly jump to a date in the future.
-     */
-    public function fastForward(DateTimeInterface $dateTime)
-    {
-        while ($this->valid() && $this->getDtEnd() <= $dateTime) {
-            $this->next();
-        }
-    }
-
-    /**
-     * Returns true if this recurring event never ends.
-     *
-     * @return bool
-     */
-    public function isInfinite()
-    {
-        return $this->recurIterator->isInfinite();
-    }
-
-    /**
-     * RRULE parser.
-     *
-     * @var RRuleIterator
-     */
-    protected $recurIterator;
-
-    /**
-     * The duration, in seconds, of the master event.
-     *
-     * We use this to calculate the DTEND for subsequent events.
-     */
-    protected $eventDuration;
-
-    /**
-     * A reference to the main (master) event.
-     *
-     * @var VEVENT
-     */
-    protected $masterEvent;
-
-    /**
-     * List of overridden events.
-     *
-     * @var array
-     */
-    protected $overriddenEvents = [];
-
-    /**
-     * Overridden event index.
-     *
-     * Key is timestamp, value is the list of indexes of the item in the $overriddenEvent
-     * property.
-     *
-     * @var array
-     */
-    protected $overriddenEventsIndex;
-
-    /**
-     * A list of recurrence-id's that are either part of EXDATE, or are
-     * overridden.
-     *
-     * @var array
-     */
-    protected $exceptions = [];
-
-    /**
-     * Internal event counter.
-     *
-     * @var int
-     */
-    protected $counter;
-
-    /**
-     * The very start of the iteration process.
-     *
-     * @var DateTimeImmutable
-     */
-    protected $startDate;
-
-    /**
-     * Where we are currently in the iteration process.
-     *
-     * @var DateTimeImmutable
-     */
-    protected $currentDate;
-
-    /**
-     * The next date from the rrule parser.
-     *
-     * Sometimes we need to temporary store the next date, because an
-     * overridden event came before.
-     *
-     * @var DateTimeImmutable
-     */
-    protected $nextDate;
-
-    /**
-     * The event that overwrites the current iteration.
-     *
-     * @var VEVENT
-     */
-    protected $currentOverriddenEvent;
+	/**
+	 * Reference timeZone for floating dates and times.
+	 *
+	 * @var DateTimeZone
+	 */
+	protected $timeZone;
+
+	/**
+	 * True if we're iterating an all-day event.
+	 *
+	 * @var bool
+	 */
+	protected $allDay = false;
+
+	/**
+	 * Creates the iterator.
+	 *
+	 * There's three ways to set up the iterator.
+	 *
+	 * 1. You can pass a VCALENDAR component and a UID.
+	 * 2. You can pass an array of VEVENTs (all UIDS should match).
+	 * 3. You can pass a single VEVENT component.
+	 *
+	 * Only the second method is recommended. The other 1 and 3 will be removed
+	 * at some point in the future.
+	 *
+	 * The $uid parameter is only required for the first method.
+	 *
+	 * @param Component|array $input
+	 * @param string|null     $uid
+	 * @param DateTimeZone    $timeZone reference timezone for floating dates and
+	 *                                  times
+	 */
+	public function __construct($input, $uid = null, DateTimeZone $timeZone = null)
+	{
+		if (is_null($timeZone)) {
+			$timeZone = new DateTimeZone('UTC');
+		}
+		$this->timeZone = $timeZone;
+
+		if (is_array($input)) {
+			$events = $input;
+		} elseif ($input instanceof VEvent) {
+			// Single instance mode.
+			$events = [$input];
+		} else {
+			// Calendar + UID mode.
+			$uid = (string) $uid;
+			if (!$uid) {
+				throw new InvalidArgumentException('The UID argument is required when a VCALENDAR is passed to this constructor');
+			}
+			if (!isset($input->VEVENT)) {
+				throw new InvalidArgumentException('No events found in this calendar');
+			}
+			$events = $input->getByUID($uid);
+		}
+
+		foreach ($events as $vevent) {
+			if (!isset($vevent->{'RECURRENCE-ID'})) {
+				$this->masterEvent = $vevent;
+			} else {
+				$this->exceptions[
+					$vevent->{'RECURRENCE-ID'}->getDateTime($this->timeZone)->getTimeStamp()
+				] = true;
+				$this->overriddenEvents[] = $vevent;
+			}
+		}
+
+		if (!$this->masterEvent) {
+			// No base event was found. CalDAV does allow cases where only
+			// overridden instances are stored.
+			//
+			// In this particular case, we're just going to grab the first
+			// event and use that instead. This may not always give the
+			// desired result.
+			if (!count($this->overriddenEvents)) {
+				throw new InvalidArgumentException('This VCALENDAR did not have an event with UID: '.$uid);
+			}
+			$this->masterEvent = array_shift($this->overriddenEvents);
+		}
+
+		$this->startDate = $this->masterEvent->DTSTART->getDateTime($this->timeZone);
+		$this->allDay = !$this->masterEvent->DTSTART->hasTime();
+
+		if (isset($this->masterEvent->EXDATE)) {
+			foreach ($this->masterEvent->EXDATE as $exDate) {
+				foreach ($exDate->getDateTimes($this->timeZone) as $dt) {
+					$this->exceptions[$dt->getTimeStamp()] = true;
+				}
+			}
+		}
+
+		if (isset($this->masterEvent->DTEND)) {
+			$this->eventDuration =
+				$this->masterEvent->DTEND->getDateTime($this->timeZone)->getTimeStamp() -
+				$this->startDate->getTimeStamp();
+		} elseif (isset($this->masterEvent->DURATION)) {
+			$duration = $this->masterEvent->DURATION->getDateInterval();
+			$end = clone $this->startDate;
+			$end = $end->add($duration);
+			$this->eventDuration = $end->getTimeStamp() - $this->startDate->getTimeStamp();
+		} elseif ($this->allDay) {
+			$this->eventDuration = 3600 * 24;
+		} else {
+			$this->eventDuration = 0;
+		}
+
+		if (isset($this->masterEvent->RDATE)) {
+			$this->recurIterator = new RDateIterator(
+				$this->masterEvent->RDATE->getParts(),
+				$this->startDate
+			);
+		} elseif (isset($this->masterEvent->RRULE)) {
+			$this->recurIterator = new RRuleIterator(
+				$this->masterEvent->RRULE->getParts(),
+				$this->startDate
+			);
+		} else {
+			$this->recurIterator = new RRuleIterator(
+				[
+					'FREQ' => 'DAILY',
+					'COUNT' => 1,
+				],
+				$this->startDate
+			);
+		}
+
+		$this->rewind();
+		if (!$this->valid()) {
+			throw new NoInstancesException('This recurrence rule does not generate any valid instances');
+		}
+	}
+
+	/**
+	 * Returns the date for the current position of the iterator.
+	 *
+	 * @return DateTimeImmutable
+	 */
+	#[\ReturnTypeWillChange]
+	public function current()
+	{
+		if ($this->currentDate) {
+			return clone $this->currentDate;
+		}
+	}
+
+	/**
+	 * This method returns the start date for the current iteration of the
+	 * event.
+	 *
+	 * @return DateTimeImmutable
+	 */
+	public function getDtStart()
+	{
+		if ($this->currentDate) {
+			return clone $this->currentDate;
+		}
+	}
+
+	/**
+	 * This method returns the end date for the current iteration of the
+	 * event.
+	 *
+	 * @return DateTimeImmutable
+	 */
+	public function getDtEnd()
+	{
+		if (!$this->valid()) {
+			return;
+		}
+		if ($this->currentOverriddenEvent && $this->currentOverriddenEvent->DTEND) {
+			return $this->currentOverriddenEvent->DTEND->getDateTime($this->timeZone);
+		} else {
+			$end = clone $this->currentDate;
+
+			return $end->modify('+'.$this->eventDuration.' seconds');
+		}
+	}
+
+	/**
+	 * Returns a VEVENT for the current iterations of the event.
+	 *
+	 * This VEVENT will have a recurrence id, and its DTSTART and DTEND
+	 * altered.
+	 *
+	 * @return VEvent
+	 */
+	public function getEventObject()
+	{
+		if ($this->currentOverriddenEvent) {
+			return $this->currentOverriddenEvent;
+		}
+
+		$event = clone $this->masterEvent;
+
+		// Ignoring the following block, because PHPUnit's code coverage
+		// ignores most of these lines, and this messes with our stats.
+		//
+		// @codeCoverageIgnoreStart
+		unset(
+			$event->RRULE,
+			$event->EXDATE,
+			$event->RDATE,
+			$event->EXRULE,
+			$event->{'RECURRENCE-ID'}
+		);
+		// @codeCoverageIgnoreEnd
+
+		$event->DTSTART->setDateTime($this->getDtStart(), $event->DTSTART->isFloating());
+		if (isset($event->DTEND)) {
+			$event->DTEND->setDateTime($this->getDtEnd(), $event->DTEND->isFloating());
+		}
+		$recurid = clone $event->DTSTART;
+		$recurid->name = 'RECURRENCE-ID';
+		$event->add($recurid);
+
+		return $event;
+	}
+
+	/**
+	 * Returns the current position of the iterator.
+	 *
+	 * This is for us simply a 0-based index.
+	 *
+	 * @return int
+	 */
+	#[\ReturnTypeWillChange]
+	public function key()
+	{
+		// The counter is always 1 ahead.
+		return $this->counter - 1;
+	}
+
+	/**
+	 * This is called after next, to see if the iterator is still at a valid
+	 * position, or if it's at the end.
+	 *
+	 * @return bool
+	 */
+	#[\ReturnTypeWillChange]
+	public function valid()
+	{
+		if ($this->counter > Settings::$maxRecurrences && -1 !== Settings::$maxRecurrences) {
+			throw new MaxInstancesExceededException('Recurring events are only allowed to generate '.Settings::$maxRecurrences);
+		}
+
+		return (bool) $this->currentDate;
+	}
+
+	/**
+	 * Sets the iterator back to the starting point.
+	 *
+	 * @return void
+	 */
+	#[\ReturnTypeWillChange]
+	public function rewind()
+	{
+		$this->recurIterator->rewind();
+		// re-creating overridden event index.
+		$index = [];
+		foreach ($this->overriddenEvents as $key => $event) {
+			$stamp = $event->DTSTART->getDateTime($this->timeZone)->getTimeStamp();
+			$index[$stamp][] = $key;
+		}
+		krsort($index);
+		$this->counter = 0;
+		$this->overriddenEventsIndex = $index;
+		$this->currentOverriddenEvent = null;
+
+		$this->nextDate = null;
+		$this->currentDate = clone $this->startDate;
+
+		$this->next();
+	}
+
+	/**
+	 * Advances the iterator with one step.
+	 *
+	 * @return void
+	 */
+	#[\ReturnTypeWillChange]
+	public function next()
+	{
+		$this->currentOverriddenEvent = null;
+		++$this->counter;
+		if ($this->nextDate) {
+			// We had a stored value.
+			$nextDate = $this->nextDate;
+			$this->nextDate = null;
+		} else {
+			// We need to ask rruleparser for the next date.
+			// We need to do this until we find a date that's not in the
+			// exception list.
+			do {
+				if (!$this->recurIterator->valid()) {
+					$nextDate = null;
+					break;
+				}
+				$nextDate = $this->recurIterator->current();
+				$this->recurIterator->next();
+			} while (isset($this->exceptions[$nextDate->getTimeStamp()]));
+		}
+
+		// $nextDate now contains what rrule thinks is the next one, but an
+		// overridden event may cut ahead.
+		if ($this->overriddenEventsIndex) {
+			$offsets = end($this->overriddenEventsIndex);
+			$timestamp = key($this->overriddenEventsIndex);
+			$offset = end($offsets);
+			if (!$nextDate || $timestamp < $nextDate->getTimeStamp()) {
+				// Overridden event comes first.
+				$this->currentOverriddenEvent = $this->overriddenEvents[$offset];
+
+				// Putting the rrule next date aside.
+				$this->nextDate = $nextDate;
+				$this->currentDate = $this->currentOverriddenEvent->DTSTART->getDateTime($this->timeZone);
+
+				// Ensuring that this item will only be used once.
+				array_pop($this->overriddenEventsIndex[$timestamp]);
+				if (!$this->overriddenEventsIndex[$timestamp]) {
+					array_pop($this->overriddenEventsIndex);
+				}
+
+				// Exit point!
+				return;
+			}
+		}
+
+		$this->currentDate = $nextDate;
+	}
+
+	/**
+	 * Quickly jump to a date in the future.
+	 */
+	public function fastForward(DateTimeInterface $dateTime)
+	{
+		while ($this->valid() && $this->getDtEnd() <= $dateTime) {
+			$this->next();
+		}
+	}
+
+	/**
+	 * Returns true if this recurring event never ends.
+	 *
+	 * @return bool
+	 */
+	public function isInfinite()
+	{
+		return $this->recurIterator->isInfinite();
+	}
+
+	/**
+	 * RRULE parser.
+	 *
+	 * @var RRuleIterator
+	 */
+	protected $recurIterator;
+
+	/**
+	 * The duration, in seconds, of the master event.
+	 *
+	 * We use this to calculate the DTEND for subsequent events.
+	 */
+	protected $eventDuration;
+
+	/**
+	 * A reference to the main (master) event.
+	 *
+	 * @var VEVENT
+	 */
+	protected $masterEvent;
+
+	/**
+	 * List of overridden events.
+	 *
+	 * @var array
+	 */
+	protected $overriddenEvents = [];
+
+	/**
+	 * Overridden event index.
+	 *
+	 * Key is timestamp, value is the list of indexes of the item in the $overriddenEvent
+	 * property.
+	 *
+	 * @var array
+	 */
+	protected $overriddenEventsIndex;
+
+	/**
+	 * A list of recurrence-id's that are either part of EXDATE, or are
+	 * overridden.
+	 *
+	 * @var array
+	 */
+	protected $exceptions = [];
+
+	/**
+	 * Internal event counter.
+	 *
+	 * @var int
+	 */
+	protected $counter;
+
+	/**
+	 * The very start of the iteration process.
+	 *
+	 * @var DateTimeImmutable
+	 */
+	protected $startDate;
+
+	/**
+	 * Where we are currently in the iteration process.
+	 *
+	 * @var DateTimeImmutable
+	 */
+	protected $currentDate;
+
+	/**
+	 * The next date from the rrule parser.
+	 *
+	 * Sometimes we need to temporary store the next date, because an
+	 * overridden event came before.
+	 *
+	 * @var DateTimeImmutable
+	 */
+	protected $nextDate;
+
+	/**
+	 * The event that overwrites the current iteration.
+	 *
+	 * @var VEVENT
+	 */
+	protected $currentOverriddenEvent;
 }

htdocs/includes/sabre/sabre/vobject/lib/Recur/RRuleIterator.php

Indentation +983 added lines, -983 removed lines

@@ -24,987 +24,987 @@
  */
 class RRuleIterator implements Iterator
 {
-    /**
-     * Constant denoting the upper limit on how long into the future
-     * we want to iterate. The value is a unix timestamp and currently
-     * corresponds to the datetime 9999-12-31 11:59:59 UTC.
-     */
-    const dateUpperLimit = 253402300799;
-
-    /**
-     * Creates the Iterator.
-     *
-     * @param string|array $rrule
-     */
-    public function __construct($rrule, DateTimeInterface $start)
-    {
-        $this->startDate = $start;
-        $this->parseRRule($rrule);
-        $this->currentDate = clone $this->startDate;
-    }
-
-    /* Implementation of the Iterator interface {{{ */
-
-    #[\ReturnTypeWillChange]
-    public function current()
-    {
-        if (!$this->valid()) {
-            return;
-        }
-
-        return clone $this->currentDate;
-    }
-
-    /**
-     * Returns the current item number.
-     *
-     * @return int
-     */
-    #[\ReturnTypeWillChange]
-    public function key()
-    {
-        return $this->counter;
-    }
-
-    /**
-     * Returns whether the current item is a valid item for the recurrence
-     * iterator. This will return false if we've gone beyond the UNTIL or COUNT
-     * statements.
-     *
-     * @return bool
-     */
-    #[\ReturnTypeWillChange]
-    public function valid()
-    {
-        if (null === $this->currentDate) {
-            return false;
-        }
-        if (!is_null($this->count)) {
-            return $this->counter < $this->count;
-        }
-
-        return is_null($this->until) || $this->currentDate <= $this->until;
-    }
-
-    /**
-     * Resets the iterator.
-     *
-     * @return void
-     */
-    #[\ReturnTypeWillChange]
-    public function rewind()
-    {
-        $this->currentDate = clone $this->startDate;
-        $this->counter = 0;
-    }
-
-    /**
-     * Goes on to the next iteration.
-     *
-     * @return void
-     */
-    #[\ReturnTypeWillChange]
-    public function next()
-    {
-        // Otherwise, we find the next event in the normal RRULE
-        // sequence.
-        switch ($this->frequency) {
-            case 'hourly':
-                $this->nextHourly();
-                break;
-
-            case 'daily':
-                $this->nextDaily();
-                break;
-
-            case 'weekly':
-                $this->nextWeekly();
-                break;
-
-            case 'monthly':
-                $this->nextMonthly();
-                break;
-
-            case 'yearly':
-                $this->nextYearly();
-                break;
-        }
-        ++$this->counter;
-    }
-
-    /* End of Iterator implementation }}} */
-
-    /**
-     * Returns true if this recurring event never ends.
-     *
-     * @return bool
-     */
-    public function isInfinite()
-    {
-        return !$this->count && !$this->until;
-    }
-
-    /**
-     * This method allows you to quickly go to the next occurrence after the
-     * specified date.
-     */
-    public function fastForward(DateTimeInterface $dt)
-    {
-        while ($this->valid() && $this->currentDate < $dt) {
-            $this->next();
-        }
-    }
-
-    /**
-     * The reference start date/time for the rrule.
-     *
-     * All calculations are based on this initial date.
-     *
-     * @var DateTimeInterface
-     */
-    protected $startDate;
-
-    /**
-     * The date of the current iteration. You can get this by calling
-     * ->current().
-     *
-     * @var DateTimeInterface
-     */
-    protected $currentDate;
-
-    /**
-     * Frequency is one of: secondly, minutely, hourly, daily, weekly, monthly,
-     * yearly.
-     *
-     * @var string
-     */
-    protected $frequency;
-
-    /**
-     * The number of recurrences, or 'null' if infinitely recurring.
-     *
-     * @var int
-     */
-    protected $count;
-
-    /**
-     * The interval.
-     *
-     * If for example frequency is set to daily, interval = 2 would mean every
-     * 2 days.
-     *
-     * @var int
-     */
-    protected $interval = 1;
-
-    /**
-     * The last instance of this recurrence, inclusively.
-     *
-     * @var DateTimeInterface|null
-     */
-    protected $until;
-
-    /**
-     * Which seconds to recur.
-     *
-     * This is an array of integers (between 0 and 60)
-     *
-     * @var array
-     */
-    protected $bySecond;
-
-    /**
-     * Which minutes to recur.
-     *
-     * This is an array of integers (between 0 and 59)
-     *
-     * @var array
-     */
-    protected $byMinute;
-
-    /**
-     * Which hours to recur.
-     *
-     * This is an array of integers (between 0 and 23)
-     *
-     * @var array
-     */
-    protected $byHour;
-
-    /**
-     * The current item in the list.
-     *
-     * You can get this number with the key() method.
-     *
-     * @var int
-     */
-    protected $counter = 0;
-
-    /**
-     * Which weekdays to recur.
-     *
-     * This is an array of weekdays
-     *
-     * This may also be preceded by a positive or negative integer. If present,
-     * this indicates the nth occurrence of a specific day within the monthly or
-     * yearly rrule. For instance, -2TU indicates the second-last tuesday of
-     * the month, or year.
-     *
-     * @var array
-     */
-    protected $byDay;
-
-    /**
-     * Which days of the month to recur.
-     *
-     * This is an array of days of the months (1-31). The value can also be
-     * negative. -5 for instance means the 5th last day of the month.
-     *
-     * @var array
-     */
-    protected $byMonthDay;
-
-    /**
-     * Which days of the year to recur.
-     *
-     * This is an array with days of the year (1 to 366). The values can also
-     * be negative. For instance, -1 will always represent the last day of the
-     * year. (December 31st).
-     *
-     * @var array
-     */
-    protected $byYearDay;
-
-    /**
-     * Which week numbers to recur.
-     *
-     * This is an array of integers from 1 to 53. The values can also be
-     * negative. -1 will always refer to the last week of the year.
-     *
-     * @var array
-     */
-    protected $byWeekNo;
-
-    /**
-     * Which months to recur.
-     *
-     * This is an array of integers from 1 to 12.
-     *
-     * @var array
-     */
-    protected $byMonth;
-
-    /**
-     * Which items in an existing st to recur.
-     *
-     * These numbers work together with an existing by* rule. It specifies
-     * exactly which items of the existing by-rule to filter.
-     *
-     * Valid values are 1 to 366 and -1 to -366. As an example, this can be
-     * used to recur the last workday of the month.
-     *
-     * This would be done by setting frequency to 'monthly', byDay to
-     * 'MO,TU,WE,TH,FR' and bySetPos to -1.
-     *
-     * @var array
-     */
-    protected $bySetPos;
-
-    /**
-     * When the week starts.
-     *
-     * @var string
-     */
-    protected $weekStart = 'MO';
-
-    /* Functions that advance the iterator {{{ */
-
-    /**
-     * Does the processing for advancing the iterator for hourly frequency.
-     */
-    protected function nextHourly()
-    {
-        $this->currentDate = $this->currentDate->modify('+'.$this->interval.' hours');
-    }
-
-    /**
-     * Does the processing for advancing the iterator for daily frequency.
-     */
-    protected function nextDaily()
-    {
-        if (!$this->byHour && !$this->byDay) {
-            $this->currentDate = $this->currentDate->modify('+'.$this->interval.' days');
-
-            return;
-        }
-
-        $recurrenceHours = [];
-        if (!empty($this->byHour)) {
-            $recurrenceHours = $this->getHours();
-        }
-
-        $recurrenceDays = [];
-        if (!empty($this->byDay)) {
-            $recurrenceDays = $this->getDays();
-        }
-
-        $recurrenceMonths = [];
-        if (!empty($this->byMonth)) {
-            $recurrenceMonths = $this->getMonths();
-        }
-
-        do {
-            if ($this->byHour) {
-                if ('23' == $this->currentDate->format('G')) {
-                    // to obey the interval rule
-                    $this->currentDate = $this->currentDate->modify('+'.($this->interval - 1).' days');
-                }
-
-                $this->currentDate = $this->currentDate->modify('+1 hours');
-            } else {
-                $this->currentDate = $this->currentDate->modify('+'.$this->interval.' days');
-            }
-
-            // Current month of the year
-            $currentMonth = $this->currentDate->format('n');
-
-            // Current day of the week
-            $currentDay = $this->currentDate->format('w');
-
-            // Current hour of the day
-            $currentHour = $this->currentDate->format('G');
-
-            if ($this->currentDate->getTimestamp() > self::dateUpperLimit) {
-                $this->currentDate = null;
-
-                return;
-            }
-        } while (
-            ($this->byDay && !in_array($currentDay, $recurrenceDays)) ||
-            ($this->byHour && !in_array($currentHour, $recurrenceHours)) ||
-            ($this->byMonth && !in_array($currentMonth, $recurrenceMonths))
-        );
-    }
-
-    /**
-     * Does the processing for advancing the iterator for weekly frequency.
-     */
-    protected function nextWeekly()
-    {
-        if (!$this->byHour && !$this->byDay) {
-            $this->currentDate = $this->currentDate->modify('+'.$this->interval.' weeks');
-
-            return;
-        }
-
-        $recurrenceHours = [];
-        if ($this->byHour) {
-            $recurrenceHours = $this->getHours();
-        }
-
-        $recurrenceDays = [];
-        if ($this->byDay) {
-            $recurrenceDays = $this->getDays();
-        }
-
-        // First day of the week:
-        $firstDay = $this->dayMap[$this->weekStart];
-
-        do {
-            if ($this->byHour) {
-                $this->currentDate = $this->currentDate->modify('+1 hours');
-            } else {
-                $this->currentDate = $this->currentDate->modify('+1 days');
-            }
-
-            // Current day of the week
-            $currentDay = (int) $this->currentDate->format('w');
-
-            // Current hour of the day
-            $currentHour = (int) $this->currentDate->format('G');
-
-            // We need to roll over to the next week
-            if ($currentDay === $firstDay && (!$this->byHour || '0' == $currentHour)) {
-                $this->currentDate = $this->currentDate->modify('+'.($this->interval - 1).' weeks');
-
-                // We need to go to the first day of this week, but only if we
-                // are not already on this first day of this week.
-                if ($this->currentDate->format('w') != $firstDay) {
-                    $this->currentDate = $this->currentDate->modify('last '.$this->dayNames[$this->dayMap[$this->weekStart]]);
-                }
-            }
-
-            // We have a match
-        } while (($this->byDay && !in_array($currentDay, $recurrenceDays)) || ($this->byHour && !in_array($currentHour, $recurrenceHours)));
-    }
-
-    /**
-     * Does the processing for advancing the iterator for monthly frequency.
-     */
-    protected function nextMonthly()
-    {
-        $currentDayOfMonth = $this->currentDate->format('j');
-        if (!$this->byMonthDay && !$this->byDay) {
-            // If the current day is higher than the 28th, rollover can
-            // occur to the next month. We Must skip these invalid
-            // entries.
-            if ($currentDayOfMonth < 29) {
-                $this->currentDate = $this->currentDate->modify('+'.$this->interval.' months');
-            } else {
-                $increase = 0;
-                do {
-                    ++$increase;
-                    $tempDate = clone $this->currentDate;
-                    $tempDate = $tempDate->modify('+ '.($this->interval * $increase).' months');
-                } while ($tempDate->format('j') != $currentDayOfMonth);
-                $this->currentDate = $tempDate;
-            }
-
-            return;
-        }
-
-        $occurrence = -1;
-        while (true) {
-            $occurrences = $this->getMonthlyOccurrences();
-
-            foreach ($occurrences as $occurrence) {
-                // The first occurrence thats higher than the current
-                // day of the month wins.
-                if ($occurrence > $currentDayOfMonth) {
-                    break 2;
-                }
-            }
-
-            // If we made it all the way here, it means there were no
-            // valid occurrences, and we need to advance to the next
-            // month.
-            //
-            // This line does not currently work in hhvm. Temporary workaround
-            // follows:
-            // $this->currentDate->modify('first day of this month');
-            $this->currentDate = new DateTimeImmutable($this->currentDate->format('Y-m-1 H:i:s'), $this->currentDate->getTimezone());
-            // end of workaround
-            $this->currentDate = $this->currentDate->modify('+ '.$this->interval.' months');
-
-            // This goes to 0 because we need to start counting at the
-            // beginning.
-            $currentDayOfMonth = 0;
-
-            // For some reason the "until" parameter was not being used here,
-            // that's why the workaround of the 10000 year bug was needed at all
-            // let's stop it before the "until" parameter date
-            if ($this->until && $this->currentDate->getTimestamp() >= $this->until->getTimestamp()) {
-                return;
-            }
-
-            // To prevent running this forever (better: until we hit the max date of DateTimeImmutable) we simply
-            // stop at 9999-12-31. Looks like the year 10000 problem is not solved in php ....
-            if ($this->currentDate->getTimestamp() > self::dateUpperLimit) {
-                $this->currentDate = null;
-
-                return;
-            }
-        }
-
-        $this->currentDate = $this->currentDate->setDate(
-            (int) $this->currentDate->format('Y'),
-            (int) $this->currentDate->format('n'),
-            (int) $occurrence
-        );
-    }
-
-    /**
-     * Does the processing for advancing the iterator for yearly frequency.
-     */
-    protected function nextYearly()
-    {
-        $currentMonth = $this->currentDate->format('n');
-        $currentYear = $this->currentDate->format('Y');
-        $currentDayOfMonth = $this->currentDate->format('j');
-
-        // No sub-rules, so we just advance by year
-        if (empty($this->byMonth)) {
-            // Unless it was a leap day!
-            if (2 == $currentMonth && 29 == $currentDayOfMonth) {
-                $counter = 0;
-                do {
-                    ++$counter;
-                    // Here we increase the year count by the interval, until
-                    // we hit a date that's also in a leap year.
-                    //
-                    // We could just find the next interval that's dividable by
-                    // 4, but that would ignore the rule that there's no leap
-                    // year every year that's dividable by a 100, but not by
-                    // 400. (1800, 1900, 2100). So we just rely on the datetime
-                    // functions instead.
-                    $nextDate = clone $this->currentDate;
-                    $nextDate = $nextDate->modify('+ '.($this->interval * $counter).' years');
-                } while (2 != $nextDate->format('n'));
-
-                $this->currentDate = $nextDate;
-
-                return;
-            }
-
-            if (null !== $this->byWeekNo) { // byWeekNo is an array with values from -53 to -1, or 1 to 53
-                $dayOffsets = [];
-                if ($this->byDay) {
-                    foreach ($this->byDay as $byDay) {
-                        $dayOffsets[] = $this->dayMap[$byDay];
-                    }
-                } else {   // default is Monday
-                    $dayOffsets[] = 1;
-                }
-
-                $currentYear = $this->currentDate->format('Y');
-
-                while (true) {
-                    $checkDates = [];
-
-                    // loop through all WeekNo and Days to check all the combinations
-                    foreach ($this->byWeekNo as $byWeekNo) {
-                        foreach ($dayOffsets as $dayOffset) {
-                            $date = clone $this->currentDate;
-                            $date = $date->setISODate($currentYear, $byWeekNo, $dayOffset);
-
-                            if ($date > $this->currentDate) {
-                                $checkDates[] = $date;
-                            }
-                        }
-                    }
-
-                    if (count($checkDates) > 0) {
-                        $this->currentDate = min($checkDates);
-
-                        return;
-                    }
-
-                    // if there is no date found, check the next year
-                    $currentYear += $this->interval;
-                }
-            }
-
-            if (null !== $this->byYearDay) { // byYearDay is an array with values from -366 to -1, or 1 to 366
-                $dayOffsets = [];
-                if ($this->byDay) {
-                    foreach ($this->byDay as $byDay) {
-                        $dayOffsets[] = $this->dayMap[$byDay];
-                    }
-                } else {   // default is Monday-Sunday
-                    $dayOffsets = [1, 2, 3, 4, 5, 6, 7];
-                }
-
-                $currentYear = $this->currentDate->format('Y');
-
-                while (true) {
-                    $checkDates = [];
-
-                    // loop through all YearDay and Days to check all the combinations
-                    foreach ($this->byYearDay as $byYearDay) {
-                        $date = clone $this->currentDate;
-                        if ($byYearDay > 0) {
-                            $date = $date->setDate($currentYear, 1, 1);
-                            $date = $date->add(new \DateInterval('P'.($byYearDay - 1).'D'));
-                        } else {
-                            $date = $date->setDate($currentYear, 12, 31);
-                            $date = $date->sub(new \DateInterval('P'.abs($byYearDay + 1).'D'));
-                        }
-
-                        if ($date > $this->currentDate && in_array($date->format('N'), $dayOffsets)) {
-                            $checkDates[] = $date;
-                        }
-                    }
-
-                    if (count($checkDates) > 0) {
-                        $this->currentDate = min($checkDates);
-
-                        return;
-                    }
-
-                    // if there is no date found, check the next year
-                    $currentYear += $this->interval;
-                }
-            }
-
-            // The easiest form
-            $this->currentDate = $this->currentDate->modify('+'.$this->interval.' years');
-
-            return;
-        }
-
-        $currentMonth = $this->currentDate->format('n');
-        $currentYear = $this->currentDate->format('Y');
-        $currentDayOfMonth = $this->currentDate->format('j');
-
-        $advancedToNewMonth = false;
-
-        // If we got a byDay or getMonthDay filter, we must first expand
-        // further.
-        if ($this->byDay || $this->byMonthDay) {
-            $occurrence = -1;
-            while (true) {
-                $occurrences = $this->getMonthlyOccurrences();
-
-                foreach ($occurrences as $occurrence) {
-                    // The first occurrence that's higher than the current
-                    // day of the month wins.
-                    // If we advanced to the next month or year, the first
-                    // occurrence is always correct.
-                    if ($occurrence > $currentDayOfMonth || $advancedToNewMonth) {
-                        break 2;
-                    }
-                }
-
-                // If we made it here, it means we need to advance to
-                // the next month or year.
-                $currentDayOfMonth = 1;
-                $advancedToNewMonth = true;
-                do {
-                    ++$currentMonth;
-                    if ($currentMonth > 12) {
-                        $currentYear += $this->interval;
-                        $currentMonth = 1;
-                    }
-                } while (!in_array($currentMonth, $this->byMonth));
-
-                $this->currentDate = $this->currentDate->setDate(
-                    (int) $currentYear,
-                    (int) $currentMonth,
-                    (int) $currentDayOfMonth
-                );
-
-                // To prevent running this forever (better: until we hit the max date of DateTimeImmutable) we simply
-                // stop at 9999-12-31. Looks like the year 10000 problem is not solved in php ....
-                if ($this->currentDate->getTimestamp() > self::dateUpperLimit) {
-                    $this->currentDate = null;
-
-                    return;
-                }
-            }
-
-            // If we made it here, it means we got a valid occurrence
-            $this->currentDate = $this->currentDate->setDate(
-                (int) $currentYear,
-                (int) $currentMonth,
-                (int) $occurrence
-            );
-
-            return;
-        } else {
-            // These are the 'byMonth' rules, if there are no byDay or
-            // byMonthDay sub-rules.
-            do {
-                ++$currentMonth;
-                if ($currentMonth > 12) {
-                    $currentYear += $this->interval;
-                    $currentMonth = 1;
-                }
-            } while (!in_array($currentMonth, $this->byMonth));
-            $this->currentDate = $this->currentDate->setDate(
-                (int) $currentYear,
-                (int) $currentMonth,
-                (int) $currentDayOfMonth
-            );
-
-            return;
-        }
-    }
-
-    /* }}} */
-
-    /**
-     * This method receives a string from an RRULE property, and populates this
-     * class with all the values.
-     *
-     * @param string|array $rrule
-     */
-    protected function parseRRule($rrule)
-    {
-        if (is_string($rrule)) {
-            $rrule = Property\ICalendar\Recur::stringToArray($rrule);
-        }
-
-        foreach ($rrule as $key => $value) {
-            $key = strtoupper($key);
-            switch ($key) {
-                case 'FREQ':
-                    $value = strtolower($value);
-                    if (!in_array(
-                        $value,
-                        ['secondly', 'minutely', 'hourly', 'daily', 'weekly', 'monthly', 'yearly']
-                    )) {
-                        throw new InvalidDataException('Unknown value for FREQ='.strtoupper($value));
-                    }
-                    $this->frequency = $value;
-                    break;
-
-                case 'UNTIL':
-                    $this->until = DateTimeParser::parse($value, $this->startDate->getTimezone());
-
-                    // In some cases events are generated with an UNTIL=
-                    // parameter before the actual start of the event.
-                    //
-                    // Not sure why this is happening. We assume that the
-                    // intention was that the event only recurs once.
-                    //
-                    // So we are modifying the parameter so our code doesn't
-                    // break.
-                    if ($this->until < $this->startDate) {
-                        $this->until = $this->startDate;
-                    }
-                    break;
-
-                case 'INTERVAL':
-                case 'COUNT':
-                    $val = (int) $value;
-                    if ($val < 1) {
-                        throw new InvalidDataException(strtoupper($key).' in RRULE must be a positive integer!');
-                    }
-                    $key = strtolower($key);
-                    $this->$key = $val;
-                    break;
-
-                case 'BYSECOND':
-                    $this->bySecond = (array) $value;
-                    break;
-
-                case 'BYMINUTE':
-                    $this->byMinute = (array) $value;
-                    break;
-
-                case 'BYHOUR':
-                    $this->byHour = (array) $value;
-                    break;
-
-                case 'BYDAY':
-                    $value = (array) $value;
-                    foreach ($value as $part) {
-                        if (!preg_match('#^  (-|\+)? ([1-5])? (MO|TU|WE|TH|FR|SA|SU) $# xi', $part)) {
-                            throw new InvalidDataException('Invalid part in BYDAY clause: '.$part);
-                        }
-                    }
-                    $this->byDay = $value;
-                    break;
-
-                case 'BYMONTHDAY':
-                    $this->byMonthDay = (array) $value;
-                    break;
-
-                case 'BYYEARDAY':
-                    $this->byYearDay = (array) $value;
-                    foreach ($this->byYearDay as $byYearDay) {
-                        if (!is_numeric($byYearDay) || (int) $byYearDay < -366 || 0 == (int) $byYearDay || (int) $byYearDay > 366) {
-                            throw new InvalidDataException('BYYEARDAY in RRULE must have value(s) from 1 to 366, or -366 to -1!');
-                        }
-                    }
-                    break;
-
-                case 'BYWEEKNO':
-                    $this->byWeekNo = (array) $value;
-                    foreach ($this->byWeekNo as $byWeekNo) {
-                        if (!is_numeric($byWeekNo) || (int) $byWeekNo < -53 || 0 == (int) $byWeekNo || (int) $byWeekNo > 53) {
-                            throw new InvalidDataException('BYWEEKNO in RRULE must have value(s) from 1 to 53, or -53 to -1!');
-                        }
-                    }
-                    break;
-
-                case 'BYMONTH':
-                    $this->byMonth = (array) $value;
-                    foreach ($this->byMonth as $byMonth) {
-                        if (!is_numeric($byMonth) || (int) $byMonth < 1 || (int) $byMonth > 12) {
-                            throw new InvalidDataException('BYMONTH in RRULE must have value(s) between 1 and 12!');
-                        }
-                    }
-                    break;
-
-                case 'BYSETPOS':
-                    $this->bySetPos = (array) $value;
-                    break;
-
-                case 'WKST':
-                    $this->weekStart = strtoupper($value);
-                    break;
-
-                default:
-                    throw new InvalidDataException('Not supported: '.strtoupper($key));
-            }
-        }
-    }
-
-    /**
-     * Mappings between the day number and english day name.
-     *
-     * @var array
-     */
-    protected $dayNames = [
-        0 => 'Sunday',
-        1 => 'Monday',
-        2 => 'Tuesday',
-        3 => 'Wednesday',
-        4 => 'Thursday',
-        5 => 'Friday',
-        6 => 'Saturday',
-    ];
-
-    /**
-     * Returns all the occurrences for a monthly frequency with a 'byDay' or
-     * 'byMonthDay' expansion for the current month.
-     *
-     * The returned list is an array of integers with the day of month (1-31).
-     *
-     * @return array
-     */
-    protected function getMonthlyOccurrences()
-    {
-        $startDate = clone $this->currentDate;
-
-        $byDayResults = [];
-
-        // Our strategy is to simply go through the byDays, advance the date to
-        // that point and add it to the results.
-        if ($this->byDay) {
-            foreach ($this->byDay as $day) {
-                $dayName = $this->dayNames[$this->dayMap[substr($day, -2)]];
-
-                // Dayname will be something like 'wednesday'. Now we need to find
-                // all wednesdays in this month.
-                $dayHits = [];
-
-                // workaround for missing 'first day of the month' support in hhvm
-                $checkDate = new \DateTime($startDate->format('Y-m-1'));
-                // workaround modify always advancing the date even if the current day is a $dayName in hhvm
-                if ($checkDate->format('l') !== $dayName) {
-                    $checkDate = $checkDate->modify($dayName);
-                }
-
-                do {
-                    $dayHits[] = $checkDate->format('j');
-                    $checkDate = $checkDate->modify('next '.$dayName);
-                } while ($checkDate->format('n') === $startDate->format('n'));
-
-                // So now we have 'all wednesdays' for month. It is however
-                // possible that the user only really wanted the 1st, 2nd or last
-                // wednesday.
-                if (strlen($day) > 2) {
-                    $offset = (int) substr($day, 0, -2);
-
-                    if ($offset > 0) {
-                        // It is possible that the day does not exist, such as a
-                        // 5th or 6th wednesday of the month.
-                        if (isset($dayHits[$offset - 1])) {
-                            $byDayResults[] = $dayHits[$offset - 1];
-                        }
-                    } else {
-                        // if it was negative we count from the end of the array
-                        // might not exist, fx. -5th tuesday
-                        if (isset($dayHits[count($dayHits) + $offset])) {
-                            $byDayResults[] = $dayHits[count($dayHits) + $offset];
-                        }
-                    }
-                } else {
-                    // There was no counter (first, second, last wednesdays), so we
-                    // just need to add the all to the list).
-                    $byDayResults = array_merge($byDayResults, $dayHits);
-                }
-            }
-        }
-
-        $byMonthDayResults = [];
-        if ($this->byMonthDay) {
-            foreach ($this->byMonthDay as $monthDay) {
-                // Removing values that are out of range for this month
-                if ($monthDay > $startDate->format('t') ||
-                    $monthDay < 0 - $startDate->format('t')) {
-                    continue;
-                }
-                if ($monthDay > 0) {
-                    $byMonthDayResults[] = $monthDay;
-                } else {
-                    // Negative values
-                    $byMonthDayResults[] = $startDate->format('t') + 1 + $monthDay;
-                }
-            }
-        }
-
-        // If there was just byDay or just byMonthDay, they just specify our
-        // (almost) final list. If both were provided, then byDay limits the
-        // list.
-        if ($this->byMonthDay && $this->byDay) {
-            $result = array_intersect($byMonthDayResults, $byDayResults);
-        } elseif ($this->byMonthDay) {
-            $result = $byMonthDayResults;
-        } else {
-            $result = $byDayResults;
-        }
-        $result = array_unique($result);
-        sort($result, SORT_NUMERIC);
-
-        // The last thing that needs checking is the BYSETPOS. If it's set, it
-        // means only certain items in the set survive the filter.
-        if (!$this->bySetPos) {
-            return $result;
-        }
-
-        $filteredResult = [];
-        foreach ($this->bySetPos as $setPos) {
-            if ($setPos < 0) {
-                $setPos = count($result) + ($setPos + 1);
-            }
-            if (isset($result[$setPos - 1])) {
-                $filteredResult[] = $result[$setPos - 1];
-            }
-        }
-
-        sort($filteredResult, SORT_NUMERIC);
-
-        return $filteredResult;
-    }
-
-    /**
-     * Simple mapping from iCalendar day names to day numbers.
-     *
-     * @var array
-     */
-    protected $dayMap = [
-        'SU' => 0,
-        'MO' => 1,
-        'TU' => 2,
-        'WE' => 3,
-        'TH' => 4,
-        'FR' => 5,
-        'SA' => 6,
-    ];
-
-    protected function getHours()
-    {
-        $recurrenceHours = [];
-        foreach ($this->byHour as $byHour) {
-            $recurrenceHours[] = $byHour;
-        }
-
-        return $recurrenceHours;
-    }
-
-    protected function getDays()
-    {
-        $recurrenceDays = [];
-        foreach ($this->byDay as $byDay) {
-            // The day may be preceded with a positive (+n) or
-            // negative (-n) integer. However, this does not make
-            // sense in 'weekly' so we ignore it here.
-            $recurrenceDays[] = $this->dayMap[substr($byDay, -2)];
-        }
-
-        return $recurrenceDays;
-    }
-
-    protected function getMonths()
-    {
-        $recurrenceMonths = [];
-        foreach ($this->byMonth as $byMonth) {
-            $recurrenceMonths[] = $byMonth;
-        }
-
-        return $recurrenceMonths;
-    }
+	/**
+	 * Constant denoting the upper limit on how long into the future
+	 * we want to iterate. The value is a unix timestamp and currently
+	 * corresponds to the datetime 9999-12-31 11:59:59 UTC.
+	 */
+	const dateUpperLimit = 253402300799;
+
+	/**
+	 * Creates the Iterator.
+	 *
+	 * @param string|array $rrule
+	 */
+	public function __construct($rrule, DateTimeInterface $start)
+	{
+		$this->startDate = $start;
+		$this->parseRRule($rrule);
+		$this->currentDate = clone $this->startDate;
+	}
+
+	/* Implementation of the Iterator interface {{{ */
+
+	#[\ReturnTypeWillChange]
+	public function current()
+	{
+		if (!$this->valid()) {
+			return;
+		}
+
+		return clone $this->currentDate;
+	}
+
+	/**
+	 * Returns the current item number.
+	 *
+	 * @return int
+	 */
+	#[\ReturnTypeWillChange]
+	public function key()
+	{
+		return $this->counter;
+	}
+
+	/**
+	 * Returns whether the current item is a valid item for the recurrence
+	 * iterator. This will return false if we've gone beyond the UNTIL or COUNT
+	 * statements.
+	 *
+	 * @return bool
+	 */
+	#[\ReturnTypeWillChange]
+	public function valid()
+	{
+		if (null === $this->currentDate) {
+			return false;
+		}
+		if (!is_null($this->count)) {
+			return $this->counter < $this->count;
+		}
+
+		return is_null($this->until) || $this->currentDate <= $this->until;
+	}
+
+	/**
+	 * Resets the iterator.
+	 *
+	 * @return void
+	 */
+	#[\ReturnTypeWillChange]
+	public function rewind()
+	{
+		$this->currentDate = clone $this->startDate;
+		$this->counter = 0;
+	}
+
+	/**
+	 * Goes on to the next iteration.
+	 *
+	 * @return void
+	 */
+	#[\ReturnTypeWillChange]
+	public function next()
+	{
+		// Otherwise, we find the next event in the normal RRULE
+		// sequence.
+		switch ($this->frequency) {
+			case 'hourly':
+				$this->nextHourly();
+				break;
+
+			case 'daily':
+				$this->nextDaily();
+				break;
+
+			case 'weekly':
+				$this->nextWeekly();
+				break;
+
+			case 'monthly':
+				$this->nextMonthly();
+				break;
+
+			case 'yearly':
+				$this->nextYearly();
+				break;
+		}
+		++$this->counter;
+	}
+
+	/* End of Iterator implementation }}} */
+
+	/**
+	 * Returns true if this recurring event never ends.
+	 *
+	 * @return bool
+	 */
+	public function isInfinite()
+	{
+		return !$this->count && !$this->until;
+	}
+
+	/**
+	 * This method allows you to quickly go to the next occurrence after the
+	 * specified date.
+	 */
+	public function fastForward(DateTimeInterface $dt)
+	{
+		while ($this->valid() && $this->currentDate < $dt) {
+			$this->next();
+		}
+	}
+
+	/**
+	 * The reference start date/time for the rrule.
+	 *
+	 * All calculations are based on this initial date.
+	 *
+	 * @var DateTimeInterface
+	 */
+	protected $startDate;
+
+	/**
+	 * The date of the current iteration. You can get this by calling
+	 * ->current().
+	 *
+	 * @var DateTimeInterface
+	 */
+	protected $currentDate;
+
+	/**
+	 * Frequency is one of: secondly, minutely, hourly, daily, weekly, monthly,
+	 * yearly.
+	 *
+	 * @var string
+	 */
+	protected $frequency;
+
+	/**
+	 * The number of recurrences, or 'null' if infinitely recurring.
+	 *
+	 * @var int
+	 */
+	protected $count;
+
+	/**
+	 * The interval.
+	 *
+	 * If for example frequency is set to daily, interval = 2 would mean every
+	 * 2 days.
+	 *
+	 * @var int
+	 */
+	protected $interval = 1;
+
+	/**
+	 * The last instance of this recurrence, inclusively.
+	 *
+	 * @var DateTimeInterface|null
+	 */
+	protected $until;
+
+	/**
+	 * Which seconds to recur.
+	 *
+	 * This is an array of integers (between 0 and 60)
+	 *
+	 * @var array
+	 */
+	protected $bySecond;
+
+	/**
+	 * Which minutes to recur.
+	 *
+	 * This is an array of integers (between 0 and 59)
+	 *
+	 * @var array
+	 */
+	protected $byMinute;
+
+	/**
+	 * Which hours to recur.
+	 *
+	 * This is an array of integers (between 0 and 23)
+	 *
+	 * @var array
+	 */
+	protected $byHour;
+
+	/**
+	 * The current item in the list.
+	 *
+	 * You can get this number with the key() method.
+	 *
+	 * @var int
+	 */
+	protected $counter = 0;
+
+	/**
+	 * Which weekdays to recur.
+	 *
+	 * This is an array of weekdays
+	 *
+	 * This may also be preceded by a positive or negative integer. If present,
+	 * this indicates the nth occurrence of a specific day within the monthly or
+	 * yearly rrule. For instance, -2TU indicates the second-last tuesday of
+	 * the month, or year.
+	 *
+	 * @var array
+	 */
+	protected $byDay;
+
+	/**
+	 * Which days of the month to recur.
+	 *
+	 * This is an array of days of the months (1-31). The value can also be
+	 * negative. -5 for instance means the 5th last day of the month.
+	 *
+	 * @var array
+	 */
+	protected $byMonthDay;
+
+	/**
+	 * Which days of the year to recur.
+	 *
+	 * This is an array with days of the year (1 to 366). The values can also
+	 * be negative. For instance, -1 will always represent the last day of the
+	 * year. (December 31st).
+	 *
+	 * @var array
+	 */
+	protected $byYearDay;
+
+	/**
+	 * Which week numbers to recur.
+	 *
+	 * This is an array of integers from 1 to 53. The values can also be
+	 * negative. -1 will always refer to the last week of the year.
+	 *
+	 * @var array
+	 */
+	protected $byWeekNo;
+
+	/**
+	 * Which months to recur.
+	 *
+	 * This is an array of integers from 1 to 12.
+	 *
+	 * @var array
+	 */
+	protected $byMonth;
+
+	/**
+	 * Which items in an existing st to recur.
+	 *
+	 * These numbers work together with an existing by* rule. It specifies
+	 * exactly which items of the existing by-rule to filter.
+	 *
+	 * Valid values are 1 to 366 and -1 to -366. As an example, this can be
+	 * used to recur the last workday of the month.
+	 *
+	 * This would be done by setting frequency to 'monthly', byDay to
+	 * 'MO,TU,WE,TH,FR' and bySetPos to -1.
+	 *
+	 * @var array
+	 */
+	protected $bySetPos;
+
+	/**
+	 * When the week starts.
+	 *
+	 * @var string
+	 */
+	protected $weekStart = 'MO';
+
+	/* Functions that advance the iterator {{{ */
+
+	/**
+	 * Does the processing for advancing the iterator for hourly frequency.
+	 */
+	protected function nextHourly()
+	{
+		$this->currentDate = $this->currentDate->modify('+'.$this->interval.' hours');
+	}
+
+	/**
+	 * Does the processing for advancing the iterator for daily frequency.
+	 */
+	protected function nextDaily()
+	{
+		if (!$this->byHour && !$this->byDay) {
+			$this->currentDate = $this->currentDate->modify('+'.$this->interval.' days');
+
+			return;
+		}
+
+		$recurrenceHours = [];
+		if (!empty($this->byHour)) {
+			$recurrenceHours = $this->getHours();
+		}
+
+		$recurrenceDays = [];
+		if (!empty($this->byDay)) {
+			$recurrenceDays = $this->getDays();
+		}
+
+		$recurrenceMonths = [];
+		if (!empty($this->byMonth)) {
+			$recurrenceMonths = $this->getMonths();
+		}
+
+		do {
+			if ($this->byHour) {
+				if ('23' == $this->currentDate->format('G')) {
+					// to obey the interval rule
+					$this->currentDate = $this->currentDate->modify('+'.($this->interval - 1).' days');
+				}
+
+				$this->currentDate = $this->currentDate->modify('+1 hours');
+			} else {
+				$this->currentDate = $this->currentDate->modify('+'.$this->interval.' days');
+			}
+
+			// Current month of the year
+			$currentMonth = $this->currentDate->format('n');
+
+			// Current day of the week
+			$currentDay = $this->currentDate->format('w');
+
+			// Current hour of the day
+			$currentHour = $this->currentDate->format('G');
+
+			if ($this->currentDate->getTimestamp() > self::dateUpperLimit) {
+				$this->currentDate = null;
+
+				return;
+			}
+		} while (
+			($this->byDay && !in_array($currentDay, $recurrenceDays)) ||
+			($this->byHour && !in_array($currentHour, $recurrenceHours)) ||
+			($this->byMonth && !in_array($currentMonth, $recurrenceMonths))
+		);
+	}
+
+	/**
+	 * Does the processing for advancing the iterator for weekly frequency.
+	 */
+	protected function nextWeekly()
+	{
+		if (!$this->byHour && !$this->byDay) {
+			$this->currentDate = $this->currentDate->modify('+'.$this->interval.' weeks');
+
+			return;
+		}
+
+		$recurrenceHours = [];
+		if ($this->byHour) {
+			$recurrenceHours = $this->getHours();
+		}
+
+		$recurrenceDays = [];
+		if ($this->byDay) {
+			$recurrenceDays = $this->getDays();
+		}
+
+		// First day of the week:
+		$firstDay = $this->dayMap[$this->weekStart];
+
+		do {
+			if ($this->byHour) {
+				$this->currentDate = $this->currentDate->modify('+1 hours');
+			} else {
+				$this->currentDate = $this->currentDate->modify('+1 days');
+			}
+
+			// Current day of the week
+			$currentDay = (int) $this->currentDate->format('w');
+
+			// Current hour of the day
+			$currentHour = (int) $this->currentDate->format('G');
+
+			// We need to roll over to the next week
+			if ($currentDay === $firstDay && (!$this->byHour || '0' == $currentHour)) {
+				$this->currentDate = $this->currentDate->modify('+'.($this->interval - 1).' weeks');
+
+				// We need to go to the first day of this week, but only if we
+				// are not already on this first day of this week.
+				if ($this->currentDate->format('w') != $firstDay) {
+					$this->currentDate = $this->currentDate->modify('last '.$this->dayNames[$this->dayMap[$this->weekStart]]);
+				}
+			}
+
+			// We have a match
+		} while (($this->byDay && !in_array($currentDay, $recurrenceDays)) || ($this->byHour && !in_array($currentHour, $recurrenceHours)));
+	}
+
+	/**
+	 * Does the processing for advancing the iterator for monthly frequency.
+	 */
+	protected function nextMonthly()
+	{
+		$currentDayOfMonth = $this->currentDate->format('j');
+		if (!$this->byMonthDay && !$this->byDay) {
+			// If the current day is higher than the 28th, rollover can
+			// occur to the next month. We Must skip these invalid
+			// entries.
+			if ($currentDayOfMonth < 29) {
+				$this->currentDate = $this->currentDate->modify('+'.$this->interval.' months');
+			} else {
+				$increase = 0;
+				do {
+					++$increase;
+					$tempDate = clone $this->currentDate;
+					$tempDate = $tempDate->modify('+ '.($this->interval * $increase).' months');
+				} while ($tempDate->format('j') != $currentDayOfMonth);
+				$this->currentDate = $tempDate;
+			}
+
+			return;
+		}
+
+		$occurrence = -1;
+		while (true) {
+			$occurrences = $this->getMonthlyOccurrences();
+
+			foreach ($occurrences as $occurrence) {
+				// The first occurrence thats higher than the current
+				// day of the month wins.
+				if ($occurrence > $currentDayOfMonth) {
+					break 2;
+				}
+			}
+
+			// If we made it all the way here, it means there were no
+			// valid occurrences, and we need to advance to the next
+			// month.
+			//
+			// This line does not currently work in hhvm. Temporary workaround
+			// follows:
+			// $this->currentDate->modify('first day of this month');
+			$this->currentDate = new DateTimeImmutable($this->currentDate->format('Y-m-1 H:i:s'), $this->currentDate->getTimezone());
+			// end of workaround
+			$this->currentDate = $this->currentDate->modify('+ '.$this->interval.' months');
+
+			// This goes to 0 because we need to start counting at the
+			// beginning.
+			$currentDayOfMonth = 0;
+
+			// For some reason the "until" parameter was not being used here,
+			// that's why the workaround of the 10000 year bug was needed at all
+			// let's stop it before the "until" parameter date
+			if ($this->until && $this->currentDate->getTimestamp() >= $this->until->getTimestamp()) {
+				return;
+			}
+
+			// To prevent running this forever (better: until we hit the max date of DateTimeImmutable) we simply
+			// stop at 9999-12-31. Looks like the year 10000 problem is not solved in php ....
+			if ($this->currentDate->getTimestamp() > self::dateUpperLimit) {
+				$this->currentDate = null;
+
+				return;
+			}
+		}
+
+		$this->currentDate = $this->currentDate->setDate(
+			(int) $this->currentDate->format('Y'),
+			(int) $this->currentDate->format('n'),
+			(int) $occurrence
+		);
+	}
+
+	/**
+	 * Does the processing for advancing the iterator for yearly frequency.
+	 */
+	protected function nextYearly()
+	{
+		$currentMonth = $this->currentDate->format('n');
+		$currentYear = $this->currentDate->format('Y');
+		$currentDayOfMonth = $this->currentDate->format('j');
+
+		// No sub-rules, so we just advance by year
+		if (empty($this->byMonth)) {
+			// Unless it was a leap day!
+			if (2 == $currentMonth && 29 == $currentDayOfMonth) {
+				$counter = 0;
+				do {
+					++$counter;
+					// Here we increase the year count by the interval, until
+					// we hit a date that's also in a leap year.
+					//
+					// We could just find the next interval that's dividable by
+					// 4, but that would ignore the rule that there's no leap
+					// year every year that's dividable by a 100, but not by
+					// 400. (1800, 1900, 2100). So we just rely on the datetime
+					// functions instead.
+					$nextDate = clone $this->currentDate;
+					$nextDate = $nextDate->modify('+ '.($this->interval * $counter).' years');
+				} while (2 != $nextDate->format('n'));
+
+				$this->currentDate = $nextDate;
+
+				return;
+			}
+
+			if (null !== $this->byWeekNo) { // byWeekNo is an array with values from -53 to -1, or 1 to 53
+				$dayOffsets = [];
+				if ($this->byDay) {
+					foreach ($this->byDay as $byDay) {
+						$dayOffsets[] = $this->dayMap[$byDay];
+					}
+				} else {   // default is Monday
+					$dayOffsets[] = 1;
+				}
+
+				$currentYear = $this->currentDate->format('Y');
+
+				while (true) {
+					$checkDates = [];
+
+					// loop through all WeekNo and Days to check all the combinations
+					foreach ($this->byWeekNo as $byWeekNo) {
+						foreach ($dayOffsets as $dayOffset) {
+							$date = clone $this->currentDate;
+							$date = $date->setISODate($currentYear, $byWeekNo, $dayOffset);
+
+							if ($date > $this->currentDate) {
+								$checkDates[] = $date;
+							}
+						}
+					}
+
+					if (count($checkDates) > 0) {
+						$this->currentDate = min($checkDates);
+
+						return;
+					}
+
+					// if there is no date found, check the next year
+					$currentYear += $this->interval;
+				}
+			}
+
+			if (null !== $this->byYearDay) { // byYearDay is an array with values from -366 to -1, or 1 to 366
+				$dayOffsets = [];
+				if ($this->byDay) {
+					foreach ($this->byDay as $byDay) {
+						$dayOffsets[] = $this->dayMap[$byDay];
+					}
+				} else {   // default is Monday-Sunday
+					$dayOffsets = [1, 2, 3, 4, 5, 6, 7];
+				}
+
+				$currentYear = $this->currentDate->format('Y');
+
+				while (true) {
+					$checkDates = [];
+
+					// loop through all YearDay and Days to check all the combinations
+					foreach ($this->byYearDay as $byYearDay) {
+						$date = clone $this->currentDate;
+						if ($byYearDay > 0) {
+							$date = $date->setDate($currentYear, 1, 1);
+							$date = $date->add(new \DateInterval('P'.($byYearDay - 1).'D'));
+						} else {
+							$date = $date->setDate($currentYear, 12, 31);
+							$date = $date->sub(new \DateInterval('P'.abs($byYearDay + 1).'D'));
+						}
+
+						if ($date > $this->currentDate && in_array($date->format('N'), $dayOffsets)) {
+							$checkDates[] = $date;
+						}
+					}
+
+					if (count($checkDates) > 0) {
+						$this->currentDate = min($checkDates);
+
+						return;
+					}
+
+					// if there is no date found, check the next year
+					$currentYear += $this->interval;
+				}
+			}
+
+			// The easiest form
+			$this->currentDate = $this->currentDate->modify('+'.$this->interval.' years');
+
+			return;
+		}
+
+		$currentMonth = $this->currentDate->format('n');
+		$currentYear = $this->currentDate->format('Y');
+		$currentDayOfMonth = $this->currentDate->format('j');
+
+		$advancedToNewMonth = false;
+
+		// If we got a byDay or getMonthDay filter, we must first expand
+		// further.
+		if ($this->byDay || $this->byMonthDay) {
+			$occurrence = -1;
+			while (true) {
+				$occurrences = $this->getMonthlyOccurrences();
+
+				foreach ($occurrences as $occurrence) {
+					// The first occurrence that's higher than the current
+					// day of the month wins.
+					// If we advanced to the next month or year, the first
+					// occurrence is always correct.
+					if ($occurrence > $currentDayOfMonth || $advancedToNewMonth) {
+						break 2;
+					}
+				}
+
+				// If we made it here, it means we need to advance to
+				// the next month or year.
+				$currentDayOfMonth = 1;
+				$advancedToNewMonth = true;
+				do {
+					++$currentMonth;
+					if ($currentMonth > 12) {
+						$currentYear += $this->interval;
+						$currentMonth = 1;
+					}
+				} while (!in_array($currentMonth, $this->byMonth));
+
+				$this->currentDate = $this->currentDate->setDate(
+					(int) $currentYear,
+					(int) $currentMonth,
+					(int) $currentDayOfMonth
+				);
+
+				// To prevent running this forever (better: until we hit the max date of DateTimeImmutable) we simply
+				// stop at 9999-12-31. Looks like the year 10000 problem is not solved in php ....
+				if ($this->currentDate->getTimestamp() > self::dateUpperLimit) {
+					$this->currentDate = null;
+
+					return;
+				}
+			}
+
+			// If we made it here, it means we got a valid occurrence
+			$this->currentDate = $this->currentDate->setDate(
+				(int) $currentYear,
+				(int) $currentMonth,
+				(int) $occurrence
+			);
+
+			return;
+		} else {
+			// These are the 'byMonth' rules, if there are no byDay or
+			// byMonthDay sub-rules.
+			do {
+				++$currentMonth;
+				if ($currentMonth > 12) {
+					$currentYear += $this->interval;
+					$currentMonth = 1;
+				}
+			} while (!in_array($currentMonth, $this->byMonth));
+			$this->currentDate = $this->currentDate->setDate(
+				(int) $currentYear,
+				(int) $currentMonth,
+				(int) $currentDayOfMonth
+			);
+
+			return;
+		}
+	}
+
+	/* }}} */
+
+	/**
+	 * This method receives a string from an RRULE property, and populates this
+	 * class with all the values.
+	 *
+	 * @param string|array $rrule
+	 */
+	protected function parseRRule($rrule)
+	{
+		if (is_string($rrule)) {
+			$rrule = Property\ICalendar\Recur::stringToArray($rrule);
+		}
+
+		foreach ($rrule as $key => $value) {
+			$key = strtoupper($key);
+			switch ($key) {
+				case 'FREQ':
+					$value = strtolower($value);
+					if (!in_array(
+						$value,
+						['secondly', 'minutely', 'hourly', 'daily', 'weekly', 'monthly', 'yearly']
+					)) {
+						throw new InvalidDataException('Unknown value for FREQ='.strtoupper($value));
+					}
+					$this->frequency = $value;
+					break;
+
+				case 'UNTIL':
+					$this->until = DateTimeParser::parse($value, $this->startDate->getTimezone());
+
+					// In some cases events are generated with an UNTIL=
+					// parameter before the actual start of the event.
+					//
+					// Not sure why this is happening. We assume that the
+					// intention was that the event only recurs once.
+					//
+					// So we are modifying the parameter so our code doesn't
+					// break.
+					if ($this->until < $this->startDate) {
+						$this->until = $this->startDate;
+					}
+					break;
+
+				case 'INTERVAL':
+				case 'COUNT':
+					$val = (int) $value;
+					if ($val < 1) {
+						throw new InvalidDataException(strtoupper($key).' in RRULE must be a positive integer!');
+					}
+					$key = strtolower($key);
+					$this->$key = $val;
+					break;
+
+				case 'BYSECOND':
+					$this->bySecond = (array) $value;
+					break;
+
+				case 'BYMINUTE':
+					$this->byMinute = (array) $value;
+					break;
+
+				case 'BYHOUR':
+					$this->byHour = (array) $value;
+					break;
+
+				case 'BYDAY':
+					$value = (array) $value;
+					foreach ($value as $part) {
+						if (!preg_match('#^  (-|\+)? ([1-5])? (MO|TU|WE|TH|FR|SA|SU) $# xi', $part)) {
+							throw new InvalidDataException('Invalid part in BYDAY clause: '.$part);
+						}
+					}
+					$this->byDay = $value;
+					break;
+
+				case 'BYMONTHDAY':
+					$this->byMonthDay = (array) $value;
+					break;
+
+				case 'BYYEARDAY':
+					$this->byYearDay = (array) $value;
+					foreach ($this->byYearDay as $byYearDay) {
+						if (!is_numeric($byYearDay) || (int) $byYearDay < -366 || 0 == (int) $byYearDay || (int) $byYearDay > 366) {
+							throw new InvalidDataException('BYYEARDAY in RRULE must have value(s) from 1 to 366, or -366 to -1!');
+						}
+					}
+					break;
+
+				case 'BYWEEKNO':
+					$this->byWeekNo = (array) $value;
+					foreach ($this->byWeekNo as $byWeekNo) {
+						if (!is_numeric($byWeekNo) || (int) $byWeekNo < -53 || 0 == (int) $byWeekNo || (int) $byWeekNo > 53) {
+							throw new InvalidDataException('BYWEEKNO in RRULE must have value(s) from 1 to 53, or -53 to -1!');
+						}
+					}
+					break;
+
+				case 'BYMONTH':
+					$this->byMonth = (array) $value;
+					foreach ($this->byMonth as $byMonth) {
+						if (!is_numeric($byMonth) || (int) $byMonth < 1 || (int) $byMonth > 12) {
+							throw new InvalidDataException('BYMONTH in RRULE must have value(s) between 1 and 12!');
+						}
+					}
+					break;
+
+				case 'BYSETPOS':
+					$this->bySetPos = (array) $value;
+					break;
+
+				case 'WKST':
+					$this->weekStart = strtoupper($value);
+					break;
+
+				default:
+					throw new InvalidDataException('Not supported: '.strtoupper($key));
+			}
+		}
+	}
+
+	/**
+	 * Mappings between the day number and english day name.
+	 *
+	 * @var array
+	 */
+	protected $dayNames = [
+		0 => 'Sunday',
+		1 => 'Monday',
+		2 => 'Tuesday',
+		3 => 'Wednesday',
+		4 => 'Thursday',
+		5 => 'Friday',
+		6 => 'Saturday',
+	];
+
+	/**
+	 * Returns all the occurrences for a monthly frequency with a 'byDay' or
+	 * 'byMonthDay' expansion for the current month.
+	 *
+	 * The returned list is an array of integers with the day of month (1-31).
+	 *
+	 * @return array
+	 */
+	protected function getMonthlyOccurrences()
+	{
+		$startDate = clone $this->currentDate;
+
+		$byDayResults = [];
+
+		// Our strategy is to simply go through the byDays, advance the date to
+		// that point and add it to the results.
+		if ($this->byDay) {
+			foreach ($this->byDay as $day) {
+				$dayName = $this->dayNames[$this->dayMap[substr($day, -2)]];
+
+				// Dayname will be something like 'wednesday'. Now we need to find
+				// all wednesdays in this month.
+				$dayHits = [];
+
+				// workaround for missing 'first day of the month' support in hhvm
+				$checkDate = new \DateTime($startDate->format('Y-m-1'));
+				// workaround modify always advancing the date even if the current day is a $dayName in hhvm
+				if ($checkDate->format('l') !== $dayName) {
+					$checkDate = $checkDate->modify($dayName);
+				}
+
+				do {
+					$dayHits[] = $checkDate->format('j');
+					$checkDate = $checkDate->modify('next '.$dayName);
+				} while ($checkDate->format('n') === $startDate->format('n'));
+
+				// So now we have 'all wednesdays' for month. It is however
+				// possible that the user only really wanted the 1st, 2nd or last
+				// wednesday.
+				if (strlen($day) > 2) {
+					$offset = (int) substr($day, 0, -2);
+
+					if ($offset > 0) {
+						// It is possible that the day does not exist, such as a
+						// 5th or 6th wednesday of the month.
+						if (isset($dayHits[$offset - 1])) {
+							$byDayResults[] = $dayHits[$offset - 1];
+						}
+					} else {
+						// if it was negative we count from the end of the array
+						// might not exist, fx. -5th tuesday
+						if (isset($dayHits[count($dayHits) + $offset])) {
+							$byDayResults[] = $dayHits[count($dayHits) + $offset];
+						}
+					}
+				} else {
+					// There was no counter (first, second, last wednesdays), so we
+					// just need to add the all to the list).
+					$byDayResults = array_merge($byDayResults, $dayHits);
+				}
+			}
+		}
+
+		$byMonthDayResults = [];
+		if ($this->byMonthDay) {
+			foreach ($this->byMonthDay as $monthDay) {
+				// Removing values that are out of range for this month
+				if ($monthDay > $startDate->format('t') ||
+					$monthDay < 0 - $startDate->format('t')) {
+					continue;
+				}
+				if ($monthDay > 0) {
+					$byMonthDayResults[] = $monthDay;
+				} else {
+					// Negative values
+					$byMonthDayResults[] = $startDate->format('t') + 1 + $monthDay;
+				}
+			}
+		}
+
+		// If there was just byDay or just byMonthDay, they just specify our
+		// (almost) final list. If both were provided, then byDay limits the
+		// list.
+		if ($this->byMonthDay && $this->byDay) {
+			$result = array_intersect($byMonthDayResults, $byDayResults);
+		} elseif ($this->byMonthDay) {
+			$result = $byMonthDayResults;
+		} else {
+			$result = $byDayResults;
+		}
+		$result = array_unique($result);
+		sort($result, SORT_NUMERIC);
+
+		// The last thing that needs checking is the BYSETPOS. If it's set, it
+		// means only certain items in the set survive the filter.
+		if (!$this->bySetPos) {
+			return $result;
+		}
+
+		$filteredResult = [];
+		foreach ($this->bySetPos as $setPos) {
+			if ($setPos < 0) {
+				$setPos = count($result) + ($setPos + 1);
+			}
+			if (isset($result[$setPos - 1])) {
+				$filteredResult[] = $result[$setPos - 1];
+			}
+		}
+
+		sort($filteredResult, SORT_NUMERIC);
+
+		return $filteredResult;
+	}
+
+	/**
+	 * Simple mapping from iCalendar day names to day numbers.
+	 *
+	 * @var array
+	 */
+	protected $dayMap = [
+		'SU' => 0,
+		'MO' => 1,
+		'TU' => 2,
+		'WE' => 3,
+		'TH' => 4,
+		'FR' => 5,
+		'SA' => 6,
+	];
+
+	protected function getHours()
+	{
+		$recurrenceHours = [];
+		foreach ($this->byHour as $byHour) {
+			$recurrenceHours[] = $byHour;
+		}
+
+		return $recurrenceHours;
+	}
+
+	protected function getDays()
+	{
+		$recurrenceDays = [];
+		foreach ($this->byDay as $byDay) {
+			// The day may be preceded with a positive (+n) or
+			// negative (-n) integer. However, this does not make
+			// sense in 'weekly' so we ignore it here.
+			$recurrenceDays[] = $this->dayMap[substr($byDay, -2)];
+		}
+
+		return $recurrenceDays;
+	}
+
+	protected function getMonths()
+	{
+		$recurrenceMonths = [];
+		foreach ($this->byMonth as $byMonth) {
+			$recurrenceMonths[] = $byMonth;
+		}
+
+		return $recurrenceMonths;
+	}
 }

htdocs/includes/sabre/sabre/vobject/lib/Recur/RDateIterator.php

Indentation +151 added lines, -151 removed lines

@@ -21,155 +21,155 @@
  */
 class RDateIterator implements Iterator
 {
-    /**
-     * Creates the Iterator.
-     *
-     * @param string|array $rrule
-     */
-    public function __construct($rrule, DateTimeInterface $start)
-    {
-        $this->startDate = $start;
-        $this->parseRDate($rrule);
-        $this->currentDate = clone $this->startDate;
-    }
-
-    /* Implementation of the Iterator interface {{{ */
-
-    #[\ReturnTypeWillChange]
-    public function current()
-    {
-        if (!$this->valid()) {
-            return;
-        }
-
-        return clone $this->currentDate;
-    }
-
-    /**
-     * Returns the current item number.
-     *
-     * @return int
-     */
-    #[\ReturnTypeWillChange]
-    public function key()
-    {
-        return $this->counter;
-    }
-
-    /**
-     * Returns whether the current item is a valid item for the recurrence
-     * iterator.
-     *
-     * @return bool
-     */
-    #[\ReturnTypeWillChange]
-    public function valid()
-    {
-        return $this->counter <= count($this->dates);
-    }
-
-    /**
-     * Resets the iterator.
-     *
-     * @return void
-     */
-    #[\ReturnTypeWillChange]
-    public function rewind()
-    {
-        $this->currentDate = clone $this->startDate;
-        $this->counter = 0;
-    }
-
-    /**
-     * Goes on to the next iteration.
-     *
-     * @return void
-     */
-    #[\ReturnTypeWillChange]
-    public function next()
-    {
-        ++$this->counter;
-        if (!$this->valid()) {
-            return;
-        }
-
-        $this->currentDate =
-            DateTimeParser::parse(
-                $this->dates[$this->counter - 1],
-                $this->startDate->getTimezone()
-            );
-    }
-
-    /* End of Iterator implementation }}} */
-
-    /**
-     * Returns true if this recurring event never ends.
-     *
-     * @return bool
-     */
-    public function isInfinite()
-    {
-        return false;
-    }
-
-    /**
-     * This method allows you to quickly go to the next occurrence after the
-     * specified date.
-     */
-    public function fastForward(DateTimeInterface $dt)
-    {
-        while ($this->valid() && $this->currentDate < $dt) {
-            $this->next();
-        }
-    }
-
-    /**
-     * The reference start date/time for the rrule.
-     *
-     * All calculations are based on this initial date.
-     *
-     * @var DateTimeInterface
-     */
-    protected $startDate;
-
-    /**
-     * The date of the current iteration. You can get this by calling
-     * ->current().
-     *
-     * @var DateTimeInterface
-     */
-    protected $currentDate;
-
-    /**
-     * The current item in the list.
-     *
-     * You can get this number with the key() method.
-     *
-     * @var int
-     */
-    protected $counter = 0;
-
-    /* }}} */
-
-    /**
-     * This method receives a string from an RRULE property, and populates this
-     * class with all the values.
-     *
-     * @param string|array $rrule
-     */
-    protected function parseRDate($rdate)
-    {
-        if (is_string($rdate)) {
-            $rdate = explode(',', $rdate);
-        }
-
-        $this->dates = $rdate;
-    }
-
-    /**
-     * Array with the RRULE dates.
-     *
-     * @var array
-     */
-    protected $dates = [];
+	/**
+	 * Creates the Iterator.
+	 *
+	 * @param string|array $rrule
+	 */
+	public function __construct($rrule, DateTimeInterface $start)
+	{
+		$this->startDate = $start;
+		$this->parseRDate($rrule);
+		$this->currentDate = clone $this->startDate;
+	}
+
+	/* Implementation of the Iterator interface {{{ */
+
+	#[\ReturnTypeWillChange]
+	public function current()
+	{
+		if (!$this->valid()) {
+			return;
+		}
+
+		return clone $this->currentDate;
+	}
+
+	/**
+	 * Returns the current item number.
+	 *
+	 * @return int
+	 */
+	#[\ReturnTypeWillChange]
+	public function key()
+	{
+		return $this->counter;
+	}
+
+	/**
+	 * Returns whether the current item is a valid item for the recurrence
+	 * iterator.
+	 *
+	 * @return bool
+	 */
+	#[\ReturnTypeWillChange]
+	public function valid()
+	{
+		return $this->counter <= count($this->dates);
+	}
+
+	/**
+	 * Resets the iterator.
+	 *
+	 * @return void
+	 */
+	#[\ReturnTypeWillChange]
+	public function rewind()
+	{
+		$this->currentDate = clone $this->startDate;
+		$this->counter = 0;
+	}
+
+	/**
+	 * Goes on to the next iteration.
+	 *
+	 * @return void
+	 */
+	#[\ReturnTypeWillChange]
+	public function next()
+	{
+		++$this->counter;
+		if (!$this->valid()) {
+			return;
+		}
+
+		$this->currentDate =
+			DateTimeParser::parse(
+				$this->dates[$this->counter - 1],
+				$this->startDate->getTimezone()
+			);
+	}
+
+	/* End of Iterator implementation }}} */
+
+	/**
+	 * Returns true if this recurring event never ends.
+	 *
+	 * @return bool
+	 */
+	public function isInfinite()
+	{
+		return false;
+	}
+
+	/**
+	 * This method allows you to quickly go to the next occurrence after the
+	 * specified date.
+	 */
+	public function fastForward(DateTimeInterface $dt)
+	{
+		while ($this->valid() && $this->currentDate < $dt) {
+			$this->next();
+		}
+	}
+
+	/**
+	 * The reference start date/time for the rrule.
+	 *
+	 * All calculations are based on this initial date.
+	 *
+	 * @var DateTimeInterface
+	 */
+	protected $startDate;
+
+	/**
+	 * The date of the current iteration. You can get this by calling
+	 * ->current().
+	 *
+	 * @var DateTimeInterface
+	 */
+	protected $currentDate;
+
+	/**
+	 * The current item in the list.
+	 *
+	 * You can get this number with the key() method.
+	 *
+	 * @var int
+	 */
+	protected $counter = 0;
+
+	/* }}} */
+
+	/**
+	 * This method receives a string from an RRULE property, and populates this
+	 * class with all the values.
+	 *
+	 * @param string|array $rrule
+	 */
+	protected function parseRDate($rdate)
+	{
+		if (is_string($rdate)) {
+			$rdate = explode(',', $rdate);
+		}
+
+		$this->dates = $rdate;
+	}
+
+	/**
+	 * Array with the RRULE dates.
+	 *
+	 * @var array
+	 */
+	protected $dates = [];
 }

htdocs/includes/sabre/sabre/vobject/lib/Property/Unknown.php

Indentation +23 added lines, -23 removed lines

@@ -14,28 +14,28 @@
  */
 class Unknown extends Text
 {
-    /**
-     * Returns the value, in the format it should be encoded for json.
-     *
-     * This method must always return an array.
-     *
-     * @return array
-     */
-    public function getJsonValue()
-    {
-        return [$this->getRawMimeDirValue()];
-    }
+	/**
+	 * Returns the value, in the format it should be encoded for json.
+	 *
+	 * This method must always return an array.
+	 *
+	 * @return array
+	 */
+	public function getJsonValue()
+	{
+		return [$this->getRawMimeDirValue()];
+	}
 
-    /**
-     * Returns the type of value.
-     *
-     * This corresponds to the VALUE= parameter. Every property also has a
-     * 'default' valueType.
-     *
-     * @return string
-     */
-    public function getValueType()
-    {
-        return 'UNKNOWN';
-    }
+	/**
+	 * Returns the type of value.
+	 *
+	 * This corresponds to the VALUE= parameter. Every property also has a
+	 * 'default' valueType.
+	 *
+	 * @return string
+	 */
+	public function getValueType()
+	{
+		return 'UNKNOWN';
+	}
 }

htdocs/includes/sabre/sabre/vobject/lib/Property/Binary.php

Indentation +80 added lines, -80 removed lines

@@ -20,90 +20,90 @@
  */
 class Binary extends Property
 {
-    /**
-     * In case this is a multi-value property. This string will be used as a
-     * delimiter.
-     *
-     * @var string
-     */
-    public $delimiter = '';
+	/**
+	 * In case this is a multi-value property. This string will be used as a
+	 * delimiter.
+	 *
+	 * @var string
+	 */
+	public $delimiter = '';
 
-    /**
-     * Updates the current value.
-     *
-     * This may be either a single, or multiple strings in an array.
-     *
-     * @param string|array $value
-     */
-    public function setValue($value)
-    {
-        if (is_array($value)) {
-            if (1 === count($value)) {
-                $this->value = $value[0];
-            } else {
-                throw new \InvalidArgumentException('The argument must either be a string or an array with only one child');
-            }
-        } else {
-            $this->value = $value;
-        }
-    }
+	/**
+	 * Updates the current value.
+	 *
+	 * This may be either a single, or multiple strings in an array.
+	 *
+	 * @param string|array $value
+	 */
+	public function setValue($value)
+	{
+		if (is_array($value)) {
+			if (1 === count($value)) {
+				$this->value = $value[0];
+			} else {
+				throw new \InvalidArgumentException('The argument must either be a string or an array with only one child');
+			}
+		} else {
+			$this->value = $value;
+		}
+	}
 
-    /**
-     * Sets a raw value coming from a mimedir (iCalendar/vCard) file.
-     *
-     * This has been 'unfolded', so only 1 line will be passed. Unescaping is
-     * not yet done, but parameters are not included.
-     *
-     * @param string $val
-     */
-    public function setRawMimeDirValue($val)
-    {
-        $this->value = base64_decode($val);
-    }
+	/**
+	 * Sets a raw value coming from a mimedir (iCalendar/vCard) file.
+	 *
+	 * This has been 'unfolded', so only 1 line will be passed. Unescaping is
+	 * not yet done, but parameters are not included.
+	 *
+	 * @param string $val
+	 */
+	public function setRawMimeDirValue($val)
+	{
+		$this->value = base64_decode($val);
+	}
 
-    /**
-     * Returns a raw mime-dir representation of the value.
-     *
-     * @return string
-     */
-    public function getRawMimeDirValue()
-    {
-        return base64_encode($this->value);
-    }
+	/**
+	 * Returns a raw mime-dir representation of the value.
+	 *
+	 * @return string
+	 */
+	public function getRawMimeDirValue()
+	{
+		return base64_encode($this->value);
+	}
 
-    /**
-     * Returns the type of value.
-     *
-     * This corresponds to the VALUE= parameter. Every property also has a
-     * 'default' valueType.
-     *
-     * @return string
-     */
-    public function getValueType()
-    {
-        return 'BINARY';
-    }
+	/**
+	 * Returns the type of value.
+	 *
+	 * This corresponds to the VALUE= parameter. Every property also has a
+	 * 'default' valueType.
+	 *
+	 * @return string
+	 */
+	public function getValueType()
+	{
+		return 'BINARY';
+	}
 
-    /**
-     * Returns the value, in the format it should be encoded for json.
-     *
-     * This method must always return an array.
-     *
-     * @return array
-     */
-    public function getJsonValue()
-    {
-        return [base64_encode($this->getValue())];
-    }
+	/**
+	 * Returns the value, in the format it should be encoded for json.
+	 *
+	 * This method must always return an array.
+	 *
+	 * @return array
+	 */
+	public function getJsonValue()
+	{
+		return [base64_encode($this->getValue())];
+	}
 
-    /**
-     * Sets the json value, as it would appear in a jCard or jCal object.
-     *
-     * The value must always be an array.
-     */
-    public function setJsonValue(array $value)
-    {
-        $value = array_map('base64_decode', $value);
-        parent::setJsonValue($value);
-    }
+	/**
+	 * Sets the json value, as it would appear in a jCard or jCal object.
+	 *
+	 * The value must always be an array.
+	 */
+	public function setJsonValue(array $value)
+	{
+		$value = array_map('base64_decode', $value);
+		parent::setJsonValue($value);
+	}
 }

htdocs/includes/sabre/sabre/vobject/lib/Property/IntegerValue.php

Indentation +53 added lines, -53 removed lines

@@ -16,61 +16,61 @@
  */
 class IntegerValue extends Property
 {
-    /**
-     * Sets a raw value coming from a mimedir (iCalendar/vCard) file.
-     *
-     * This has been 'unfolded', so only 1 line will be passed. Unescaping is
-     * not yet done, but parameters are not included.
-     *
-     * @param string $val
-     */
-    public function setRawMimeDirValue($val)
-    {
-        $this->setValue((int) $val);
-    }
+	/**
+	 * Sets a raw value coming from a mimedir (iCalendar/vCard) file.
+	 *
+	 * This has been 'unfolded', so only 1 line will be passed. Unescaping is
+	 * not yet done, but parameters are not included.
+	 *
+	 * @param string $val
+	 */
+	public function setRawMimeDirValue($val)
+	{
+		$this->setValue((int) $val);
+	}
 
-    /**
-     * Returns a raw mime-dir representation of the value.
-     *
-     * @return string
-     */
-    public function getRawMimeDirValue()
-    {
-        return $this->value;
-    }
+	/**
+	 * Returns a raw mime-dir representation of the value.
+	 *
+	 * @return string
+	 */
+	public function getRawMimeDirValue()
+	{
+		return $this->value;
+	}
 
-    /**
-     * Returns the type of value.
-     *
-     * This corresponds to the VALUE= parameter. Every property also has a
-     * 'default' valueType.
-     *
-     * @return string
-     */
-    public function getValueType()
-    {
-        return 'INTEGER';
-    }
+	/**
+	 * Returns the type of value.
+	 *
+	 * This corresponds to the VALUE= parameter. Every property also has a
+	 * 'default' valueType.
+	 *
+	 * @return string
+	 */
+	public function getValueType()
+	{
+		return 'INTEGER';
+	}
 
-    /**
-     * Returns the value, in the format it should be encoded for json.
-     *
-     * This method must always return an array.
-     *
-     * @return array
-     */
-    public function getJsonValue()
-    {
-        return [(int) $this->getValue()];
-    }
+	/**
+	 * Returns the value, in the format it should be encoded for json.
+	 *
+	 * This method must always return an array.
+	 *
+	 * @return array
+	 */
+	public function getJsonValue()
+	{
+		return [(int) $this->getValue()];
+	}
 
-    /**
-     * Hydrate data from a XML subtree, as it would appear in a xCard or xCal
-     * object.
-     */
-    public function setXmlValue(array $value)
-    {
-        $value = array_map('intval', $value);
-        parent::setXmlValue($value);
-    }
+	/**
+	 * Hydrate data from a XML subtree, as it would appear in a xCard or xCal
+	 * object.
+	 */
+	public function setXmlValue(array $value)
+	{
+		$value = array_map('intval', $value);
+		parent::setXmlValue($value);
+	}
 }

htdocs/includes/sabre/sabre/vobject/lib/Property/Text.php

Indentation +366 added lines, -366 removed lines

@@ -19,374 +19,374 @@
  */
 class Text extends Property
 {
-    /**
-     * In case this is a multi-value property. This string will be used as a
-     * delimiter.
-     *
-     * @var string
-     */
-    public $delimiter = ',';
-
-    /**
-     * List of properties that are considered 'structured'.
-     *
-     * @var array
-     */
-    protected $structuredValues = [
-        // vCard
-        'N',
-        'ADR',
-        'ORG',
-        'GENDER',
-        'CLIENTPIDMAP',
-
-        // iCalendar
-        'REQUEST-STATUS',
-    ];
-
-    /**
-     * Some text components have a minimum number of components.
-     *
-     * N must for instance be represented as 5 components, separated by ;, even
-     * if the last few components are unused.
-     *
-     * @var array
-     */
-    protected $minimumPropertyValues = [
-        'N' => 5,
-        'ADR' => 7,
-    ];
-
-    /**
-     * Creates the property.
-     *
-     * You can specify the parameters either in key=>value syntax, in which case
-     * parameters will automatically be created, or you can just pass a list of
-     * Parameter objects.
-     *
-     * @param Component         $root       The root document
-     * @param string            $name
-     * @param string|array|null $value
-     * @param array             $parameters List of parameters
-     * @param string            $group      The vcard property group
-     */
-    public function __construct(Component $root, $name, $value = null, array $parameters = [], $group = null)
-    {
-        // There's two types of multi-valued text properties:
-        // 1. multivalue properties.
-        // 2. structured value properties
-        //
-        // The former is always separated by a comma, the latter by semi-colon.
-        if (in_array($name, $this->structuredValues)) {
-            $this->delimiter = ';';
-        }
-
-        parent::__construct($root, $name, $value, $parameters, $group);
-    }
-
-    /**
-     * Sets a raw value coming from a mimedir (iCalendar/vCard) file.
-     *
-     * This has been 'unfolded', so only 1 line will be passed. Unescaping is
-     * not yet done, but parameters are not included.
-     *
-     * @param string $val
-     */
-    public function setRawMimeDirValue($val)
-    {
-        $this->setValue(MimeDir::unescapeValue($val, $this->delimiter));
-    }
-
-    /**
-     * Sets the value as a quoted-printable encoded string.
-     *
-     * @param string $val
-     */
-    public function setQuotedPrintableValue($val)
-    {
-        $val = quoted_printable_decode($val);
-
-        // Quoted printable only appears in vCard 2.1, and the only character
-        // that may be escaped there is ;. So we are simply splitting on just
-        // that.
-        //
-        // We also don't have to unescape \\, so all we need to look for is a ;
-        // that's not preceded with a \.
-        $regex = '# (?<!\\\\) ; #x';
-        $matches = preg_split($regex, $val);
-        $this->setValue($matches);
-    }
-
-    /**
-     * Returns a raw mime-dir representation of the value.
-     *
-     * @return string
-     */
-    public function getRawMimeDirValue()
-    {
-        $val = $this->getParts();
-
-        if (isset($this->minimumPropertyValues[$this->name])) {
-            $val = array_pad($val, $this->minimumPropertyValues[$this->name], '');
-        }
-
-        foreach ($val as &$item) {
-            if (!is_array($item)) {
-                $item = [$item];
-            }
-
-            foreach ($item as &$subItem) {
-                if (!is_null($subItem)) {
-                    $subItem = strtr(
-                        $subItem,
-                        [
-                            '\\' => '\\\\',
-                            ';' => '\;',
-                            ',' => '\,',
-                            "\n" => '\n',
-                            "\r" => '',
-                        ]
-                    );
-                }
-            }
-            $item = implode(',', $item);
-        }
-
-        return implode($this->delimiter, $val);
-    }
-
-    /**
-     * Returns the value, in the format it should be encoded for json.
-     *
-     * This method must always return an array.
-     *
-     * @return array
-     */
-    public function getJsonValue()
-    {
-        // Structured text values should always be returned as a single
-        // array-item. Multi-value text should be returned as multiple items in
-        // the top-array.
-        if (in_array($this->name, $this->structuredValues)) {
-            return [$this->getParts()];
-        }
-
-        return $this->getParts();
-    }
-
-    /**
-     * Returns the type of value.
-     *
-     * This corresponds to the VALUE= parameter. Every property also has a
-     * 'default' valueType.
-     *
-     * @return string
-     */
-    public function getValueType()
-    {
-        return 'TEXT';
-    }
-
-    /**
-     * Turns the object back into a serialized blob.
-     *
-     * @return string
-     */
-    public function serialize()
-    {
-        // We need to kick in a special type of encoding, if it's a 2.1 vcard.
-        if (Document::VCARD21 !== $this->root->getDocumentType()) {
-            return parent::serialize();
-        }
-
-        $val = $this->getParts();
-
-        if (isset($this->minimumPropertyValues[$this->name])) {
-            $val = \array_pad($val, $this->minimumPropertyValues[$this->name], '');
-        }
-
-        // Imploding multiple parts into a single value, and splitting the
-        // values with ;.
-        if (\count($val) > 1) {
-            foreach ($val as $k => $v) {
-                $val[$k] = \str_replace(';', '\;', $v);
-            }
-            $val = \implode(';', $val);
-        } else {
-            $val = $val[0];
-        }
-
-        $str = $this->name;
-        if ($this->group) {
-            $str = $this->group.'.'.$this->name;
-        }
-        foreach ($this->parameters as $param) {
-            if ('QUOTED-PRINTABLE' === $param->getValue()) {
-                continue;
-            }
-            $str .= ';'.$param->serialize();
-        }
-
-        // If the resulting value contains a \n, we must encode it as
-        // quoted-printable.
-        if (false !== \strpos($val, "\n")) {
-            $str .= ';ENCODING=QUOTED-PRINTABLE:';
-            $lastLine = $str;
-            $out = null;
-
-            // The PHP built-in quoted-printable-encode does not correctly
-            // encode newlines for us. Specifically, the \r\n sequence must in
-            // vcards be encoded as =0D=OA and we must insert soft-newlines
-            // every 75 bytes.
-            for ($ii = 0; $ii < \strlen($val); ++$ii) {
-                $ord = \ord($val[$ii]);
-                // These characters are encoded as themselves.
-                if ($ord >= 32 && $ord <= 126) {
-                    $lastLine .= $val[$ii];
-                } else {
-                    $lastLine .= '='.\strtoupper(\bin2hex($val[$ii]));
-                }
-                if (\strlen($lastLine) >= 75) {
-                    // Soft line break
-                    $out .= $lastLine."=\r\n ";
-                    $lastLine = null;
-                }
-            }
-            if (!\is_null($lastLine)) {
-                $out .= $lastLine."\r\n";
-            }
-
-            return $out;
-        } else {
-            $str .= ':'.$val;
-
-            $str = \preg_replace(
-                '/(
+	/**
+	 * In case this is a multi-value property. This string will be used as a
+	 * delimiter.
+	 *
+	 * @var string
+	 */
+	public $delimiter = ',';
+
+	/**
+	 * List of properties that are considered 'structured'.
+	 *
+	 * @var array
+	 */
+	protected $structuredValues = [
+		// vCard
+		'N',
+		'ADR',
+		'ORG',
+		'GENDER',
+		'CLIENTPIDMAP',
+
+		// iCalendar
+		'REQUEST-STATUS',
+	];
+
+	/**
+	 * Some text components have a minimum number of components.
+	 *
+	 * N must for instance be represented as 5 components, separated by ;, even
+	 * if the last few components are unused.
+	 *
+	 * @var array
+	 */
+	protected $minimumPropertyValues = [
+		'N' => 5,
+		'ADR' => 7,
+	];
+
+	/**
+	 * Creates the property.
+	 *
+	 * You can specify the parameters either in key=>value syntax, in which case
+	 * parameters will automatically be created, or you can just pass a list of
+	 * Parameter objects.
+	 *
+	 * @param Component         $root       The root document
+	 * @param string            $name
+	 * @param string|array|null $value
+	 * @param array             $parameters List of parameters
+	 * @param string            $group      The vcard property group
+	 */
+	public function __construct(Component $root, $name, $value = null, array $parameters = [], $group = null)
+	{
+		// There's two types of multi-valued text properties:
+		// 1. multivalue properties.
+		// 2. structured value properties
+		//
+		// The former is always separated by a comma, the latter by semi-colon.
+		if (in_array($name, $this->structuredValues)) {
+			$this->delimiter = ';';
+		}
+
+		parent::__construct($root, $name, $value, $parameters, $group);
+	}
+
+	/**
+	 * Sets a raw value coming from a mimedir (iCalendar/vCard) file.
+	 *
+	 * This has been 'unfolded', so only 1 line will be passed. Unescaping is
+	 * not yet done, but parameters are not included.
+	 *
+	 * @param string $val
+	 */
+	public function setRawMimeDirValue($val)
+	{
+		$this->setValue(MimeDir::unescapeValue($val, $this->delimiter));
+	}
+
+	/**
+	 * Sets the value as a quoted-printable encoded string.
+	 *
+	 * @param string $val
+	 */
+	public function setQuotedPrintableValue($val)
+	{
+		$val = quoted_printable_decode($val);
+
+		// Quoted printable only appears in vCard 2.1, and the only character
+		// that may be escaped there is ;. So we are simply splitting on just
+		// that.
+		//
+		// We also don't have to unescape \\, so all we need to look for is a ;
+		// that's not preceded with a \.
+		$regex = '# (?<!\\\\) ; #x';
+		$matches = preg_split($regex, $val);
+		$this->setValue($matches);
+	}
+
+	/**
+	 * Returns a raw mime-dir representation of the value.
+	 *
+	 * @return string
+	 */
+	public function getRawMimeDirValue()
+	{
+		$val = $this->getParts();
+
+		if (isset($this->minimumPropertyValues[$this->name])) {
+			$val = array_pad($val, $this->minimumPropertyValues[$this->name], '');
+		}
+
+		foreach ($val as &$item) {
+			if (!is_array($item)) {
+				$item = [$item];
+			}
+
+			foreach ($item as &$subItem) {
+				if (!is_null($subItem)) {
+					$subItem = strtr(
+						$subItem,
+						[
+							'\\' => '\\\\',
+							';' => '\;',
+							',' => '\,',
+							"\n" => '\n',
+							"\r" => '',
+						]
+					);
+				}
+			}
+			$item = implode(',', $item);
+		}
+
+		return implode($this->delimiter, $val);
+	}
+
+	/**
+	 * Returns the value, in the format it should be encoded for json.
+	 *
+	 * This method must always return an array.
+	 *
+	 * @return array
+	 */
+	public function getJsonValue()
+	{
+		// Structured text values should always be returned as a single
+		// array-item. Multi-value text should be returned as multiple items in
+		// the top-array.
+		if (in_array($this->name, $this->structuredValues)) {
+			return [$this->getParts()];
+		}
+
+		return $this->getParts();
+	}
+
+	/**
+	 * Returns the type of value.
+	 *
+	 * This corresponds to the VALUE= parameter. Every property also has a
+	 * 'default' valueType.
+	 *
+	 * @return string
+	 */
+	public function getValueType()
+	{
+		return 'TEXT';
+	}
+
+	/**
+	 * Turns the object back into a serialized blob.
+	 *
+	 * @return string
+	 */
+	public function serialize()
+	{
+		// We need to kick in a special type of encoding, if it's a 2.1 vcard.
+		if (Document::VCARD21 !== $this->root->getDocumentType()) {
+			return parent::serialize();
+		}
+
+		$val = $this->getParts();
+
+		if (isset($this->minimumPropertyValues[$this->name])) {
+			$val = \array_pad($val, $this->minimumPropertyValues[$this->name], '');
+		}
+
+		// Imploding multiple parts into a single value, and splitting the
+		// values with ;.
+		if (\count($val) > 1) {
+			foreach ($val as $k => $v) {
+				$val[$k] = \str_replace(';', '\;', $v);
+			}
+			$val = \implode(';', $val);
+		} else {
+			$val = $val[0];
+		}
+
+		$str = $this->name;
+		if ($this->group) {
+			$str = $this->group.'.'.$this->name;
+		}
+		foreach ($this->parameters as $param) {
+			if ('QUOTED-PRINTABLE' === $param->getValue()) {
+				continue;
+			}
+			$str .= ';'.$param->serialize();
+		}
+
+		// If the resulting value contains a \n, we must encode it as
+		// quoted-printable.
+		if (false !== \strpos($val, "\n")) {
+			$str .= ';ENCODING=QUOTED-PRINTABLE:';
+			$lastLine = $str;
+			$out = null;
+
+			// The PHP built-in quoted-printable-encode does not correctly
+			// encode newlines for us. Specifically, the \r\n sequence must in
+			// vcards be encoded as =0D=OA and we must insert soft-newlines
+			// every 75 bytes.
+			for ($ii = 0; $ii < \strlen($val); ++$ii) {
+				$ord = \ord($val[$ii]);
+				// These characters are encoded as themselves.
+				if ($ord >= 32 && $ord <= 126) {
+					$lastLine .= $val[$ii];
+				} else {
+					$lastLine .= '='.\strtoupper(\bin2hex($val[$ii]));
+				}
+				if (\strlen($lastLine) >= 75) {
+					// Soft line break
+					$out .= $lastLine."=\r\n ";
+					$lastLine = null;
+				}
+			}
+			if (!\is_null($lastLine)) {
+				$out .= $lastLine."\r\n";
+			}
+
+			return $out;
+		} else {
+			$str .= ':'.$val;
+
+			$str = \preg_replace(
+				'/(
                     (?:^.)?         # 1 additional byte in first line because of missing single space (see next line)
                     .{1,74}         # max 75 bytes per line (1 byte is used for a single space added after every CRLF)
                     (?![\x80-\xbf]) # prevent splitting multibyte characters
                 )/x',
-                "$1\r\n ",
-                $str
-            );
-
-            // remove single space after last CRLF
-            return \substr($str, 0, -1);
-        }
-    }
-
-    /**
-     * This method serializes only the value of a property. This is used to
-     * create xCard or xCal documents.
-     *
-     * @param Xml\Writer $writer XML writer
-     */
-    protected function xmlSerializeValue(Xml\Writer $writer)
-    {
-        $values = $this->getParts();
-
-        $map = function ($items) use ($values, $writer) {
-            foreach ($items as $i => $item) {
-                $writer->writeElement(
-                    $item,
-                    !empty($values[$i]) ? $values[$i] : null
-                );
-            }
-        };
-
-        switch ($this->name) {
-            // Special-casing the REQUEST-STATUS property.
-            //
-            // See:
-            // http://tools.ietf.org/html/rfc6321#section-3.4.1.3
-            case 'REQUEST-STATUS':
-                $writer->writeElement('code', $values[0]);
-                $writer->writeElement('description', $values[1]);
-
-                if (isset($values[2])) {
-                    $writer->writeElement('data', $values[2]);
-                }
-                break;
-
-            case 'N':
-                $map([
-                    'surname',
-                    'given',
-                    'additional',
-                    'prefix',
-                    'suffix',
-                ]);
-                break;
-
-            case 'GENDER':
-                $map([
-                    'sex',
-                    'text',
-                ]);
-                break;
-
-            case 'ADR':
-                $map([
-                    'pobox',
-                    'ext',
-                    'street',
-                    'locality',
-                    'region',
-                    'code',
-                    'country',
-                ]);
-                break;
-
-            case 'CLIENTPIDMAP':
-                $map([
-                    'sourceid',
-                    'uri',
-                ]);
-                break;
-
-            default:
-                parent::xmlSerializeValue($writer);
-        }
-    }
-
-    /**
-     * Validates the node for correctness.
-     *
-     * The following options are supported:
-     *   - Node::REPAIR - If something is broken, and automatic repair may
-     *                    be attempted.
-     *
-     * An array is returned with warnings.
-     *
-     * Every item in the array has the following properties:
-     *    * level - (number between 1 and 3 with severity information)
-     *    * message - (human readable message)
-     *    * node - (reference to the offending node)
-     *
-     * @param int $options
-     *
-     * @return array
-     */
-    public function validate($options = 0)
-    {
-        $warnings = parent::validate($options);
-
-        if (isset($this->minimumPropertyValues[$this->name])) {
-            $minimum = $this->minimumPropertyValues[$this->name];
-            $parts = $this->getParts();
-            if (count($parts) < $minimum) {
-                $warnings[] = [
-                    'level' => $options & self::REPAIR ? 1 : 3,
-                    'message' => 'The '.$this->name.' property must have at least '.$minimum.' values. It only has '.count($parts),
-                    'node' => $this,
-                ];
-                if ($options & self::REPAIR) {
-                    $parts = array_pad($parts, $minimum, '');
-                    $this->setParts($parts);
-                }
-            }
-        }
-
-        return $warnings;
-    }
+				"$1\r\n ",
+				$str
+			);
+
+			// remove single space after last CRLF
+			return \substr($str, 0, -1);
+		}
+	}
+
+	/**
+	 * This method serializes only the value of a property. This is used to
+	 * create xCard or xCal documents.
+	 *
+	 * @param Xml\Writer $writer XML writer
+	 */
+	protected function xmlSerializeValue(Xml\Writer $writer)
+	{
+		$values = $this->getParts();
+
+		$map = function ($items) use ($values, $writer) {
+			foreach ($items as $i => $item) {
+				$writer->writeElement(
+					$item,
+					!empty($values[$i]) ? $values[$i] : null
+				);
+			}
+		};
+
+		switch ($this->name) {
+			// Special-casing the REQUEST-STATUS property.
+			//
+			// See:
+			// http://tools.ietf.org/html/rfc6321#section-3.4.1.3
+			case 'REQUEST-STATUS':
+				$writer->writeElement('code', $values[0]);
+				$writer->writeElement('description', $values[1]);
+
+				if (isset($values[2])) {
+					$writer->writeElement('data', $values[2]);
+				}
+				break;
+
+			case 'N':
+				$map([
+					'surname',
+					'given',
+					'additional',
+					'prefix',
+					'suffix',
+				]);
+				break;
+
+			case 'GENDER':
+				$map([
+					'sex',
+					'text',
+				]);
+				break;
+
+			case 'ADR':
+				$map([
+					'pobox',
+					'ext',
+					'street',
+					'locality',
+					'region',
+					'code',
+					'country',
+				]);
+				break;
+
+			case 'CLIENTPIDMAP':
+				$map([
+					'sourceid',
+					'uri',
+				]);
+				break;
+
+			default:
+				parent::xmlSerializeValue($writer);
+		}
+	}
+
+	/**
+	 * Validates the node for correctness.
+	 *
+	 * The following options are supported:
+	 *   - Node::REPAIR - If something is broken, and automatic repair may
+	 *                    be attempted.
+	 *
+	 * An array is returned with warnings.
+	 *
+	 * Every item in the array has the following properties:
+	 *    * level - (number between 1 and 3 with severity information)
+	 *    * message - (human readable message)
+	 *    * node - (reference to the offending node)
+	 *
+	 * @param int $options
+	 *
+	 * @return array
+	 */
+	public function validate($options = 0)
+	{
+		$warnings = parent::validate($options);
+
+		if (isset($this->minimumPropertyValues[$this->name])) {
+			$minimum = $this->minimumPropertyValues[$this->name];
+			$parts = $this->getParts();
+			if (count($parts) < $minimum) {
+				$warnings[] = [
+					'level' => $options & self::REPAIR ? 1 : 3,
+					'message' => 'The '.$this->name.' property must have at least '.$minimum.' values. It only has '.count($parts),
+					'node' => $this,
+				];
+				if ($options & self::REPAIR) {
+					$parts = array_pad($parts, $minimum, '');
+					$this->setParts($parts);
+				}
+			}
+		}
+
+		return $warnings;
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -285,7 +285,7 @@
     {
         $values = $this->getParts();
 
-        $map = function ($items) use ($values, $writer) {
+        $map = function($items) use ($values, $writer) {
             foreach ($items as $i => $item) {
                 $writer->writeElement(
                     $item,

htdocs/includes/sabre/sabre/vobject/lib/Property/Uri.php

Indentation +91 added lines, -91 removed lines

@@ -16,101 +16,101 @@
  */
 class Uri extends Text
 {
-    /**
-     * In case this is a multi-value property. This string will be used as a
-     * delimiter.
-     *
-     * @var string
-     */
-    public $delimiter = '';
+	/**
+	 * In case this is a multi-value property. This string will be used as a
+	 * delimiter.
+	 *
+	 * @var string
+	 */
+	public $delimiter = '';
 
-    /**
-     * Returns the type of value.
-     *
-     * This corresponds to the VALUE= parameter. Every property also has a
-     * 'default' valueType.
-     *
-     * @return string
-     */
-    public function getValueType()
-    {
-        return 'URI';
-    }
+	/**
+	 * Returns the type of value.
+	 *
+	 * This corresponds to the VALUE= parameter. Every property also has a
+	 * 'default' valueType.
+	 *
+	 * @return string
+	 */
+	public function getValueType()
+	{
+		return 'URI';
+	}
 
-    /**
-     * Returns an iterable list of children.
-     *
-     * @return array
-     */
-    public function parameters()
-    {
-        $parameters = parent::parameters();
-        if (!isset($parameters['VALUE']) && in_array($this->name, ['URL', 'PHOTO'])) {
-            // If we are encoding a URI value, and this URI value has no
-            // VALUE=URI parameter, we add it anyway.
-            //
-            // This is not required by any spec, but both Apple iCal and Apple
-            // AddressBook (at least in version 10.8) will trip over this if
-            // this is not set, and so it improves compatibility.
-            //
-            // See Issue #227 and #235
-            $parameters['VALUE'] = new Parameter($this->root, 'VALUE', 'URI');
-        }
+	/**
+	 * Returns an iterable list of children.
+	 *
+	 * @return array
+	 */
+	public function parameters()
+	{
+		$parameters = parent::parameters();
+		if (!isset($parameters['VALUE']) && in_array($this->name, ['URL', 'PHOTO'])) {
+			// If we are encoding a URI value, and this URI value has no
+			// VALUE=URI parameter, we add it anyway.
+			//
+			// This is not required by any spec, but both Apple iCal and Apple
+			// AddressBook (at least in version 10.8) will trip over this if
+			// this is not set, and so it improves compatibility.
+			//
+			// See Issue #227 and #235
+			$parameters['VALUE'] = new Parameter($this->root, 'VALUE', 'URI');
+		}
 
-        return $parameters;
-    }
+		return $parameters;
+	}
 
-    /**
-     * Sets a raw value coming from a mimedir (iCalendar/vCard) file.
-     *
-     * This has been 'unfolded', so only 1 line will be passed. Unescaping is
-     * not yet done, but parameters are not included.
-     *
-     * @param string $val
-     */
-    public function setRawMimeDirValue($val)
-    {
-        // Normally we don't need to do any type of unescaping for these
-        // properties, however.. we've noticed that Google Contacts
-        // specifically escapes the colon (:) with a backslash. While I have
-        // no clue why they thought that was a good idea, I'm unescaping it
-        // anyway.
-        //
-        // Good thing backslashes are not allowed in urls. Makes it easy to
-        // assume that a backslash is always intended as an escape character.
-        if ('URL' === $this->name) {
-            $regex = '#  (?: (\\\\ (?: \\\\ | : ) ) ) #x';
-            $matches = preg_split($regex, $val, -1, PREG_SPLIT_DELIM_CAPTURE | PREG_SPLIT_NO_EMPTY);
-            $newVal = '';
-            foreach ($matches as $match) {
-                switch ($match) {
-                    case '\:':
-                        $newVal .= ':';
-                        break;
-                    default:
-                        $newVal .= $match;
-                        break;
-                }
-            }
-            $this->value = $newVal;
-        } else {
-            $this->value = strtr($val, ['\,' => ',']);
-        }
-    }
+	/**
+	 * Sets a raw value coming from a mimedir (iCalendar/vCard) file.
+	 *
+	 * This has been 'unfolded', so only 1 line will be passed. Unescaping is
+	 * not yet done, but parameters are not included.
+	 *
+	 * @param string $val
+	 */
+	public function setRawMimeDirValue($val)
+	{
+		// Normally we don't need to do any type of unescaping for these
+		// properties, however.. we've noticed that Google Contacts
+		// specifically escapes the colon (:) with a backslash. While I have
+		// no clue why they thought that was a good idea, I'm unescaping it
+		// anyway.
+		//
+		// Good thing backslashes are not allowed in urls. Makes it easy to
+		// assume that a backslash is always intended as an escape character.
+		if ('URL' === $this->name) {
+			$regex = '#  (?: (\\\\ (?: \\\\ | : ) ) ) #x';
+			$matches = preg_split($regex, $val, -1, PREG_SPLIT_DELIM_CAPTURE | PREG_SPLIT_NO_EMPTY);
+			$newVal = '';
+			foreach ($matches as $match) {
+				switch ($match) {
+					case '\:':
+						$newVal .= ':';
+						break;
+					default:
+						$newVal .= $match;
+						break;
+				}
+			}
+			$this->value = $newVal;
+		} else {
+			$this->value = strtr($val, ['\,' => ',']);
+		}
+	}
 
-    /**
-     * Returns a raw mime-dir representation of the value.
-     *
-     * @return string
-     */
-    public function getRawMimeDirValue()
-    {
-        if (is_array($this->value)) {
-            $value = $this->value[0];
-        } else {
-            $value = $this->value;
-        }
+	/**
+	 * Returns a raw mime-dir representation of the value.
+	 *
+	 * @return string
+	 */
+	public function getRawMimeDirValue()
+	{
+		if (is_array($this->value)) {
+			$value = $this->value[0];
+		} else {
+			$value = $this->value;
+		}
 
-        return strtr($value, [',' => '\,']);
-    }
+		return strtr($value, [',' => '\,']);
+	}
 }

htdocs/includes/sabre/sabre/vobject/lib/Property/UtcOffset.php

Indentation +51 added lines, -51 removed lines

@@ -13,58 +13,58 @@
  */
 class UtcOffset extends Text
 {
-    /**
-     * In case this is a multi-value property. This string will be used as a
-     * delimiter.
-     *
-     * @var string
-     */
-    public $delimiter = '';
+	/**
+	 * In case this is a multi-value property. This string will be used as a
+	 * delimiter.
+	 *
+	 * @var string
+	 */
+	public $delimiter = '';
 
-    /**
-     * Returns the type of value.
-     *
-     * This corresponds to the VALUE= parameter. Every property also has a
-     * 'default' valueType.
-     *
-     * @return string
-     */
-    public function getValueType()
-    {
-        return 'UTC-OFFSET';
-    }
+	/**
+	 * Returns the type of value.
+	 *
+	 * This corresponds to the VALUE= parameter. Every property also has a
+	 * 'default' valueType.
+	 *
+	 * @return string
+	 */
+	public function getValueType()
+	{
+		return 'UTC-OFFSET';
+	}
 
-    /**
-     * Sets the JSON value, as it would appear in a jCard or jCal object.
-     *
-     * The value must always be an array.
-     */
-    public function setJsonValue(array $value)
-    {
-        $value = array_map(
-            function ($value) {
-                return str_replace(':', '', $value);
-            },
-            $value
-        );
-        parent::setJsonValue($value);
-    }
+	/**
+	 * Sets the JSON value, as it would appear in a jCard or jCal object.
+	 *
+	 * The value must always be an array.
+	 */
+	public function setJsonValue(array $value)
+	{
+		$value = array_map(
+			function ($value) {
+				return str_replace(':', '', $value);
+			},
+			$value
+		);
+		parent::setJsonValue($value);
+	}
 
-    /**
-     * Returns the value, in the format it should be encoded for JSON.
-     *
-     * This method must always return an array.
-     *
-     * @return array
-     */
-    public function getJsonValue()
-    {
-        return array_map(
-            function ($value) {
-                return substr($value, 0, -2).':'.
-                       substr($value, -2);
-            },
-            parent::getJsonValue()
-        );
-    }
+	/**
+	 * Returns the value, in the format it should be encoded for JSON.
+	 *
+	 * This method must always return an array.
+	 *
+	 * @return array
+	 */
+	public function getJsonValue()
+	{
+		return array_map(
+			function ($value) {
+				return substr($value, 0, -2).':'.
+					   substr($value, -2);
+			},
+			parent::getJsonValue()
+		);
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
     public function setJsonValue(array $value)
     {
         $value = array_map(
-            function ($value) {
+            function($value) {
                 return str_replace(':', '', $value);
             },
             $value
@@ -60,7 +60,7 @@ 
     public function getJsonValue()
     {
         return array_map(
-            function ($value) {
+            function($value) {
                 return substr($value, 0, -2).':'.
                        substr($value, -2);
             },