YetiForceCompany / YetiForceCRM

libraries/SabreDAV/CalDAV/CalendarQueryValidator.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,8 +2,8 @@
 
 namespace Sabre\CalDAV;
 
-use Sabre\VObject;
 use DateTime;
+use Sabre\VObject;
 
 /**
  * CalendarQuery Validator

Spacing +2 added lines, -2 removed lines

@@ -301,7 +301,7 @@ 
                 if ($component->parent->name === 'VEVENT' && $component->parent->RRULE) {
 
                     // Fire up the iterator!
-                    $it = new VObject\Recur\EventIterator($component->parent->parent, (string)$component->parent->UID);
+                    $it = new VObject\Recur\EventIterator($component->parent->parent, (string) $component->parent->UID);
                     while ($it->valid()) {
                         $expandedEvent = $it->getEventObject();
 
@@ -317,7 +317,7 @@ 
                                     return true;
                                 }
 
-                                if ((string)$expandedAlarm->TRIGGER['VALUE'] === 'DATE-TIME') {
+                                if ((string) $expandedAlarm->TRIGGER['VALUE'] === 'DATE-TIME') {
                                     // This is an alarm with a non-relative trigger
                                     // time, likely created by a buggy client. The
                                     // implication is that every alarm in this

Indentation +351 added lines, -351 removed lines

@@ -20,356 +20,356 @@
  */
 class CalendarQueryValidator {
 
-    /**
-     * Verify if a list of filters applies to the calendar data object
-     *
-     * The list of filters must be formatted as parsed by \Sabre\CalDAV\CalendarQueryParser
-     *
-     * @param VObject\Component $vObject
-     * @param array $filters
-     * @return bool
-     */
-    public function validate(VObject\Component\VCalendar $vObject, array $filters) {
-
-        // The top level object is always a component filter.
-        // We'll parse it manually, as it's pretty simple.
-        if ($vObject->name !== $filters['name']) {
-            return false;
-        }
-
-        return
-            $this->validateCompFilters($vObject, $filters['comp-filters']) &&
-            $this->validatePropFilters($vObject, $filters['prop-filters']);
-
-
-    }
-
-    /**
-     * This method checks the validity of comp-filters.
-     *
-     * A list of comp-filters needs to be specified. Also the parent of the
-     * component we're checking should be specified, not the component to check
-     * itself.
-     *
-     * @param VObject\Component $parent
-     * @param array $filters
-     * @return bool
-     */
-    protected function validateCompFilters(VObject\Component $parent, array $filters) {
-
-        foreach ($filters as $filter) {
-
-            $isDefined = isset($parent->{$filter['name']});
-
-            if ($filter['is-not-defined']) {
-
-                if ($isDefined) {
-                    return false;
-                } else {
-                    continue;
-                }
-
-            }
-            if (!$isDefined) {
-                return false;
-            }
-
-            if ($filter['time-range']) {
-                foreach ($parent->{$filter['name']} as $subComponent) {
-                    if ($this->validateTimeRange($subComponent, $filter['time-range']['start'], $filter['time-range']['end'])) {
-                        continue 2;
-                    }
-                }
-                return false;
-            }
-
-            if (!$filter['comp-filters'] && !$filter['prop-filters']) {
-                continue;
-            }
-
-            // If there are sub-filters, we need to find at least one component
-            // for which the subfilters hold true.
-            foreach ($parent->{$filter['name']} as $subComponent) {
-
-                if (
-                    $this->validateCompFilters($subComponent, $filter['comp-filters']) &&
-                    $this->validatePropFilters($subComponent, $filter['prop-filters'])) {
-                        // We had a match, so this comp-filter succeeds
-                        continue 2;
-                }
-
-            }
-
-            // If we got here it means there were sub-comp-filters or
-            // sub-prop-filters and there was no match. This means this filter
-            // needs to return false.
-            return false;
-
-        }
-
-        // If we got here it means we got through all comp-filters alive so the
-        // filters were all true.
-        return true;
-
-    }
-
-    /**
-     * This method checks the validity of prop-filters.
-     *
-     * A list of prop-filters needs to be specified. Also the parent of the
-     * property we're checking should be specified, not the property to check
-     * itself.
-     *
-     * @param VObject\Component $parent
-     * @param array $filters
-     * @return bool
-     */
-    protected function validatePropFilters(VObject\Component $parent, array $filters) {
-
-        foreach ($filters as $filter) {
-
-            $isDefined = isset($parent->{$filter['name']});
-
-            if ($filter['is-not-defined']) {
-
-                if ($isDefined) {
-                    return false;
-                } else {
-                    continue;
-                }
-
-            }
-            if (!$isDefined) {
-                return false;
-            }
-
-            if ($filter['time-range']) {
-                foreach ($parent->{$filter['name']} as $subComponent) {
-                    if ($this->validateTimeRange($subComponent, $filter['time-range']['start'], $filter['time-range']['end'])) {
-                        continue 2;
-                    }
-                }
-                return false;
-            }
-
-            if (!$filter['param-filters'] && !$filter['text-match']) {
-                continue;
-            }
-
-            // If there are sub-filters, we need to find at least one property
-            // for which the subfilters hold true.
-            foreach ($parent->{$filter['name']} as $subComponent) {
-
-                if (
-                    $this->validateParamFilters($subComponent, $filter['param-filters']) &&
-                    (!$filter['text-match'] || $this->validateTextMatch($subComponent, $filter['text-match']))
-                ) {
-                    // We had a match, so this prop-filter succeeds
-                    continue 2;
-                }
-
-            }
-
-            // If we got here it means there were sub-param-filters or
-            // text-match filters and there was no match. This means the
-            // filter needs to return false.
-            return false;
-
-        }
-
-        // If we got here it means we got through all prop-filters alive so the
-        // filters were all true.
-        return true;
-
-    }
-
-    /**
-     * This method checks the validity of param-filters.
-     *
-     * A list of param-filters needs to be specified. Also the parent of the
-     * parameter we're checking should be specified, not the parameter to check
-     * itself.
-     *
-     * @param VObject\Property $parent
-     * @param array $filters
-     * @return bool
-     */
-    protected function validateParamFilters(VObject\Property $parent, array $filters) {
-
-        foreach ($filters as $filter) {
-
-            $isDefined = isset($parent[$filter['name']]);
-
-            if ($filter['is-not-defined']) {
-
-                if ($isDefined) {
-                    return false;
-                } else {
-                    continue;
-                }
-
-            }
-            if (!$isDefined) {
-                return false;
-            }
-
-            if (!$filter['text-match']) {
-                continue;
-            }
-
-            // If there are sub-filters, we need to find at least one parameter
-            // for which the subfilters hold true.
-            foreach ($parent[$filter['name']]->getParts() as $paramPart) {
-
-                if ($this->validateTextMatch($paramPart, $filter['text-match'])) {
-                    // We had a match, so this param-filter succeeds
-                    continue 2;
-                }
-
-            }
-
-            // If we got here it means there was a text-match filter and there
-            // were no matches. This means the filter needs to return false.
-            return false;
-
-        }
-
-        // If we got here it means we got through all param-filters alive so the
-        // filters were all true.
-        return true;
-
-    }
-
-    /**
-     * This method checks the validity of a text-match.
-     *
-     * A single text-match should be specified as well as the specific property
-     * or parameter we need to validate.
-     *
-     * @param VObject\Node|string $check Value to check against.
-     * @param array $textMatch
-     * @return bool
-     */
-    protected function validateTextMatch($check, array $textMatch) {
-
-        if ($check instanceof VObject\Node) {
-            $check = $check->getValue();
-        }
-
-        $isMatching = \Sabre\DAV\StringUtil::textMatch($check, $textMatch['value'], $textMatch['collation']);
-
-        return ($textMatch['negate-condition'] xor $isMatching);
-
-    }
-
-    /**
-     * Validates if a component matches the given time range.
-     *
-     * This is all based on the rules specified in rfc4791, which are quite
-     * complex.
-     *
-     * @param VObject\Node $component
-     * @param DateTime $start
-     * @param DateTime $end
-     * @return bool
-     */
-    protected function validateTimeRange(VObject\Node $component, $start, $end) {
-
-        if (is_null($start)) {
-            $start = new DateTime('1900-01-01');
-        }
-        if (is_null($end)) {
-            $end = new DateTime('3000-01-01');
-        }
-
-        switch ($component->name) {
-
-            case 'VEVENT' :
-            case 'VTODO' :
-            case 'VJOURNAL' :
-
-                return $component->isInTimeRange($start, $end);
-
-            case 'VALARM' :
-
-                // If the valarm is wrapped in a recurring event, we need to
-                // expand the recursions, and validate each.
-                //
-                // Our datamodel doesn't easily allow us to do this straight
-                // in the VALARM component code, so this is a hack, and an
-                // expensive one too.
-                if ($component->parent->name === 'VEVENT' && $component->parent->RRULE) {
-
-                    // Fire up the iterator!
-                    $it = new VObject\Recur\EventIterator($component->parent->parent, (string)$component->parent->UID);
-                    while ($it->valid()) {
-                        $expandedEvent = $it->getEventObject();
-
-                        // We need to check from these expanded alarms, which
-                        // one is the first to trigger. Based on this, we can
-                        // determine if we can 'give up' expanding events.
-                        $firstAlarm = null;
-                        if ($expandedEvent->VALARM !== null) {
-                            foreach ($expandedEvent->VALARM as $expandedAlarm) {
-
-                                $effectiveTrigger = $expandedAlarm->getEffectiveTriggerTime();
-                                if ($expandedAlarm->isInTimeRange($start, $end)) {
-                                    return true;
-                                }
-
-                                if ((string)$expandedAlarm->TRIGGER['VALUE'] === 'DATE-TIME') {
-                                    // This is an alarm with a non-relative trigger
-                                    // time, likely created by a buggy client. The
-                                    // implication is that every alarm in this
-                                    // recurring event trigger at the exact same
-                                    // time. It doesn't make sense to traverse
-                                    // further.
-                                } else {
-                                    // We store the first alarm as a means to
-                                    // figure out when we can stop traversing.
-                                    if (!$firstAlarm || $effectiveTrigger < $firstAlarm) {
-                                        $firstAlarm = $effectiveTrigger;
-                                    }
-                                }
-                            }
-                        }
-                        if (is_null($firstAlarm)) {
-                            // No alarm was found.
-                            //
-                            // Or technically: No alarm that will change for
-                            // every instance of the recurrence was found,
-                            // which means we can assume there was no match.
-                            return false;
-                        }
-                        if ($firstAlarm > $end) {
-                            return false;
-                        }
-                        $it->next();
-                    }
-                    return false;
-                } else {
-                    return $component->isInTimeRange($start, $end);
-                }
-
-            case 'VFREEBUSY' :
-                throw new \Sabre\DAV\Exception\NotImplemented('time-range filters are currently not supported on ' . $component->name . ' components');
-
-            case 'COMPLETED' :
-            case 'CREATED' :
-            case 'DTEND' :
-            case 'DTSTAMP' :
-            case 'DTSTART' :
-            case 'DUE' :
-            case 'LAST-MODIFIED' :
-                return ($start <= $component->getDateTime() && $end >= $component->getDateTime());
-
-
-
-            default :
-                throw new \Sabre\DAV\Exception\BadRequest('You cannot create a time-range filter on a ' . $component->name . ' component');
-
-        }
-
-    }
+	/**
+	 * Verify if a list of filters applies to the calendar data object
+	 *
+	 * The list of filters must be formatted as parsed by \Sabre\CalDAV\CalendarQueryParser
+	 *
+	 * @param VObject\Component $vObject
+	 * @param array $filters
+	 * @return bool
+	 */
+	public function validate(VObject\Component\VCalendar $vObject, array $filters) {
+
+		// The top level object is always a component filter.
+		// We'll parse it manually, as it's pretty simple.
+		if ($vObject->name !== $filters['name']) {
+			return false;
+		}
+
+		return
+			$this->validateCompFilters($vObject, $filters['comp-filters']) &&
+			$this->validatePropFilters($vObject, $filters['prop-filters']);
+
+
+	}
+
+	/**
+	 * This method checks the validity of comp-filters.
+	 *
+	 * A list of comp-filters needs to be specified. Also the parent of the
+	 * component we're checking should be specified, not the component to check
+	 * itself.
+	 *
+	 * @param VObject\Component $parent
+	 * @param array $filters
+	 * @return bool
+	 */
+	protected function validateCompFilters(VObject\Component $parent, array $filters) {
+
+		foreach ($filters as $filter) {
+
+			$isDefined = isset($parent->{$filter['name']});
+
+			if ($filter['is-not-defined']) {
+
+				if ($isDefined) {
+					return false;
+				} else {
+					continue;
+				}
+
+			}
+			if (!$isDefined) {
+				return false;
+			}
+
+			if ($filter['time-range']) {
+				foreach ($parent->{$filter['name']} as $subComponent) {
+					if ($this->validateTimeRange($subComponent, $filter['time-range']['start'], $filter['time-range']['end'])) {
+						continue 2;
+					}
+				}
+				return false;
+			}
+
+			if (!$filter['comp-filters'] && !$filter['prop-filters']) {
+				continue;
+			}
+
+			// If there are sub-filters, we need to find at least one component
+			// for which the subfilters hold true.
+			foreach ($parent->{$filter['name']} as $subComponent) {
+
+				if (
+					$this->validateCompFilters($subComponent, $filter['comp-filters']) &&
+					$this->validatePropFilters($subComponent, $filter['prop-filters'])) {
+						// We had a match, so this comp-filter succeeds
+						continue 2;
+				}
+
+			}
+
+			// If we got here it means there were sub-comp-filters or
+			// sub-prop-filters and there was no match. This means this filter
+			// needs to return false.
+			return false;
+
+		}
+
+		// If we got here it means we got through all comp-filters alive so the
+		// filters were all true.
+		return true;
+
+	}
+
+	/**
+	 * This method checks the validity of prop-filters.
+	 *
+	 * A list of prop-filters needs to be specified. Also the parent of the
+	 * property we're checking should be specified, not the property to check
+	 * itself.
+	 *
+	 * @param VObject\Component $parent
+	 * @param array $filters
+	 * @return bool
+	 */
+	protected function validatePropFilters(VObject\Component $parent, array $filters) {
+
+		foreach ($filters as $filter) {
+
+			$isDefined = isset($parent->{$filter['name']});
+
+			if ($filter['is-not-defined']) {
+
+				if ($isDefined) {
+					return false;
+				} else {
+					continue;
+				}
+
+			}
+			if (!$isDefined) {
+				return false;
+			}
+
+			if ($filter['time-range']) {
+				foreach ($parent->{$filter['name']} as $subComponent) {
+					if ($this->validateTimeRange($subComponent, $filter['time-range']['start'], $filter['time-range']['end'])) {
+						continue 2;
+					}
+				}
+				return false;
+			}
+
+			if (!$filter['param-filters'] && !$filter['text-match']) {
+				continue;
+			}
+
+			// If there are sub-filters, we need to find at least one property
+			// for which the subfilters hold true.
+			foreach ($parent->{$filter['name']} as $subComponent) {
+
+				if (
+					$this->validateParamFilters($subComponent, $filter['param-filters']) &&
+					(!$filter['text-match'] || $this->validateTextMatch($subComponent, $filter['text-match']))
+				) {
+					// We had a match, so this prop-filter succeeds
+					continue 2;
+				}
+
+			}
+
+			// If we got here it means there were sub-param-filters or
+			// text-match filters and there was no match. This means the
+			// filter needs to return false.
+			return false;
+
+		}
+
+		// If we got here it means we got through all prop-filters alive so the
+		// filters were all true.
+		return true;
+
+	}
+
+	/**
+	 * This method checks the validity of param-filters.
+	 *
+	 * A list of param-filters needs to be specified. Also the parent of the
+	 * parameter we're checking should be specified, not the parameter to check
+	 * itself.
+	 *
+	 * @param VObject\Property $parent
+	 * @param array $filters
+	 * @return bool
+	 */
+	protected function validateParamFilters(VObject\Property $parent, array $filters) {
+
+		foreach ($filters as $filter) {
+
+			$isDefined = isset($parent[$filter['name']]);
+
+			if ($filter['is-not-defined']) {
+
+				if ($isDefined) {
+					return false;
+				} else {
+					continue;
+				}
+
+			}
+			if (!$isDefined) {
+				return false;
+			}
+
+			if (!$filter['text-match']) {
+				continue;
+			}
+
+			// If there are sub-filters, we need to find at least one parameter
+			// for which the subfilters hold true.
+			foreach ($parent[$filter['name']]->getParts() as $paramPart) {
+
+				if ($this->validateTextMatch($paramPart, $filter['text-match'])) {
+					// We had a match, so this param-filter succeeds
+					continue 2;
+				}
+
+			}
+
+			// If we got here it means there was a text-match filter and there
+			// were no matches. This means the filter needs to return false.
+			return false;
+
+		}
+
+		// If we got here it means we got through all param-filters alive so the
+		// filters were all true.
+		return true;
+
+	}
+
+	/**
+	 * This method checks the validity of a text-match.
+	 *
+	 * A single text-match should be specified as well as the specific property
+	 * or parameter we need to validate.
+	 *
+	 * @param VObject\Node|string $check Value to check against.
+	 * @param array $textMatch
+	 * @return bool
+	 */
+	protected function validateTextMatch($check, array $textMatch) {
+
+		if ($check instanceof VObject\Node) {
+			$check = $check->getValue();
+		}
+
+		$isMatching = \Sabre\DAV\StringUtil::textMatch($check, $textMatch['value'], $textMatch['collation']);
+
+		return ($textMatch['negate-condition'] xor $isMatching);
+
+	}
+
+	/**
+	 * Validates if a component matches the given time range.
+	 *
+	 * This is all based on the rules specified in rfc4791, which are quite
+	 * complex.
+	 *
+	 * @param VObject\Node $component
+	 * @param DateTime $start
+	 * @param DateTime $end
+	 * @return bool
+	 */
+	protected function validateTimeRange(VObject\Node $component, $start, $end) {
+
+		if (is_null($start)) {
+			$start = new DateTime('1900-01-01');
+		}
+		if (is_null($end)) {
+			$end = new DateTime('3000-01-01');
+		}
+
+		switch ($component->name) {
+
+			case 'VEVENT' :
+			case 'VTODO' :
+			case 'VJOURNAL' :
+
+				return $component->isInTimeRange($start, $end);
+
+			case 'VALARM' :
+
+				// If the valarm is wrapped in a recurring event, we need to
+				// expand the recursions, and validate each.
+				//
+				// Our datamodel doesn't easily allow us to do this straight
+				// in the VALARM component code, so this is a hack, and an
+				// expensive one too.
+				if ($component->parent->name === 'VEVENT' && $component->parent->RRULE) {
+
+					// Fire up the iterator!
+					$it = new VObject\Recur\EventIterator($component->parent->parent, (string)$component->parent->UID);
+					while ($it->valid()) {
+						$expandedEvent = $it->getEventObject();
+
+						// We need to check from these expanded alarms, which
+						// one is the first to trigger. Based on this, we can
+						// determine if we can 'give up' expanding events.
+						$firstAlarm = null;
+						if ($expandedEvent->VALARM !== null) {
+							foreach ($expandedEvent->VALARM as $expandedAlarm) {
+
+								$effectiveTrigger = $expandedAlarm->getEffectiveTriggerTime();
+								if ($expandedAlarm->isInTimeRange($start, $end)) {
+									return true;
+								}
+
+								if ((string)$expandedAlarm->TRIGGER['VALUE'] === 'DATE-TIME') {
+									// This is an alarm with a non-relative trigger
+									// time, likely created by a buggy client. The
+									// implication is that every alarm in this
+									// recurring event trigger at the exact same
+									// time. It doesn't make sense to traverse
+									// further.
+								} else {
+									// We store the first alarm as a means to
+									// figure out when we can stop traversing.
+									if (!$firstAlarm || $effectiveTrigger < $firstAlarm) {
+										$firstAlarm = $effectiveTrigger;
+									}
+								}
+							}
+						}
+						if (is_null($firstAlarm)) {
+							// No alarm was found.
+							//
+							// Or technically: No alarm that will change for
+							// every instance of the recurrence was found,
+							// which means we can assume there was no match.
+							return false;
+						}
+						if ($firstAlarm > $end) {
+							return false;
+						}
+						$it->next();
+					}
+					return false;
+				} else {
+					return $component->isInTimeRange($start, $end);
+				}
+
+			case 'VFREEBUSY' :
+				throw new \Sabre\DAV\Exception\NotImplemented('time-range filters are currently not supported on ' . $component->name . ' components');
+
+			case 'COMPLETED' :
+			case 'CREATED' :
+			case 'DTEND' :
+			case 'DTSTAMP' :
+			case 'DTSTART' :
+			case 'DUE' :
+			case 'LAST-MODIFIED' :
+				return ($start <= $component->getDateTime() && $end >= $component->getDateTime());
+
+
+
+			default :
+				throw new \Sabre\DAV\Exception\BadRequest('You cannot create a time-range filter on a ' . $component->name . ' component');
+
+		}
+
+	}
 
 }

Doc Comments +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
      *
      * The list of filters must be formatted as parsed by \Sabre\CalDAV\CalendarQueryParser
      *
-     * @param VObject\Component $vObject
+     * @param VObject\Parser\Sabre\VObject\Document|null $vObject
      * @param array $filters
      * @return bool
      */

libraries/SabreDAV/CalDAV/ICSExportPlugin.php

Unused Use Statements +3 added lines, -3 removed lines

@@ -2,13 +2,13 @@
 
 namespace Sabre\CalDAV;
 
+use DateTime;
 use DateTimeZone;
 use Sabre\DAV;
-use Sabre\VObject;
+use Sabre\DAV\Exception\BadRequest;
 use Sabre\HTTP\RequestInterface;
 use Sabre\HTTP\ResponseInterface;
-use Sabre\DAV\Exception\BadRequest;
-use DateTime;
+use Sabre\VObject;
 
 /**
  * ICS Exporter

Spacing +1 added lines, -1 removed lines

@@ -306,7 +306,7 @@
                     // VTIMEZONE is special, because we need to filter out the duplicates
                     case 'VTIMEZONE' :
                         // Naively just checking tzid.
-                        if (in_array((string)$child->TZID, $collectedTimezones)) continue;
+                        if (in_array((string) $child->TZID, $collectedTimezones)) continue;
 
                         $timezones[] = clone $child;
                         $collectedTimezones[] = $child->TZID;

Indentation +316 added lines, -316 removed lines

@@ -46,321 +46,321 @@
  */
 class ICSExportPlugin extends DAV\ServerPlugin {
 
-    /**
-     * Reference to Server class
-     *
-     * @var \Sabre\DAV\Server
-     */
-    protected $server;
-
-    /**
-     * Initializes the plugin and registers event handlers
-     *
-     * @param \Sabre\DAV\Server $server
-     * @return void
-     */
-    public function initialize(DAV\Server $server) {
-
-        $this->server = $server;
-        $server->on('method:GET', [$this, 'httpGet'], 90);
-        $server->on('browserButtonActions', function($path, $node, &$actions) {
-            if ($node instanceof ICalendar) {
-                $actions .= '<a href="' . htmlspecialchars($path, ENT_QUOTES, 'UTF-8') . '?export"><span class="oi" data-glyph="calendar"></span></a>';
-            }
-        });
-
-    }
-
-    /**
-     * Intercepts GET requests on calendar urls ending with ?export.
-     *
-     * @param RequestInterface $request
-     * @param ResponseInterface $response
-     * @return bool
-     */
-    public function httpGet(RequestInterface $request, ResponseInterface $response) {
-
-        $queryParams = $request->getQueryParameters();
-        if (!array_key_exists('export', $queryParams)) return;
-
-        $path = $request->getPath();
-
-        $node = $this->server->getProperties($path, [
-            '{DAV:}resourcetype',
-            '{DAV:}displayname',
-            '{http://sabredav.org/ns}sync-token',
-            '{DAV:}sync-token',
-            '{http://apple.com/ns/ical/}calendar-color',
-        ]);
-
-        if (!isset($node['{DAV:}resourcetype']) || !$node['{DAV:}resourcetype']->is('{' . Plugin::NS_CALDAV . '}calendar')) {
-            return;
-        }
-        // Marking the transactionType, for logging purposes.
-        $this->server->transactionType = 'get-calendar-export';
-
-        $properties = $node;
-
-        $start = null;
-        $end = null;
-        $expand = false;
-        $componentType = false;
-        if (isset($queryParams['start'])) {
-            if (!ctype_digit($queryParams['start'])) {
-                throw new BadRequest('The start= parameter must contain a unix timestamp');
-            }
-            $start = DateTime::createFromFormat('U', $queryParams['start']);
-        }
-        if (isset($queryParams['end'])) {
-            if (!ctype_digit($queryParams['end'])) {
-                throw new BadRequest('The end= parameter must contain a unix timestamp');
-            }
-            $end = DateTime::createFromFormat('U', $queryParams['end']);
-        }
-        if (isset($queryParams['expand']) && !!$queryParams['expand']) {
-            if (!$start || !$end) {
-                throw new BadRequest('If you\'d like to expand recurrences, you must specify both a start= and end= parameter.');
-            }
-            $expand = true;
-            $componentType = 'VEVENT';
-        }
-        if (isset($queryParams['componentType'])) {
-            if (!in_array($queryParams['componentType'], ['VEVENT', 'VTODO', 'VJOURNAL'])) {
-                throw new BadRequest('You are not allowed to search for components of type: ' . $queryParams['componentType'] . ' here');
-            }
-            $componentType = $queryParams['componentType'];
-        }
-
-        $format = \Sabre\HTTP\Util::Negotiate(
-            $request->getHeader('Accept'),
-            [
-                'text/calendar',
-                'application/calendar+json',
-            ]
-        );
-
-        if (isset($queryParams['accept'])) {
-            if ($queryParams['accept'] === 'application/calendar+json' || $queryParams['accept'] === 'jcal') {
-                $format = 'application/calendar+json';
-            }
-        }
-        if (!$format) {
-            $format = 'text/calendar';
-        }
-
-        $this->generateResponse($path, $start, $end, $expand, $componentType, $format, $properties, $response);
-
-        // Returning false to break the event chain
-        return false;
-
-    }
-
-    /**
-     * This method is responsible for generating the actual, full response.
-     *
-     * @param string $path
-     * @param DateTime|null $start
-     * @param DateTime|null $end
-     * @param bool $expand
-     * @param string $componentType
-     * @param string $format
-     * @param array $properties
-     * @param ResponseInterface $response
-     */
-    protected function generateResponse($path, $start, $end, $expand, $componentType, $format, $properties, ResponseInterface $response) {
-
-        $calDataProp = '{' . Plugin::NS_CALDAV . '}calendar-data';
-
-        $blobs = [];
-        if ($start || $end || $componentType) {
-
-            // If there was a start or end filter, we need to enlist
-            // calendarQuery for speed.
-            $calendarNode = $this->server->tree->getNodeForPath($path);
-            $queryResult = $calendarNode->calendarQuery([
-                'name'         => 'VCALENDAR',
-                'comp-filters' => [
-                    [
-                        'name'           => $componentType,
-                        'comp-filters'   => [],
-                        'prop-filters'   => [],
-                        'is-not-defined' => false,
-                        'time-range'     => [
-                            'start' => $start,
-                            'end'   => $end,
-                        ],
-                    ],
-                ],
-                'prop-filters'   => [],
-                'is-not-defined' => false,
-                'time-range'     => null,
-            ]);
-
-            // queryResult is just a list of base urls. We need to prefix the
-            // calendar path.
-            $queryResult = array_map(
-                function($item) use ($path) {
-                    return $path . '/' . $item;
-                },
-                $queryResult
-            );
-            $nodes = $this->server->getPropertiesForMultiplePaths($queryResult, [$calDataProp]);
-            unset($queryResult);
-
-        } else {
-            $nodes = $this->server->getPropertiesForPath($path, [$calDataProp], 1);
-        }
-
-        // Flattening the arrays
-        foreach ($nodes as $node) {
-            if (isset($node[200][$calDataProp])) {
-                $blobs[$node['href']] = $node[200][$calDataProp];
-            }
-        }
-        unset($nodes);
-
-        $mergedCalendar = $this->mergeObjects(
-            $properties,
-            $blobs
-        );
-
-        if ($expand) {
-            $calendarTimeZone = null;
-            // We're expanding, and for that we need to figure out the
-            // calendar's timezone.
-            $tzProp = '{' . Plugin::NS_CALDAV . '}calendar-timezone';
-            $tzResult = $this->server->getProperties($path, [$tzProp]);
-            if (isset($tzResult[$tzProp])) {
-                // This property contains a VCALENDAR with a single
-                // VTIMEZONE.
-                $vtimezoneObj = VObject\Reader::read($tzResult[$tzProp]);
-                $calendarTimeZone = $vtimezoneObj->VTIMEZONE->getTimeZone();
-                // Destroy circular references to PHP will GC the object.
-                $vtimezoneObj->destroy();
-                unset($vtimezoneObj);
-            } else {
-                // Defaulting to UTC.
-                $calendarTimeZone = new DateTimeZone('UTC');
-            }
-
-            $mergedCalendar = $mergedCalendar->expand($start, $end, $calendarTimeZone);
-        }
-
-        $response->setHeader('Content-Type', $format);
-
-        switch ($format) {
-            case 'text/calendar' :
-                $mergedCalendar = $mergedCalendar->serialize();
-                break;
-            case 'application/calendar+json' :
-                $mergedCalendar = json_encode($mergedCalendar->jsonSerialize());
-                break;
-        }
-
-        $response->setStatus(200);
-        $response->setBody($mergedCalendar);
-
-    }
-
-    /**
-     * Merges all calendar objects, and builds one big iCalendar blob.
-     *
-     * @param array $properties Some CalDAV properties
-     * @param array $inputObjects
-     * @return VObject\Component\VCalendar
-     */
-    public function mergeObjects(array $properties, array $inputObjects) {
-
-        $calendar = new VObject\Component\VCalendar();
-        $calendar->version = '2.0';
-        if (DAV\Server::$exposeVersion) {
-            $calendar->prodid = '-//SabreDAV//SabreDAV ' . DAV\Version::VERSION . '//EN';
-        } else {
-            $calendar->prodid = '-//SabreDAV//SabreDAV//EN';
-        }
-        if (isset($properties['{DAV:}displayname'])) {
-            $calendar->{'X-WR-CALNAME'} = $properties['{DAV:}displayname'];
-        }
-        if (isset($properties['{http://apple.com/ns/ical/}calendar-color'])) {
-            $calendar->{'X-APPLE-CALENDAR-COLOR'} = $properties['{http://apple.com/ns/ical/}calendar-color'];
-        }
-
-        $collectedTimezones = [];
-
-        $timezones = [];
-        $objects = [];
-
-        foreach ($inputObjects as $href => $inputObject) {
-
-            $nodeComp = VObject\Reader::read($inputObject);
-
-            foreach ($nodeComp->children() as $child) {
-
-                switch ($child->name) {
-                    case 'VEVENT' :
-                    case 'VTODO' :
-                    case 'VJOURNAL' :
-                        $objects[] = clone $child;
-                        break;
-
-                    // VTIMEZONE is special, because we need to filter out the duplicates
-                    case 'VTIMEZONE' :
-                        // Naively just checking tzid.
-                        if (in_array((string)$child->TZID, $collectedTimezones)) continue;
-
-                        $timezones[] = clone $child;
-                        $collectedTimezones[] = $child->TZID;
-                        break;
-
-                }
-
-            }
-            // Destroy circular references to PHP will GC the object.
-            $nodeComp->destroy();
-            unset($nodeComp);
-
-        }
-
-        foreach ($timezones as $tz) $calendar->add($tz);
-        foreach ($objects as $obj) $calendar->add($obj);
-
-        return $calendar;
-
-    }
-
-    /**
-     * Returns a plugin name.
-     *
-     * Using this name other plugins will be able to access other plugins
-     * using \Sabre\DAV\Server::getPlugin
-     *
-     * @return string
-     */
-    public function getPluginName() {
-
-        return 'ics-export';
-
-    }
-
-    /**
-     * Returns a bunch of meta-data about the plugin.
-     *
-     * Providing this information is optional, and is mainly displayed by the
-     * Browser plugin.
-     *
-     * The description key in the returned array may contain html and will not
-     * be sanitized.
-     *
-     * @return array
-     */
-    public function getPluginInfo() {
-
-        return [
-            'name'        => $this->getPluginName(),
-            'description' => 'Adds the ability to export CalDAV calendars as a single iCalendar file.',
-            'link'        => 'http://sabre.io/dav/ics-export-plugin/',
-        ];
-
-    }
+	/**
+	 * Reference to Server class
+	 *
+	 * @var \Sabre\DAV\Server
+	 */
+	protected $server;
+
+	/**
+	 * Initializes the plugin and registers event handlers
+	 *
+	 * @param \Sabre\DAV\Server $server
+	 * @return void
+	 */
+	public function initialize(DAV\Server $server) {
+
+		$this->server = $server;
+		$server->on('method:GET', [$this, 'httpGet'], 90);
+		$server->on('browserButtonActions', function($path, $node, &$actions) {
+			if ($node instanceof ICalendar) {
+				$actions .= '<a href="' . htmlspecialchars($path, ENT_QUOTES, 'UTF-8') . '?export"><span class="oi" data-glyph="calendar"></span></a>';
+			}
+		});
+
+	}
+
+	/**
+	 * Intercepts GET requests on calendar urls ending with ?export.
+	 *
+	 * @param RequestInterface $request
+	 * @param ResponseInterface $response
+	 * @return bool
+	 */
+	public function httpGet(RequestInterface $request, ResponseInterface $response) {
+
+		$queryParams = $request->getQueryParameters();
+		if (!array_key_exists('export', $queryParams)) return;
+
+		$path = $request->getPath();
+
+		$node = $this->server->getProperties($path, [
+			'{DAV:}resourcetype',
+			'{DAV:}displayname',
+			'{http://sabredav.org/ns}sync-token',
+			'{DAV:}sync-token',
+			'{http://apple.com/ns/ical/}calendar-color',
+		]);
+
+		if (!isset($node['{DAV:}resourcetype']) || !$node['{DAV:}resourcetype']->is('{' . Plugin::NS_CALDAV . '}calendar')) {
+			return;
+		}
+		// Marking the transactionType, for logging purposes.
+		$this->server->transactionType = 'get-calendar-export';
+
+		$properties = $node;
+
+		$start = null;
+		$end = null;
+		$expand = false;
+		$componentType = false;
+		if (isset($queryParams['start'])) {
+			if (!ctype_digit($queryParams['start'])) {
+				throw new BadRequest('The start= parameter must contain a unix timestamp');
+			}
+			$start = DateTime::createFromFormat('U', $queryParams['start']);
+		}
+		if (isset($queryParams['end'])) {
+			if (!ctype_digit($queryParams['end'])) {
+				throw new BadRequest('The end= parameter must contain a unix timestamp');
+			}
+			$end = DateTime::createFromFormat('U', $queryParams['end']);
+		}
+		if (isset($queryParams['expand']) && !!$queryParams['expand']) {
+			if (!$start || !$end) {
+				throw new BadRequest('If you\'d like to expand recurrences, you must specify both a start= and end= parameter.');
+			}
+			$expand = true;
+			$componentType = 'VEVENT';
+		}
+		if (isset($queryParams['componentType'])) {
+			if (!in_array($queryParams['componentType'], ['VEVENT', 'VTODO', 'VJOURNAL'])) {
+				throw new BadRequest('You are not allowed to search for components of type: ' . $queryParams['componentType'] . ' here');
+			}
+			$componentType = $queryParams['componentType'];
+		}
+
+		$format = \Sabre\HTTP\Util::Negotiate(
+			$request->getHeader('Accept'),
+			[
+				'text/calendar',
+				'application/calendar+json',
+			]
+		);
+
+		if (isset($queryParams['accept'])) {
+			if ($queryParams['accept'] === 'application/calendar+json' || $queryParams['accept'] === 'jcal') {
+				$format = 'application/calendar+json';
+			}
+		}
+		if (!$format) {
+			$format = 'text/calendar';
+		}
+
+		$this->generateResponse($path, $start, $end, $expand, $componentType, $format, $properties, $response);
+
+		// Returning false to break the event chain
+		return false;
+
+	}
+
+	/**
+	 * This method is responsible for generating the actual, full response.
+	 *
+	 * @param string $path
+	 * @param DateTime|null $start
+	 * @param DateTime|null $end
+	 * @param bool $expand
+	 * @param string $componentType
+	 * @param string $format
+	 * @param array $properties
+	 * @param ResponseInterface $response
+	 */
+	protected function generateResponse($path, $start, $end, $expand, $componentType, $format, $properties, ResponseInterface $response) {
+
+		$calDataProp = '{' . Plugin::NS_CALDAV . '}calendar-data';
+
+		$blobs = [];
+		if ($start || $end || $componentType) {
+
+			// If there was a start or end filter, we need to enlist
+			// calendarQuery for speed.
+			$calendarNode = $this->server->tree->getNodeForPath($path);
+			$queryResult = $calendarNode->calendarQuery([
+				'name'         => 'VCALENDAR',
+				'comp-filters' => [
+					[
+						'name'           => $componentType,
+						'comp-filters'   => [],
+						'prop-filters'   => [],
+						'is-not-defined' => false,
+						'time-range'     => [
+							'start' => $start,
+							'end'   => $end,
+						],
+					],
+				],
+				'prop-filters'   => [],
+				'is-not-defined' => false,
+				'time-range'     => null,
+			]);
+
+			// queryResult is just a list of base urls. We need to prefix the
+			// calendar path.
+			$queryResult = array_map(
+				function($item) use ($path) {
+					return $path . '/' . $item;
+				},
+				$queryResult
+			);
+			$nodes = $this->server->getPropertiesForMultiplePaths($queryResult, [$calDataProp]);
+			unset($queryResult);
+
+		} else {
+			$nodes = $this->server->getPropertiesForPath($path, [$calDataProp], 1);
+		}
+
+		// Flattening the arrays
+		foreach ($nodes as $node) {
+			if (isset($node[200][$calDataProp])) {
+				$blobs[$node['href']] = $node[200][$calDataProp];
+			}
+		}
+		unset($nodes);
+
+		$mergedCalendar = $this->mergeObjects(
+			$properties,
+			$blobs
+		);
+
+		if ($expand) {
+			$calendarTimeZone = null;
+			// We're expanding, and for that we need to figure out the
+			// calendar's timezone.
+			$tzProp = '{' . Plugin::NS_CALDAV . '}calendar-timezone';
+			$tzResult = $this->server->getProperties($path, [$tzProp]);
+			if (isset($tzResult[$tzProp])) {
+				// This property contains a VCALENDAR with a single
+				// VTIMEZONE.
+				$vtimezoneObj = VObject\Reader::read($tzResult[$tzProp]);
+				$calendarTimeZone = $vtimezoneObj->VTIMEZONE->getTimeZone();
+				// Destroy circular references to PHP will GC the object.
+				$vtimezoneObj->destroy();
+				unset($vtimezoneObj);
+			} else {
+				// Defaulting to UTC.
+				$calendarTimeZone = new DateTimeZone('UTC');
+			}
+
+			$mergedCalendar = $mergedCalendar->expand($start, $end, $calendarTimeZone);
+		}
+
+		$response->setHeader('Content-Type', $format);
+
+		switch ($format) {
+			case 'text/calendar' :
+				$mergedCalendar = $mergedCalendar->serialize();
+				break;
+			case 'application/calendar+json' :
+				$mergedCalendar = json_encode($mergedCalendar->jsonSerialize());
+				break;
+		}
+
+		$response->setStatus(200);
+		$response->setBody($mergedCalendar);
+
+	}
+
+	/**
+	 * Merges all calendar objects, and builds one big iCalendar blob.
+	 *
+	 * @param array $properties Some CalDAV properties
+	 * @param array $inputObjects
+	 * @return VObject\Component\VCalendar
+	 */
+	public function mergeObjects(array $properties, array $inputObjects) {
+
+		$calendar = new VObject\Component\VCalendar();
+		$calendar->version = '2.0';
+		if (DAV\Server::$exposeVersion) {
+			$calendar->prodid = '-//SabreDAV//SabreDAV ' . DAV\Version::VERSION . '//EN';
+		} else {
+			$calendar->prodid = '-//SabreDAV//SabreDAV//EN';
+		}
+		if (isset($properties['{DAV:}displayname'])) {
+			$calendar->{'X-WR-CALNAME'} = $properties['{DAV:}displayname'];
+		}
+		if (isset($properties['{http://apple.com/ns/ical/}calendar-color'])) {
+			$calendar->{'X-APPLE-CALENDAR-COLOR'} = $properties['{http://apple.com/ns/ical/}calendar-color'];
+		}
+
+		$collectedTimezones = [];
+
+		$timezones = [];
+		$objects = [];
+
+		foreach ($inputObjects as $href => $inputObject) {
+
+			$nodeComp = VObject\Reader::read($inputObject);
+
+			foreach ($nodeComp->children() as $child) {
+
+				switch ($child->name) {
+					case 'VEVENT' :
+					case 'VTODO' :
+					case 'VJOURNAL' :
+						$objects[] = clone $child;
+						break;
+
+					// VTIMEZONE is special, because we need to filter out the duplicates
+					case 'VTIMEZONE' :
+						// Naively just checking tzid.
+						if (in_array((string)$child->TZID, $collectedTimezones)) continue;
+
+						$timezones[] = clone $child;
+						$collectedTimezones[] = $child->TZID;
+						break;
+
+				}
+
+			}
+			// Destroy circular references to PHP will GC the object.
+			$nodeComp->destroy();
+			unset($nodeComp);
+
+		}
+
+		foreach ($timezones as $tz) $calendar->add($tz);
+		foreach ($objects as $obj) $calendar->add($obj);
+
+		return $calendar;
+
+	}
+
+	/**
+	 * Returns a plugin name.
+	 *
+	 * Using this name other plugins will be able to access other plugins
+	 * using \Sabre\DAV\Server::getPlugin
+	 *
+	 * @return string
+	 */
+	public function getPluginName() {
+
+		return 'ics-export';
+
+	}
+
+	/**
+	 * Returns a bunch of meta-data about the plugin.
+	 *
+	 * Providing this information is optional, and is mainly displayed by the
+	 * Browser plugin.
+	 *
+	 * The description key in the returned array may contain html and will not
+	 * be sanitized.
+	 *
+	 * @return array
+	 */
+	public function getPluginInfo() {
+
+		return [
+			'name'        => $this->getPluginName(),
+			'description' => 'Adds the ability to export CalDAV calendars as a single iCalendar file.',
+			'link'        => 'http://sabre.io/dav/ics-export-plugin/',
+		];
+
+	}
 
 }

Doc Comments +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
      *
      * @param RequestInterface $request
      * @param ResponseInterface $response
-     * @return bool
+     * @return null|false
      */
     public function httpGet(RequestInterface $request, ResponseInterface $response) {
 

Braces +12 added lines, -4 removed lines

@@ -81,7 +81,9 @@ 
     public function httpGet(RequestInterface $request, ResponseInterface $response) {
 
         $queryParams = $request->getQueryParameters();
-        if (!array_key_exists('export', $queryParams)) return;
+        if (!array_key_exists('export', $queryParams)) {
+        	return;
+        }
 
         $path = $request->getPath();
 
@@ -306,7 +308,9 @@ 
                     // VTIMEZONE is special, because we need to filter out the duplicates
                     case 'VTIMEZONE' :
                         // Naively just checking tzid.
-                        if (in_array((string)$child->TZID, $collectedTimezones)) continue;
+                        if (in_array((string)$child->TZID, $collectedTimezones)) {
+                        	continue;
+                        }
 
                         $timezones[] = clone $child;
                         $collectedTimezones[] = $child->TZID;
@@ -321,8 +325,12 @@ 
 
         }
 
-        foreach ($timezones as $tz) $calendar->add($tz);
-        foreach ($objects as $obj) $calendar->add($obj);
+        foreach ($timezones as $tz) {
+        	$calendar->add($tz);
+        }
+        foreach ($objects as $obj) {
+        	$calendar->add($obj);
+        }
 
         return $calendar;
 

libraries/SabreDAV/CalDAV/Notifications/Collection.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,8 +2,8 @@
 
 namespace Sabre\CalDAV\Notifications;
 
-use Sabre\DAV;
 use Sabre\CalDAV;
+use Sabre\DAV;
 use Sabre\DAVACL;
 
 /**

Indentation +147 added lines, -147 removed lines

@@ -22,152 +22,152 @@
  */
 class Collection extends DAV\Collection implements ICollection, DAVACL\IACL {
 
-    /**
-     * The notification backend
-     *
-     * @var Sabre\CalDAV\Backend\NotificationSupport
-     */
-    protected $caldavBackend;
-
-    /**
-     * Principal uri
-     *
-     * @var string
-     */
-    protected $principalUri;
-
-    /**
-     * Constructor
-     *
-     * @param CalDAV\Backend\NotificationSupport $caldavBackend
-     * @param string $principalUri
-     */
-    public function __construct(CalDAV\Backend\NotificationSupport $caldavBackend, $principalUri) {
-
-        $this->caldavBackend = $caldavBackend;
-        $this->principalUri = $principalUri;
-
-    }
-
-    /**
-     * Returns all notifications for a principal
-     *
-     * @return array
-     */
-    public function getChildren() {
-
-        $children = [];
-        $notifications = $this->caldavBackend->getNotificationsForPrincipal($this->principalUri);
-
-        foreach ($notifications as $notification) {
-
-            $children[] = new Node(
-                $this->caldavBackend,
-                $this->principalUri,
-                $notification
-            );
-        }
-
-        return $children;
-
-    }
-
-    /**
-     * Returns the name of this object
-     *
-     * @return string
-     */
-    public function getName() {
-
-        return 'notifications';
-
-    }
-
-    /**
-     * Returns the owner principal
-     *
-     * This must be a url to a principal, or null if there's no owner
-     *
-     * @return string|null
-     */
-    public function getOwner() {
-
-        return $this->principalUri;
-
-    }
-
-    /**
-     * Returns a group principal
-     *
-     * This must be a url to a principal, or null if there's no owner
-     *
-     * @return string|null
-     */
-    public function getGroup() {
-
-        return null;
-
-    }
-
-    /**
-     * Returns a list of ACE's for this node.
-     *
-     * Each ACE has the following properties:
-     *   * 'privilege', a string such as {DAV:}read or {DAV:}write. These are
-     *     currently the only supported privileges
-     *   * 'principal', a url to the principal who owns the node
-     *   * 'protected' (optional), indicating that this ACE is not allowed to
-     *      be updated.
-     *
-     * @return array
-     */
-    public function getACL() {
-
-        return [
-            [
-                'principal' => $this->getOwner(),
-                'privilege' => '{DAV:}read',
-                'protected' => true,
-            ],
-            [
-                'principal' => $this->getOwner(),
-                'privilege' => '{DAV:}write',
-                'protected' => true,
-            ]
-        ];
-
-    }
-
-    /**
-     * Updates the ACL
-     *
-     * This method will receive a list of new ACE's as an array argument.
-     *
-     * @param array $acl
-     * @return void
-     */
-    public function setACL(array $acl) {
-
-        throw new DAV\Exception\NotImplemented('Updating ACLs is not implemented here');
-
-    }
-
-    /**
-     * Returns the list of supported privileges for this node.
-     *
-     * The returned data structure is a list of nested privileges.
-     * See Sabre\DAVACL\Plugin::getDefaultSupportedPrivilegeSet for a simple
-     * standard structure.
-     *
-     * If null is returned from this method, the default privilege set is used,
-     * which is fine for most common usecases.
-     *
-     * @return array|null
-     */
-    public function getSupportedPrivilegeSet() {
-
-        return null;
-
-    }
+	/**
+	 * The notification backend
+	 *
+	 * @var Sabre\CalDAV\Backend\NotificationSupport
+	 */
+	protected $caldavBackend;
+
+	/**
+	 * Principal uri
+	 *
+	 * @var string
+	 */
+	protected $principalUri;
+
+	/**
+	 * Constructor
+	 *
+	 * @param CalDAV\Backend\NotificationSupport $caldavBackend
+	 * @param string $principalUri
+	 */
+	public function __construct(CalDAV\Backend\NotificationSupport $caldavBackend, $principalUri) {
+
+		$this->caldavBackend = $caldavBackend;
+		$this->principalUri = $principalUri;
+
+	}
+
+	/**
+	 * Returns all notifications for a principal
+	 *
+	 * @return array
+	 */
+	public function getChildren() {
+
+		$children = [];
+		$notifications = $this->caldavBackend->getNotificationsForPrincipal($this->principalUri);
+
+		foreach ($notifications as $notification) {
+
+			$children[] = new Node(
+				$this->caldavBackend,
+				$this->principalUri,
+				$notification
+			);
+		}
+
+		return $children;
+
+	}
+
+	/**
+	 * Returns the name of this object
+	 *
+	 * @return string
+	 */
+	public function getName() {
+
+		return 'notifications';
+
+	}
+
+	/**
+	 * Returns the owner principal
+	 *
+	 * This must be a url to a principal, or null if there's no owner
+	 *
+	 * @return string|null
+	 */
+	public function getOwner() {
+
+		return $this->principalUri;
+
+	}
+
+	/**
+	 * Returns a group principal
+	 *
+	 * This must be a url to a principal, or null if there's no owner
+	 *
+	 * @return string|null
+	 */
+	public function getGroup() {
+
+		return null;
+
+	}
+
+	/**
+	 * Returns a list of ACE's for this node.
+	 *
+	 * Each ACE has the following properties:
+	 *   * 'privilege', a string such as {DAV:}read or {DAV:}write. These are
+	 *     currently the only supported privileges
+	 *   * 'principal', a url to the principal who owns the node
+	 *   * 'protected' (optional), indicating that this ACE is not allowed to
+	 *      be updated.
+	 *
+	 * @return array
+	 */
+	public function getACL() {
+
+		return [
+			[
+				'principal' => $this->getOwner(),
+				'privilege' => '{DAV:}read',
+				'protected' => true,
+			],
+			[
+				'principal' => $this->getOwner(),
+				'privilege' => '{DAV:}write',
+				'protected' => true,
+			]
+		];
+
+	}
+
+	/**
+	 * Updates the ACL
+	 *
+	 * This method will receive a list of new ACE's as an array argument.
+	 *
+	 * @param array $acl
+	 * @return void
+	 */
+	public function setACL(array $acl) {
+
+		throw new DAV\Exception\NotImplemented('Updating ACLs is not implemented here');
+
+	}
+
+	/**
+	 * Returns the list of supported privileges for this node.
+	 *
+	 * The returned data structure is a list of nested privileges.
+	 * See Sabre\DAVACL\Plugin::getDefaultSupportedPrivilegeSet for a simple
+	 * standard structure.
+	 *
+	 * If null is returned from this method, the default privilege set is used,
+	 * which is fine for most common usecases.
+	 *
+	 * @return array|null
+	 */
+	public function getSupportedPrivilegeSet() {
+
+		return null;
+
+	}
 
 }

libraries/SabreDAV/CalDAV/Notifications/Node.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,9 +2,9 @@
 
 namespace Sabre\CalDAV\Notifications;
 
-use Sabre\DAV;
 use Sabre\CalDAV;
 use Sabre\CalDAV\Xml\Notification\NotificationInterface;
+use Sabre\DAV;
 use Sabre\DAVACL;
 
 /**

Indentation +169 added lines, -169 removed lines

@@ -20,174 +20,174 @@
  */
 class Node extends DAV\File implements INode, DAVACL\IACL {
 
-    /**
-     * The notification backend
-     *
-     * @var Sabre\CalDAV\Backend\NotificationSupport
-     */
-    protected $caldavBackend;
-
-    /**
-     * The actual notification
-     *
-     * @var Sabre\CalDAV\Notifications\INotificationType
-     */
-    protected $notification;
-
-    /**
-     * Owner principal of the notification
-     *
-     * @var string
-     */
-    protected $principalUri;
-
-    /**
-     * Constructor
-     *
-     * @param CalDAV\Backend\NotificationSupport $caldavBackend
-     * @param string $principalUri
-     * @param NotificationInterface $notification
-     */
-    public function __construct(CalDAV\Backend\NotificationSupport $caldavBackend, $principalUri, NotificationInterface $notification) {
-
-        $this->caldavBackend = $caldavBackend;
-        $this->principalUri = $principalUri;
-        $this->notification = $notification;
-
-    }
-
-    /**
-     * Returns the path name for this notification
-     *
-     * @return id
-     */
-    public function getName() {
-
-        return $this->notification->getId() . '.xml';
-
-    }
-
-    /**
-     * Returns the etag for the notification.
-     *
-     * The etag must be surrounded by litteral double-quotes.
-     *
-     * @return string
-     */
-    public function getETag() {
-
-        return $this->notification->getETag();
-
-    }
-
-    /**
-     * This method must return an xml element, using the
-     * Sabre\CalDAV\Notifications\INotificationType classes.
-     *
-     * @return INotificationType
-     */
-    public function getNotificationType() {
-
-        return $this->notification;
-
-    }
-
-    /**
-     * Deletes this notification
-     *
-     * @return void
-     */
-    public function delete() {
-
-        $this->caldavBackend->deleteNotification($this->getOwner(), $this->notification);
-
-    }
-
-    /**
-     * Returns the owner principal
-     *
-     * This must be a url to a principal, or null if there's no owner
-     *
-     * @return string|null
-     */
-    public function getOwner() {
-
-        return $this->principalUri;
-
-    }
-
-    /**
-     * Returns a group principal
-     *
-     * This must be a url to a principal, or null if there's no owner
-     *
-     * @return string|null
-     */
-    public function getGroup() {
-
-        return null;
-
-    }
-
-    /**
-     * Returns a list of ACE's for this node.
-     *
-     * Each ACE has the following properties:
-     *   * 'privilege', a string such as {DAV:}read or {DAV:}write. These are
-     *     currently the only supported privileges
-     *   * 'principal', a url to the principal who owns the node
-     *   * 'protected' (optional), indicating that this ACE is not allowed to
-     *      be updated.
-     *
-     * @return array
-     */
-    public function getACL() {
-
-        return [
-            [
-                'principal' => $this->getOwner(),
-                'privilege' => '{DAV:}read',
-                'protected' => true,
-            ],
-            [
-                'principal' => $this->getOwner(),
-                'privilege' => '{DAV:}write',
-                'protected' => true,
-            ]
-        ];
-
-    }
-
-    /**
-     * Updates the ACL
-     *
-     * This method will receive a list of new ACE's as an array argument.
-     *
-     * @param array $acl
-     * @return void
-     */
-    public function setACL(array $acl) {
-
-        throw new DAV\Exception\NotImplemented('Updating ACLs is not implemented here');
-
-    }
-
-    /**
-     * Returns the list of supported privileges for this node.
-     *
-     * The returned data structure is a list of nested privileges.
-     * See Sabre\DAVACL\Plugin::getDefaultSupportedPrivilegeSet for a simple
-     * standard structure.
-     *
-     * If null is returned from this method, the default privilege set is used,
-     * which is fine for most common usecases.
-     *
-     * @return array|null
-     */
-    public function getSupportedPrivilegeSet() {
-
-        return null;
-
-    }
+	/**
+	 * The notification backend
+	 *
+	 * @var Sabre\CalDAV\Backend\NotificationSupport
+	 */
+	protected $caldavBackend;
+
+	/**
+	 * The actual notification
+	 *
+	 * @var Sabre\CalDAV\Notifications\INotificationType
+	 */
+	protected $notification;
+
+	/**
+	 * Owner principal of the notification
+	 *
+	 * @var string
+	 */
+	protected $principalUri;
+
+	/**
+	 * Constructor
+	 *
+	 * @param CalDAV\Backend\NotificationSupport $caldavBackend
+	 * @param string $principalUri
+	 * @param NotificationInterface $notification
+	 */
+	public function __construct(CalDAV\Backend\NotificationSupport $caldavBackend, $principalUri, NotificationInterface $notification) {
+
+		$this->caldavBackend = $caldavBackend;
+		$this->principalUri = $principalUri;
+		$this->notification = $notification;
+
+	}
+
+	/**
+	 * Returns the path name for this notification
+	 *
+	 * @return id
+	 */
+	public function getName() {
+
+		return $this->notification->getId() . '.xml';
+
+	}
+
+	/**
+	 * Returns the etag for the notification.
+	 *
+	 * The etag must be surrounded by litteral double-quotes.
+	 *
+	 * @return string
+	 */
+	public function getETag() {
+
+		return $this->notification->getETag();
+
+	}
+
+	/**
+	 * This method must return an xml element, using the
+	 * Sabre\CalDAV\Notifications\INotificationType classes.
+	 *
+	 * @return INotificationType
+	 */
+	public function getNotificationType() {
+
+		return $this->notification;
+
+	}
+
+	/**
+	 * Deletes this notification
+	 *
+	 * @return void
+	 */
+	public function delete() {
+
+		$this->caldavBackend->deleteNotification($this->getOwner(), $this->notification);
+
+	}
+
+	/**
+	 * Returns the owner principal
+	 *
+	 * This must be a url to a principal, or null if there's no owner
+	 *
+	 * @return string|null
+	 */
+	public function getOwner() {
+
+		return $this->principalUri;
+
+	}
+
+	/**
+	 * Returns a group principal
+	 *
+	 * This must be a url to a principal, or null if there's no owner
+	 *
+	 * @return string|null
+	 */
+	public function getGroup() {
+
+		return null;
+
+	}
+
+	/**
+	 * Returns a list of ACE's for this node.
+	 *
+	 * Each ACE has the following properties:
+	 *   * 'privilege', a string such as {DAV:}read or {DAV:}write. These are
+	 *     currently the only supported privileges
+	 *   * 'principal', a url to the principal who owns the node
+	 *   * 'protected' (optional), indicating that this ACE is not allowed to
+	 *      be updated.
+	 *
+	 * @return array
+	 */
+	public function getACL() {
+
+		return [
+			[
+				'principal' => $this->getOwner(),
+				'privilege' => '{DAV:}read',
+				'protected' => true,
+			],
+			[
+				'principal' => $this->getOwner(),
+				'privilege' => '{DAV:}write',
+				'protected' => true,
+			]
+		];
+
+	}
+
+	/**
+	 * Updates the ACL
+	 *
+	 * This method will receive a list of new ACE's as an array argument.
+	 *
+	 * @param array $acl
+	 * @return void
+	 */
+	public function setACL(array $acl) {
+
+		throw new DAV\Exception\NotImplemented('Updating ACLs is not implemented here');
+
+	}
+
+	/**
+	 * Returns the list of supported privileges for this node.
+	 *
+	 * The returned data structure is a list of nested privileges.
+	 * See Sabre\DAVACL\Plugin::getDefaultSupportedPrivilegeSet for a simple
+	 * standard structure.
+	 *
+	 * If null is returned from this method, the default privilege set is used,
+	 * which is fine for most common usecases.
+	 *
+	 * @return array|null
+	 */
+	public function getSupportedPrivilegeSet() {
+
+		return null;
+
+	}
 
 }

Doc Comments +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
     /**
      * Returns the path name for this notification
      *
-     * @return id
+     * @return string
      */
     public function getName() {
 

libraries/SabreDAV/CalDAV/Notifications/Plugin.php

Unused Use Statements +3 added lines, -3 removed lines

@@ -3,11 +3,11 @@
 namespace Sabre\CalDAV\Notifications;
 
 use Sabre\DAV;
-use Sabre\DAV\PropFind;
+use Sabre\DAVACL;
 use Sabre\DAV\INode as BaseINode;
-use Sabre\DAV\ServerPlugin;
+use Sabre\DAV\PropFind;
 use Sabre\DAV\Server;
-use Sabre\DAVACL;
+use Sabre\DAV\ServerPlugin;
 use Sabre\HTTP\RequestInterface;
 use Sabre\HTTP\ResponseInterface;
 

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
 
         $this->server = $server;
         $server->on('method:GET', [$this, 'httpGet'], 90);
-        $server->on('propFind',   [$this, 'propFind']);
+        $server->on('propFind', [$this, 'propFind']);
 
         $server->xml->namespaceMap[self::NS_CALENDARSERVER] = 'cs';
         $server->resourceTypeMapping['\\Sabre\\CalDAV\\Notifications\\ICollection'] = '{' . self::NS_CALENDARSERVER . '}notification';

Braces +3 added lines, -2 removed lines

@@ -135,8 +135,9 @@
             return;
         }
 
-        if (!$node instanceof INode)
-            return;
+        if (!$node instanceof INode) {
+                    return;
+        }
 
         $writer = $this->server->xml->getWriter();
         $writer->contextUri = $this->server->getBaseUri();

Indentation +150 added lines, -150 removed lines

@@ -26,155 +26,155 @@
  */
 class Plugin extends ServerPlugin {
 
-    /**
-     * This is the namespace for the proprietary calendarserver extensions
-     */
-    const NS_CALENDARSERVER = 'http://calendarserver.org/ns/';
-
-    /**
-     * Reference to the main server object.
-     *
-     * @var Server
-     */
-    protected $server;
-
-    /**
-     * Returns a plugin name.
-     *
-     * Using this name other plugins will be able to access other plugins
-     * using \Sabre\DAV\Server::getPlugin
-     *
-     * @return string
-     */
-    public function getPluginName() {
-
-        return 'notifications';
-
-    }
-
-    /**
-     * This initializes the plugin.
-     *
-     * This function is called by Sabre\DAV\Server, after
-     * addPlugin is called.
-     *
-     * This method should set up the required event subscriptions.
-     *
-     * @param Server $server
-     * @return void
-     */
-    public function initialize(Server $server) {
-
-        $this->server = $server;
-        $server->on('method:GET', [$this, 'httpGet'], 90);
-        $server->on('propFind',   [$this, 'propFind']);
-
-        $server->xml->namespaceMap[self::NS_CALENDARSERVER] = 'cs';
-        $server->resourceTypeMapping['\\Sabre\\CalDAV\\Notifications\\ICollection'] = '{' . self::NS_CALENDARSERVER . '}notification';
-
-        array_push($server->protectedProperties,
-            '{' . self::NS_CALENDARSERVER . '}notification-URL',
-            '{' . self::NS_CALENDARSERVER . '}notificationtype'
-        );
-
-    }
-
-    /**
-     * PropFind
-     *
-     * @param PropFind $propFind
-     * @param BaseINode $node
-     * @return void
-     */
-    public function propFind(PropFind $propFind, BaseINode $node) {
-
-        $caldavPlugin = $this->server->getPlugin('caldav');
-
-        if ($node instanceof DAVACL\IPrincipal) {
-
-            $principalUrl = $node->getPrincipalUrl();
-
-            // notification-URL property
-            $propFind->handle('{' . self::NS_CALENDARSERVER . '}notification-URL', function() use ($principalUrl, $caldavPlugin) {
-
-                $notificationPath = $caldavPlugin->getCalendarHomeForPrincipal($principalUrl) . '/notifications/';
-                return new DAV\Xml\Property\Href($notificationPath);
-
-            });
-
-        }
-
-        if ($node instanceof INode) {
-
-            $propFind->handle(
-                '{' . self::NS_CALENDARSERVER . '}notificationtype',
-                [$node, 'getNotificationType']
-            );
-
-        }
-
-    }
-
-    /**
-     * This event is triggered before the usual GET request handler.
-     *
-     * We use this to intercept GET calls to notification nodes, and return the
-     * proper response.
-     *
-     * @param RequestInterface $request
-     * @param ResponseInterface $response
-     * @return void
-     */
-    public function httpGet(RequestInterface $request, ResponseInterface $response) {
-
-        $path = $request->getPath();
-
-        try {
-            $node = $this->server->tree->getNodeForPath($path);
-        } catch (DAV\Exception\NotFound $e) {
-            return;
-        }
-
-        if (!$node instanceof INode)
-            return;
-
-        $writer = $this->server->xml->getWriter();
-        $writer->contextUri = $this->server->getBaseUri();
-        $writer->openMemory();
-        $writer->startDocument('1.0', 'UTF-8');
-        $writer->startElement('{http://calendarserver.org/ns/}notification');
-        $node->getNotificationType()->xmlSerializeFull($writer);
-        $writer->endElement();
-
-        $response->setHeader('Content-Type', 'application/xml');
-        $response->setHeader('ETag', $node->getETag());
-        $response->setStatus(200);
-        $response->setBody($writer->outputMemory());
-
-        // Return false to break the event chain.
-        return false;
-
-    }
-
-    /**
-     * Returns a bunch of meta-data about the plugin.
-     *
-     * Providing this information is optional, and is mainly displayed by the
-     * Browser plugin.
-     *
-     * The description key in the returned array may contain html and will not
-     * be sanitized.
-     *
-     * @return array
-     */
-    public function getPluginInfo() {
-
-        return [
-            'name'        => $this->getPluginName(),
-            'description' => 'Adds support for caldav-notifications, which is required to enable caldav-sharing.',
-            'link'        => 'http://sabre.io/dav/caldav-sharing/',
-        ];
-
-    }
+	/**
+	 * This is the namespace for the proprietary calendarserver extensions
+	 */
+	const NS_CALENDARSERVER = 'http://calendarserver.org/ns/';
+
+	/**
+	 * Reference to the main server object.
+	 *
+	 * @var Server
+	 */
+	protected $server;
+
+	/**
+	 * Returns a plugin name.
+	 *
+	 * Using this name other plugins will be able to access other plugins
+	 * using \Sabre\DAV\Server::getPlugin
+	 *
+	 * @return string
+	 */
+	public function getPluginName() {
+
+		return 'notifications';
+
+	}
+
+	/**
+	 * This initializes the plugin.
+	 *
+	 * This function is called by Sabre\DAV\Server, after
+	 * addPlugin is called.
+	 *
+	 * This method should set up the required event subscriptions.
+	 *
+	 * @param Server $server
+	 * @return void
+	 */
+	public function initialize(Server $server) {
+
+		$this->server = $server;
+		$server->on('method:GET', [$this, 'httpGet'], 90);
+		$server->on('propFind',   [$this, 'propFind']);
+
+		$server->xml->namespaceMap[self::NS_CALENDARSERVER] = 'cs';
+		$server->resourceTypeMapping['\\Sabre\\CalDAV\\Notifications\\ICollection'] = '{' . self::NS_CALENDARSERVER . '}notification';
+
+		array_push($server->protectedProperties,
+			'{' . self::NS_CALENDARSERVER . '}notification-URL',
+			'{' . self::NS_CALENDARSERVER . '}notificationtype'
+		);
+
+	}
+
+	/**
+	 * PropFind
+	 *
+	 * @param PropFind $propFind
+	 * @param BaseINode $node
+	 * @return void
+	 */
+	public function propFind(PropFind $propFind, BaseINode $node) {
+
+		$caldavPlugin = $this->server->getPlugin('caldav');
+
+		if ($node instanceof DAVACL\IPrincipal) {
+
+			$principalUrl = $node->getPrincipalUrl();
+
+			// notification-URL property
+			$propFind->handle('{' . self::NS_CALENDARSERVER . '}notification-URL', function() use ($principalUrl, $caldavPlugin) {
+
+				$notificationPath = $caldavPlugin->getCalendarHomeForPrincipal($principalUrl) . '/notifications/';
+				return new DAV\Xml\Property\Href($notificationPath);
+
+			});
+
+		}
+
+		if ($node instanceof INode) {
+
+			$propFind->handle(
+				'{' . self::NS_CALENDARSERVER . '}notificationtype',
+				[$node, 'getNotificationType']
+			);
+
+		}
+
+	}
+
+	/**
+	 * This event is triggered before the usual GET request handler.
+	 *
+	 * We use this to intercept GET calls to notification nodes, and return the
+	 * proper response.
+	 *
+	 * @param RequestInterface $request
+	 * @param ResponseInterface $response
+	 * @return void
+	 */
+	public function httpGet(RequestInterface $request, ResponseInterface $response) {
+
+		$path = $request->getPath();
+
+		try {
+			$node = $this->server->tree->getNodeForPath($path);
+		} catch (DAV\Exception\NotFound $e) {
+			return;
+		}
+
+		if (!$node instanceof INode)
+			return;
+
+		$writer = $this->server->xml->getWriter();
+		$writer->contextUri = $this->server->getBaseUri();
+		$writer->openMemory();
+		$writer->startDocument('1.0', 'UTF-8');
+		$writer->startElement('{http://calendarserver.org/ns/}notification');
+		$node->getNotificationType()->xmlSerializeFull($writer);
+		$writer->endElement();
+
+		$response->setHeader('Content-Type', 'application/xml');
+		$response->setHeader('ETag', $node->getETag());
+		$response->setStatus(200);
+		$response->setBody($writer->outputMemory());
+
+		// Return false to break the event chain.
+		return false;
+
+	}
+
+	/**
+	 * Returns a bunch of meta-data about the plugin.
+	 *
+	 * Providing this information is optional, and is mainly displayed by the
+	 * Browser plugin.
+	 *
+	 * The description key in the returned array may contain html and will not
+	 * be sanitized.
+	 *
+	 * @return array
+	 */
+	public function getPluginInfo() {
+
+		return [
+			'name'        => $this->getPluginName(),
+			'description' => 'Adds support for caldav-notifications, which is required to enable caldav-sharing.',
+			'link'        => 'http://sabre.io/dav/caldav-sharing/',
+		];
+
+	}
 
 }

Doc Comments +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
      *
      * @param RequestInterface $request
      * @param ResponseInterface $response
-     * @return void
+     * @return null|false
      */
     public function httpGet(RequestInterface $request, ResponseInterface $response) {
 

libraries/SabreDAV/CalDAV/Plugin.php

Unused Use Statements +3 added lines, -3 removed lines

@@ -4,15 +4,15 @@
 
 use DateTimeZone;
 use Sabre\DAV;
+use Sabre\DAVACL;
 use Sabre\DAV\Exception\BadRequest;
 use Sabre\DAV\MkCol;
 use Sabre\DAV\Xml\Property\Href;
-use Sabre\DAVACL;
-use Sabre\VObject;
 use Sabre\HTTP;
-use Sabre\Uri;
 use Sabre\HTTP\RequestInterface;
 use Sabre\HTTP\ResponseInterface;
+use Sabre\Uri;
+use Sabre\VObject;
 
 /**
  * CalDAV plugin

Spacing +10 added lines, -10 removed lines

@@ -179,13 +179,13 @@ 
 
         $this->server = $server;
 
-        $server->on('method:MKCALENDAR',   [$this, 'httpMkCalendar']);
-        $server->on('report',              [$this, 'report']);
-        $server->on('propFind',            [$this, 'propFind']);
-        $server->on('onHTMLActionsPanel',  [$this, 'htmlActionsPanel']);
-        $server->on('beforeCreateFile',    [$this, 'beforeCreateFile']);
-        $server->on('beforeWriteContent',  [$this, 'beforeWriteContent']);
-        $server->on('afterMethod:GET',     [$this, 'httpAfterGET']);
+        $server->on('method:MKCALENDAR', [$this, 'httpMkCalendar']);
+        $server->on('report', [$this, 'report']);
+        $server->on('propFind', [$this, 'propFind']);
+        $server->on('onHTMLActionsPanel', [$this, 'htmlActionsPanel']);
+        $server->on('beforeCreateFile', [$this, 'beforeCreateFile']);
+        $server->on('beforeWriteContent', [$this, 'beforeWriteContent']);
+        $server->on('afterMethod:GET', [$this, 'httpAfterGET']);
 
         $server->xml->namespaceMap[self::NS_CALDAV] = 'cal';
         $server->xml->namespaceMap[self::NS_CALENDARSERVER] = 'cs';
@@ -294,7 +294,7 @@ 
         if (isset($properties['{DAV:}resourcetype'])) {
             $resourceType = $properties['{DAV:}resourcetype']->getValue();
         } else {
-            $resourceType = ['{DAV:}collection','{urn:ietf:params:xml:ns:caldav}calendar'];
+            $resourceType = ['{DAV:}collection', '{urn:ietf:params:xml:ns:caldav}calendar'];
         }
 
         $this->server->createCollection($path, new MkCol($resourceType, $properties));
@@ -881,12 +881,12 @@ 
                         if (!isset($component->UID)) {
                             throw new DAV\Exception\BadRequest('Every ' . $component->name . ' component must have an UID');
                         }
-                        $foundUID = (string)$component->UID;
+                        $foundUID = (string) $component->UID;
                     } else {
                         if ($foundType !== $component->name) {
                             throw new DAV\Exception\BadRequest('A calendar object must only contain 1 component. We found a ' . $component->name . ' as well as a ' . $foundType);
                         }
-                        if ($foundUID !== (string)$component->UID) {
+                        if ($foundUID !== (string) $component->UID) {
                             throw new DAV\Exception\BadRequest('Every ' . $component->name . ' in this object must have identical UIDs');
                         }
                     }

Indentation +986 added lines, -986 removed lines

@@ -26,929 +26,929 @@ 
  */
 class Plugin extends DAV\ServerPlugin {
 
-    /**
-     * This is the official CalDAV namespace
-     */
-    const NS_CALDAV = 'urn:ietf:params:xml:ns:caldav';
-
-    /**
-     * This is the namespace for the proprietary calendarserver extensions
-     */
-    const NS_CALENDARSERVER = 'http://calendarserver.org/ns/';
-
-    /**
-     * The hardcoded root for calendar objects. It is unfortunate
-     * that we're stuck with it, but it will have to do for now
-     */
-    const CALENDAR_ROOT = 'calendars';
-
-    /**
-     * Reference to server object
-     *
-     * @var DAV\Server
-     */
-    protected $server;
-
-    /**
-     * The default PDO storage uses a MySQL MEDIUMBLOB for iCalendar data,
-     * which can hold up to 2^24 = 16777216 bytes. This is plenty. We're
-     * capping it to 10M here.
-     */
-    protected $maxResourceSize = 10000000;
-
-    /**
-     * Use this method to tell the server this plugin defines additional
-     * HTTP methods.
-     *
-     * This method is passed a uri. It should only return HTTP methods that are
-     * available for the specified uri.
-     *
-     * @param string $uri
-     * @return array
-     */
-    public function getHTTPMethods($uri) {
-
-        // The MKCALENDAR is only available on unmapped uri's, whose
-        // parents extend IExtendedCollection
-        list($parent, $name) = Uri\split($uri);
-
-        $node = $this->server->tree->getNodeForPath($parent);
-
-        if ($node instanceof DAV\IExtendedCollection) {
-            try {
-                $node->getChild($name);
-            } catch (DAV\Exception\NotFound $e) {
-                return ['MKCALENDAR'];
-            }
-        }
-        return [];
-
-    }
-
-    /**
-     * Returns the path to a principal's calendar home.
-     *
-     * The return url must not end with a slash.
-     * This function should return null in case a principal did not have
-     * a calendar home.
-     *
-     * @param string $principalUrl
-     * @return string
-     */
-    public function getCalendarHomeForPrincipal($principalUrl) {
-
-        // The default behavior for most sabre/dav servers is that there is a
-        // principals root node, which contains users directly under it.
-        //
-        // This function assumes that there are two components in a principal
-        // path. If there's more, we don't return a calendar home. This
-        // excludes things like the calendar-proxy-read principal (which it
-        // should).
-        $parts = explode('/', trim($principalUrl, '/'));
-        if (count($parts) !== 2) return;
-        if ($parts[0] !== 'principals') return;
-
-        return self::CALENDAR_ROOT . '/' . $parts[1];
-
-    }
-
-    /**
-     * Returns a list of features for the DAV: HTTP header.
-     *
-     * @return array
-     */
-    public function getFeatures() {
-
-        return ['calendar-access', 'calendar-proxy'];
-
-    }
-
-    /**
-     * Returns a plugin name.
-     *
-     * Using this name other plugins will be able to access other plugins
-     * using DAV\Server::getPlugin
-     *
-     * @return string
-     */
-    public function getPluginName() {
-
-        return 'caldav';
-
-    }
-
-    /**
-     * Returns a list of reports this plugin supports.
-     *
-     * This will be used in the {DAV:}supported-report-set property.
-     * Note that you still need to subscribe to the 'report' event to actually
-     * implement them
-     *
-     * @param string $uri
-     * @return array
-     */
-    public function getSupportedReportSet($uri) {
-
-        $node = $this->server->tree->getNodeForPath($uri);
-
-        $reports = [];
-        if ($node instanceof ICalendarObjectContainer || $node instanceof ICalendarObject) {
-            $reports[] = '{' . self::NS_CALDAV . '}calendar-multiget';
-            $reports[] = '{' . self::NS_CALDAV . '}calendar-query';
-        }
-        if ($node instanceof ICalendar) {
-            $reports[] = '{' . self::NS_CALDAV . '}free-busy-query';
-        }
-        // iCal has a bug where it assumes that sync support is enabled, only
-        // if we say we support it on the calendar-home, even though this is
-        // not actually the case.
-        if ($node instanceof CalendarHome && $this->server->getPlugin('sync')) {
-            $reports[] = '{DAV:}sync-collection';
-        }
-        return $reports;
-
-    }
-
-    /**
-     * Initializes the plugin
-     *
-     * @param DAV\Server $server
-     * @return void
-     */
-    public function initialize(DAV\Server $server) {
-
-        $this->server = $server;
-
-        $server->on('method:MKCALENDAR',   [$this, 'httpMkCalendar']);
-        $server->on('report',              [$this, 'report']);
-        $server->on('propFind',            [$this, 'propFind']);
-        $server->on('onHTMLActionsPanel',  [$this, 'htmlActionsPanel']);
-        $server->on('beforeCreateFile',    [$this, 'beforeCreateFile']);
-        $server->on('beforeWriteContent',  [$this, 'beforeWriteContent']);
-        $server->on('afterMethod:GET',     [$this, 'httpAfterGET']);
-
-        $server->xml->namespaceMap[self::NS_CALDAV] = 'cal';
-        $server->xml->namespaceMap[self::NS_CALENDARSERVER] = 'cs';
-
-        $server->xml->elementMap['{' . self::NS_CALDAV . '}supported-calendar-component-set'] = 'Sabre\\CalDAV\\Xml\\Property\\SupportedCalendarComponentSet';
-        $server->xml->elementMap['{' . self::NS_CALDAV . '}calendar-query'] = 'Sabre\\CalDAV\\Xml\\Request\\CalendarQueryReport';
-        $server->xml->elementMap['{' . self::NS_CALDAV . '}calendar-multiget'] = 'Sabre\\CalDAV\\Xml\\Request\\CalendarMultiGetReport';
-        $server->xml->elementMap['{' . self::NS_CALDAV . '}free-busy-query'] = 'Sabre\\CalDAV\\Xml\\Request\\FreeBusyQueryReport';
-        $server->xml->elementMap['{' . self::NS_CALDAV . '}mkcalendar'] = 'Sabre\\CalDAV\\Xml\\Request\\MkCalendar';
-        $server->xml->elementMap['{' . self::NS_CALDAV . '}schedule-calendar-transp'] = 'Sabre\\CalDAV\\Xml\\Property\\ScheduleCalendarTransp';
-        $server->xml->elementMap['{' . self::NS_CALDAV . '}supported-calendar-component-set'] = 'Sabre\\CalDAV\\Xml\\Property\\SupportedCalendarComponentSet';
-
-        $server->resourceTypeMapping['\\Sabre\\CalDAV\\ICalendar'] = '{urn:ietf:params:xml:ns:caldav}calendar';
-
-        $server->resourceTypeMapping['\\Sabre\\CalDAV\\Principal\\IProxyRead'] = '{http://calendarserver.org/ns/}calendar-proxy-read';
-        $server->resourceTypeMapping['\\Sabre\\CalDAV\\Principal\\IProxyWrite'] = '{http://calendarserver.org/ns/}calendar-proxy-write';
-
-        array_push($server->protectedProperties,
-
-            '{' . self::NS_CALDAV . '}supported-calendar-component-set',
-            '{' . self::NS_CALDAV . '}supported-calendar-data',
-            '{' . self::NS_CALDAV . '}max-resource-size',
-            '{' . self::NS_CALDAV . '}min-date-time',
-            '{' . self::NS_CALDAV . '}max-date-time',
-            '{' . self::NS_CALDAV . '}max-instances',
-            '{' . self::NS_CALDAV . '}max-attendees-per-instance',
-            '{' . self::NS_CALDAV . '}calendar-home-set',
-            '{' . self::NS_CALDAV . '}supported-collation-set',
-            '{' . self::NS_CALDAV . '}calendar-data',
-
-            // CalendarServer extensions
-            '{' . self::NS_CALENDARSERVER . '}getctag',
-            '{' . self::NS_CALENDARSERVER . '}calendar-proxy-read-for',
-            '{' . self::NS_CALENDARSERVER . '}calendar-proxy-write-for'
-
-        );
-
-        if ($aclPlugin = $server->getPlugin('acl')) {
-            $aclPlugin->principalSearchPropertySet['{' . self::NS_CALDAV . '}calendar-user-address-set'] = 'Calendar address';
-        }
-    }
-
-    /**
-     * This functions handles REPORT requests specific to CalDAV
-     *
-     * @param string $reportName
-     * @param mixed $report
-     * @return bool
-     */
-    public function report($reportName, $report) {
-
-        switch ($reportName) {
-            case '{' . self::NS_CALDAV . '}calendar-multiget' :
-                $this->server->transactionType = 'report-calendar-multiget';
-                $this->calendarMultiGetReport($report);
-                return false;
-            case '{' . self::NS_CALDAV . '}calendar-query' :
-                $this->server->transactionType = 'report-calendar-query';
-                $this->calendarQueryReport($report);
-                return false;
-            case '{' . self::NS_CALDAV . '}free-busy-query' :
-                $this->server->transactionType = 'report-free-busy-query';
-                $this->freeBusyQueryReport($report);
-                return false;
-
-        }
-
-
-    }
-
-    /**
-     * This function handles the MKCALENDAR HTTP method, which creates
-     * a new calendar.
-     *
-     * @param RequestInterface $request
-     * @param ResponseInterface $response
-     * @return bool
-     */
-    public function httpMkCalendar(RequestInterface $request, ResponseInterface $response) {
-
-        $body = $request->getBodyAsString();
-        $path = $request->getPath();
-
-        $properties = [];
-
-        if ($body) {
-
-            try {
-                $mkcalendar = $this->server->xml->expect(
-                    '{urn:ietf:params:xml:ns:caldav}mkcalendar',
-                    $body
-                );
-            } catch (\Sabre\Xml\ParseException $e) {
-                throw new BadRequest($e->getMessage(), null, $e);
-            }
-            $properties = $mkcalendar->getProperties();
-
-        }
-
-        // iCal abuses MKCALENDAR since iCal 10.9.2 to create server-stored
-        // subscriptions. Before that it used MKCOL which was the correct way
-        // to do this.
-        //
-        // If the body had a {DAV:}resourcetype, it means we stumbled upon this
-        // request, and we simply use it instead of the pre-defined list.
-        if (isset($properties['{DAV:}resourcetype'])) {
-            $resourceType = $properties['{DAV:}resourcetype']->getValue();
-        } else {
-            $resourceType = ['{DAV:}collection','{urn:ietf:params:xml:ns:caldav}calendar'];
-        }
-
-        $this->server->createCollection($path, new MkCol($resourceType, $properties));
-
-        $this->server->httpResponse->setStatus(201);
-        $this->server->httpResponse->setHeader('Content-Length', 0);
-
-        // This breaks the method chain.
-        return false;
-    }
-
-    /**
-     * PropFind
-     *
-     * This method handler is invoked before any after properties for a
-     * resource are fetched. This allows us to add in any CalDAV specific
-     * properties.
-     *
-     * @param DAV\PropFind $propFind
-     * @param DAV\INode $node
-     * @return void
-     */
-    public function propFind(DAV\PropFind $propFind, DAV\INode $node) {
-
-        $ns = '{' . self::NS_CALDAV . '}';
-
-        if ($node instanceof ICalendarObjectContainer) {
-
-            $propFind->handle($ns . 'max-resource-size', $this->maxResourceSize);
-            $propFind->handle($ns . 'supported-calendar-data', function() {
-                return new Xml\Property\SupportedCalendarData();
-            });
-            $propFind->handle($ns . 'supported-collation-set', function() {
-                return new Xml\Property\SupportedCollationSet();
-            });
-
-        }
-
-        if ($node instanceof DAVACL\IPrincipal) {
-
-            $principalUrl = $node->getPrincipalUrl();
-
-            $propFind->handle('{' . self::NS_CALDAV . '}calendar-home-set', function() use ($principalUrl) {
-
-                $calendarHomePath = $this->getCalendarHomeForPrincipal($principalUrl);
-                if (is_null($calendarHomePath)) return null;
-                return new Href($calendarHomePath . '/');
-
-            });
-            // The calendar-user-address-set property is basically mapped to
-            // the {DAV:}alternate-URI-set property.
-            $propFind->handle('{' . self::NS_CALDAV . '}calendar-user-address-set', function() use ($node) {
-                $addresses = $node->getAlternateUriSet();
-                $addresses[] = $this->server->getBaseUri() . $node->getPrincipalUrl() . '/';
-                return new Href($addresses, false);
-            });
-            // For some reason somebody thought it was a good idea to add
-            // another one of these properties. We're supporting it too.
-            $propFind->handle('{' . self::NS_CALENDARSERVER . '}email-address-set', function() use ($node) {
-                $addresses = $node->getAlternateUriSet();
-                $emails = [];
-                foreach ($addresses as $address) {
-                    if (substr($address, 0, 7) === 'mailto:') {
-                        $emails[] = substr($address, 7);
-                    }
-                }
-                return new Xml\Property\EmailAddressSet($emails);
-            });
-
-            // These two properties are shortcuts for ical to easily find
-            // other principals this principal has access to.
-            $propRead = '{' . self::NS_CALENDARSERVER . '}calendar-proxy-read-for';
-            $propWrite = '{' . self::NS_CALENDARSERVER . '}calendar-proxy-write-for';
-
-            if ($propFind->getStatus($propRead) === 404 || $propFind->getStatus($propWrite) === 404) {
-
-                $aclPlugin = $this->server->getPlugin('acl');
-                $membership = $aclPlugin->getPrincipalMembership($propFind->getPath());
-                $readList = [];
-                $writeList = [];
-
-                foreach ($membership as $group) {
-
-                    $groupNode = $this->server->tree->getNodeForPath($group);
-
-                    $listItem = Uri\split($group)[0] . '/';
-
-                    // If the node is either ap proxy-read or proxy-write
-                    // group, we grab the parent principal and add it to the
-                    // list.
-                    if ($groupNode instanceof Principal\IProxyRead) {
-                        $readList[] = $listItem;
-                    }
-                    if ($groupNode instanceof Principal\IProxyWrite) {
-                        $writeList[] = $listItem;
-                    }
-
-                }
-
-                $propFind->set($propRead, new Href($readList));
-                $propFind->set($propWrite, new Href($writeList));
-
-            }
-
-        } // instanceof IPrincipal
-
-        if ($node instanceof ICalendarObject) {
-
-            // The calendar-data property is not supposed to be a 'real'
-            // property, but in large chunks of the spec it does act as such.
-            // Therefore we simply expose it as a property.
-            $propFind->handle('{' . self::NS_CALDAV . '}calendar-data', function() use ($node) {
-                $val = $node->get();
-                if (is_resource($val))
-                    $val = stream_get_contents($val);
-
-                // Taking out \r to not screw up the xml output
-                return str_replace("\r", "", $val);
-
-            });
-
-        }
-
-    }
-
-    /**
-     * This function handles the calendar-multiget REPORT.
-     *
-     * This report is used by the client to fetch the content of a series
-     * of urls. Effectively avoiding a lot of redundant requests.
-     *
-     * @param CalendarMultiGetReport $report
-     * @return void
-     */
-    public function calendarMultiGetReport($report) {
-
-        $needsJson = $report->contentType === 'application/calendar+json';
-
-        $timeZones = [];
-        $propertyList = [];
-
-        $paths = array_map(
-            [$this->server, 'calculateUri'],
-            $report->hrefs
-        );
-
-        foreach ($this->server->getPropertiesForMultiplePaths($paths, $report->properties) as $uri => $objProps) {
-
-            if (($needsJson || $report->expand) && isset($objProps[200]['{' . self::NS_CALDAV . '}calendar-data'])) {
-                $vObject = VObject\Reader::read($objProps[200]['{' . self::NS_CALDAV . '}calendar-data']);
-
-                if ($report->expand) {
-                    // We're expanding, and for that we need to figure out the
-                    // calendar's timezone.
-                    list($calendarPath) = Uri\split($uri);
-                    if (!isset($timeZones[$calendarPath])) {
-                        // Checking the calendar-timezone property.
-                        $tzProp = '{' . self::NS_CALDAV . '}calendar-timezone';
-                        $tzResult = $this->server->getProperties($calendarPath, [$tzProp]);
-                        if (isset($tzResult[$tzProp])) {
-                            // This property contains a VCALENDAR with a single
-                            // VTIMEZONE.
-                            $vtimezoneObj = VObject\Reader::read($tzResult[$tzProp]);
-                            $timeZone = $vtimezoneObj->VTIMEZONE->getTimeZone();
-                        } else {
-                            // Defaulting to UTC.
-                            $timeZone = new DateTimeZone('UTC');
-                        }
-                        $timeZones[$calendarPath] = $timeZone;
-                    }
-
-                    $vObject = $vObject->expand($report->expand['start'], $report->expand['end'], $timeZones[$calendarPath]);
-                }
-                if ($needsJson) {
-                    $objProps[200]['{' . self::NS_CALDAV . '}calendar-data'] = json_encode($vObject->jsonSerialize());
-                } else {
-                    $objProps[200]['{' . self::NS_CALDAV . '}calendar-data'] = $vObject->serialize();
-                }
-                // Destroy circular references so PHP will garbage collect the
-                // object.
-                $vObject->destroy();
-            }
-
-            $propertyList[] = $objProps;
-
-        }
-
-        $prefer = $this->server->getHTTPPrefer();
-
-        $this->server->httpResponse->setStatus(207);
-        $this->server->httpResponse->setHeader('Content-Type', 'application/xml; charset=utf-8');
-        $this->server->httpResponse->setHeader('Vary', 'Brief,Prefer');
-        $this->server->httpResponse->setBody($this->server->generateMultiStatus($propertyList, $prefer['return'] === 'minimal'));
-
-    }
-
-    /**
-     * This function handles the calendar-query REPORT
-     *
-     * This report is used by clients to request calendar objects based on
-     * complex conditions.
-     *
-     * @param Xml\Request\CalendarQueryReport $report
-     * @return void
-     */
-    public function calendarQueryReport($report) {
-
-        $path = $this->server->getRequestUri();
-
-        $needsJson = $report->contentType === 'application/calendar+json';
-
-        $node = $this->server->tree->getNodeForPath($this->server->getRequestUri());
-        $depth = $this->server->getHTTPDepth(0);
-
-        // The default result is an empty array
-        $result = [];
-
-        $calendarTimeZone = null;
-        if ($report->expand) {
-            // We're expanding, and for that we need to figure out the
-            // calendar's timezone.
-            $tzProp = '{' . self::NS_CALDAV . '}calendar-timezone';
-            $tzResult = $this->server->getProperties($path, [$tzProp]);
-            if (isset($tzResult[$tzProp])) {
-                // This property contains a VCALENDAR with a single
-                // VTIMEZONE.
-                $vtimezoneObj = VObject\Reader::read($tzResult[$tzProp]);
-                $calendarTimeZone = $vtimezoneObj->VTIMEZONE->getTimeZone();
-
-                // Destroy circular references so PHP will garbage collect the
-                // object.
-                $vtimezoneObj->destroy();
-            } else {
-                // Defaulting to UTC.
-                $calendarTimeZone = new DateTimeZone('UTC');
-            }
-        }
-
-        // The calendarobject was requested directly. In this case we handle
-        // this locally.
-        if ($depth == 0 && $node instanceof ICalendarObject) {
-
-            $requestedCalendarData = true;
-            $requestedProperties = $report->properties;
-
-            if (!in_array('{urn:ietf:params:xml:ns:caldav}calendar-data', $requestedProperties)) {
-
-                // We always retrieve calendar-data, as we need it for filtering.
-                $requestedProperties[] = '{urn:ietf:params:xml:ns:caldav}calendar-data';
-
-                // If calendar-data wasn't explicitly requested, we need to remove
-                // it after processing.
-                $requestedCalendarData = false;
-            }
-
-            $properties = $this->server->getPropertiesForPath(
-                $path,
-                $requestedProperties,
-                0
-            );
-
-            // This array should have only 1 element, the first calendar
-            // object.
-            $properties = current($properties);
-
-            // If there wasn't any calendar-data returned somehow, we ignore
-            // this.
-            if (isset($properties[200]['{urn:ietf:params:xml:ns:caldav}calendar-data'])) {
-
-                $validator = new CalendarQueryValidator();
-
-                $vObject = VObject\Reader::read($properties[200]['{urn:ietf:params:xml:ns:caldav}calendar-data']);
-                if ($validator->validate($vObject, $report->filters)) {
-
-                    // If the client didn't require the calendar-data property,
-                    // we won't give it back.
-                    if (!$requestedCalendarData) {
-                        unset($properties[200]['{urn:ietf:params:xml:ns:caldav}calendar-data']);
-                    } else {
-
-
-                        if ($report->expand) {
-                            $vObject = $vObject->expand($report->expand['start'], $report->expand['end'], $calendarTimeZone);
-                        }
-                        if ($needsJson) {
-                            $properties[200]['{' . self::NS_CALDAV . '}calendar-data'] = json_encode($vObject->jsonSerialize());
-                        } elseif ($report->expand) {
-                            $properties[200]['{' . self::NS_CALDAV . '}calendar-data'] = $vObject->serialize();
-                        }
-                    }
-
-                    $result = [$properties];
-
-                }
-                // Destroy circular references so PHP will garbage collect the
-                // object.
-                $vObject->destroy();
-
-            }
-
-        }
-
-        if ($node instanceof ICalendarObjectContainer && $depth === 0) {
-
-            if (strpos($this->server->httpRequest->getHeader('User-Agent'), 'MSFT-') === 0) {
-                // Microsoft clients incorrectly supplied depth as 0, when it actually
-                // should have set depth to 1. We're implementing a workaround here
-                // to deal with this.
-                //
-                // This targets at least the following clients:
-                //   Windows 10
-                //   Windows Phone 8, 10
-                $depth = 1;
-            } else {
-                throw new BadRequest('A calendar-query REPORT on a calendar with a Depth: 0 is undefined. Set Depth to 1');
-            }
-
-        }
-
-        // If we're dealing with a calendar, the calendar itself is responsible
-        // for the calendar-query.
-        if ($node instanceof ICalendarObjectContainer && $depth == 1) {
-
-            $nodePaths = $node->calendarQuery($report->filters);
-
-            foreach ($nodePaths as $path) {
-
-                list($properties) =
-                    $this->server->getPropertiesForPath($this->server->getRequestUri() . '/' . $path, $report->properties);
-
-                if (($needsJson || $report->expand)) {
-                    $vObject = VObject\Reader::read($properties[200]['{' . self::NS_CALDAV . '}calendar-data']);
-
-                    if ($report->expand) {
-                        $vObject = $vObject->expand($report->expand['start'], $report->expand['end'], $calendarTimeZone);
-                    }
-
-                    if ($needsJson) {
-                        $properties[200]['{' . self::NS_CALDAV . '}calendar-data'] = json_encode($vObject->jsonSerialize());
-                    } else {
-                        $properties[200]['{' . self::NS_CALDAV . '}calendar-data'] = $vObject->serialize();
-                    }
-
-                    // Destroy circular references so PHP will garbage collect the
-                    // object.
-                    $vObject->destroy();
-                }
-                $result[] = $properties;
-
-            }
-
-        }
-
-        $prefer = $this->server->getHTTPPrefer();
-
-        $this->server->httpResponse->setStatus(207);
-        $this->server->httpResponse->setHeader('Content-Type', 'application/xml; charset=utf-8');
-        $this->server->httpResponse->setHeader('Vary', 'Brief,Prefer');
-        $this->server->httpResponse->setBody($this->server->generateMultiStatus($result, $prefer['return'] === 'minimal'));
-
-    }
-
-    /**
-     * This method is responsible for parsing the request and generating the
-     * response for the CALDAV:free-busy-query REPORT.
-     *
-     * @param Xml\Request\FreeBusyQueryReport $report
-     * @return void
-     */
-    protected function freeBusyQueryReport(Xml\Request\FreeBusyQueryReport $report) {
-
-        $uri = $this->server->getRequestUri();
-
-        $acl = $this->server->getPlugin('acl');
-        if ($acl) {
-            $acl->checkPrivileges($uri, '{' . self::NS_CALDAV . '}read-free-busy');
-        }
-
-        $calendar = $this->server->tree->getNodeForPath($uri);
-        if (!$calendar instanceof ICalendar) {
-            throw new DAV\Exception\NotImplemented('The free-busy-query REPORT is only implemented on calendars');
-        }
-
-        $tzProp = '{' . self::NS_CALDAV . '}calendar-timezone';
-
-        // Figuring out the default timezone for the calendar, for floating
-        // times.
-        $calendarProps = $this->server->getProperties($uri, [$tzProp]);
-
-        if (isset($calendarProps[$tzProp])) {
-            $vtimezoneObj = VObject\Reader::read($calendarProps[$tzProp]);
-            $calendarTimeZone = $vtimezoneObj->VTIMEZONE->getTimeZone();
-            // Destroy circular references so PHP will garbage collect the object.
-            $vtimezoneObj->destroy();
-        } else {
-            $calendarTimeZone = new DateTimeZone('UTC');
-        }
-
-        // Doing a calendar-query first, to make sure we get the most
-        // performance.
-        $urls = $calendar->calendarQuery([
-            'name'         => 'VCALENDAR',
-            'comp-filters' => [
-                [
-                    'name'           => 'VEVENT',
-                    'comp-filters'   => [],
-                    'prop-filters'   => [],
-                    'is-not-defined' => false,
-                    'time-range'     => [
-                        'start' => $report->start,
-                        'end'   => $report->end,
-                    ],
-                ],
-            ],
-            'prop-filters'   => [],
-            'is-not-defined' => false,
-            'time-range'     => null,
-        ]);
-
-        $objects = array_map(function($url) use ($calendar) {
-            $obj = $calendar->getChild($url)->get();
-            return $obj;
-        }, $urls);
-
-        $generator = new VObject\FreeBusyGenerator();
-        $generator->setObjects($objects);
-        $generator->setTimeRange($report->start, $report->end);
-        $generator->setTimeZone($calendarTimeZone);
-        $result = $generator->getResult();
-        $result = $result->serialize();
-
-        $this->server->httpResponse->setStatus(200);
-        $this->server->httpResponse->setHeader('Content-Type', 'text/calendar');
-        $this->server->httpResponse->setHeader('Content-Length', strlen($result));
-        $this->server->httpResponse->setBody($result);
-
-    }
-
-    /**
-     * This method is triggered before a file gets updated with new content.
-     *
-     * This plugin uses this method to ensure that CalDAV objects receive
-     * valid calendar data.
-     *
-     * @param string $path
-     * @param DAV\IFile $node
-     * @param resource $data
-     * @param bool $modified Should be set to true, if this event handler
-     *                       changed &$data.
-     * @return void
-     */
-    public function beforeWriteContent($path, DAV\IFile $node, &$data, &$modified) {
-
-        if (!$node instanceof ICalendarObject)
-            return;
-
-        // We're onyl interested in ICalendarObject nodes that are inside of a
-        // real calendar. This is to avoid triggering validation and scheduling
-        // for non-calendars (such as an inbox).
-        list($parent) = Uri\split($path);
-        $parentNode = $this->server->tree->getNodeForPath($parent);
-
-        if (!$parentNode instanceof ICalendar)
-            return;
-
-        $this->validateICalendar(
-            $data,
-            $path,
-            $modified,
-            $this->server->httpRequest,
-            $this->server->httpResponse,
-            false
-        );
-
-    }
-
-    /**
-     * This method is triggered before a new file is created.
-     *
-     * This plugin uses this method to ensure that newly created calendar
-     * objects contain valid calendar data.
-     *
-     * @param string $path
-     * @param resource $data
-     * @param DAV\ICollection $parentNode
-     * @param bool $modified Should be set to true, if this event handler
-     *                       changed &$data.
-     * @return void
-     */
-    public function beforeCreateFile($path, &$data, DAV\ICollection $parentNode, &$modified) {
-
-        if (!$parentNode instanceof ICalendar)
-            return;
-
-        $this->validateICalendar(
-            $data,
-            $path,
-            $modified,
-            $this->server->httpRequest,
-            $this->server->httpResponse,
-            true
-        );
-
-    }
-
-    /**
-     * Checks if the submitted iCalendar data is in fact, valid.
-     *
-     * An exception is thrown if it's not.
-     *
-     * @param resource|string $data
-     * @param string $path
-     * @param bool $modified Should be set to true, if this event handler
-     *                       changed &$data.
-     * @param RequestInterface $request The http request.
-     * @param ResponseInterface $response The http response.
-     * @param bool $isNew Is the item a new one, or an update.
-     * @return void
-     */
-    protected function validateICalendar(&$data, $path, &$modified, RequestInterface $request, ResponseInterface $response, $isNew) {
-
-        // If it's a stream, we convert it to a string first.
-        if (is_resource($data)) {
-            $data = stream_get_contents($data);
-        }
-
-        $before = md5($data);
-        // Converting the data to unicode, if needed.
-        $data = DAV\StringUtil::ensureUTF8($data);
-
-        if ($before !== md5($data)) $modified = true;
-
-        try {
-
-            // If the data starts with a [, we can reasonably assume we're dealing
-            // with a jCal object.
-            if (substr($data, 0, 1) === '[') {
-                $vobj = VObject\Reader::readJson($data);
-
-                // Converting $data back to iCalendar, as that's what we
-                // technically support everywhere.
-                $data = $vobj->serialize();
-                $modified = true;
-            } else {
-                $vobj = VObject\Reader::read($data);
-            }
-
-        } catch (VObject\ParseException $e) {
-
-            throw new DAV\Exception\UnsupportedMediaType('This resource only supports valid iCalendar 2.0 data. Parse error: ' . $e->getMessage());
-
-        }
-
-        if ($vobj->name !== 'VCALENDAR') {
-            throw new DAV\Exception\UnsupportedMediaType('This collection can only support iCalendar objects.');
-        }
-
-        $sCCS = '{urn:ietf:params:xml:ns:caldav}supported-calendar-component-set';
-
-        // Get the Supported Components for the target calendar
-        list($parentPath) = Uri\split($path);
-        $calendarProperties = $this->server->getProperties($parentPath, [$sCCS]);
-
-        if (isset($calendarProperties[$sCCS])) {
-            $supportedComponents = $calendarProperties[$sCCS]->getValue();
-        } else {
-            $supportedComponents = ['VJOURNAL', 'VTODO', 'VEVENT'];
-        }
-
-        $foundType = null;
-        $foundUID = null;
-        foreach ($vobj->getComponents() as $component) {
-            switch ($component->name) {
-                case 'VTIMEZONE' :
-                    continue 2;
-                case 'VEVENT' :
-                case 'VTODO' :
-                case 'VJOURNAL' :
-                    if (is_null($foundType)) {
-                        $foundType = $component->name;
-                        if (!in_array($foundType, $supportedComponents)) {
-                            throw new Exception\InvalidComponentType('This calendar only supports ' . implode(', ', $supportedComponents) . '. We found a ' . $foundType);
-                        }
-                        if (!isset($component->UID)) {
-                            throw new DAV\Exception\BadRequest('Every ' . $component->name . ' component must have an UID');
-                        }
-                        $foundUID = (string)$component->UID;
-                    } else {
-                        if ($foundType !== $component->name) {
-                            throw new DAV\Exception\BadRequest('A calendar object must only contain 1 component. We found a ' . $component->name . ' as well as a ' . $foundType);
-                        }
-                        if ($foundUID !== (string)$component->UID) {
-                            throw new DAV\Exception\BadRequest('Every ' . $component->name . ' in this object must have identical UIDs');
-                        }
-                    }
-                    break;
-                default :
-                    throw new DAV\Exception\BadRequest('You are not allowed to create components of type: ' . $component->name . ' here');
-
-            }
-        }
-        if (!$foundType)
-            throw new DAV\Exception\BadRequest('iCalendar object must contain at least 1 of VEVENT, VTODO or VJOURNAL');
-
-        // We use an extra variable to allow event handles to tell us wether
-        // the object was modified or not.
-        //
-        // This helps us determine if we need to re-serialize the object.
-        $subModified = false;
-
-        $this->server->emit(
-            'calendarObjectChange',
-            [
-                $request,
-                $response,
-                $vobj,
-                $parentPath,
-                &$subModified,
-                $isNew
-            ]
-        );
-
-        if ($subModified) {
-            // An event handler told us that it modified the object.
-            $data = $vobj->serialize();
-
-            // Using md5 to figure out if there was an *actual* change.
-            if (!$modified && $before !== md5($data)) {
-                $modified = true;
-            }
-
-        }
-
-        // Destroy circular references so PHP will garbage collect the object.
-        $vobj->destroy();
-
-    }
-
-
-    /**
-     * This method is used to generate HTML output for the
-     * DAV\Browser\Plugin. This allows us to generate an interface users
-     * can use to create new calendars.
-     *
-     * @param DAV\INode $node
-     * @param string $output
-     * @return bool
-     */
-    public function htmlActionsPanel(DAV\INode $node, &$output) {
-
-        if (!$node instanceof CalendarHome)
-            return;
-
-        $output .= '<tr><td colspan="2"><form method="post" action="">
+	/**
+	 * This is the official CalDAV namespace
+	 */
+	const NS_CALDAV = 'urn:ietf:params:xml:ns:caldav';
+
+	/**
+	 * This is the namespace for the proprietary calendarserver extensions
+	 */
+	const NS_CALENDARSERVER = 'http://calendarserver.org/ns/';
+
+	/**
+	 * The hardcoded root for calendar objects. It is unfortunate
+	 * that we're stuck with it, but it will have to do for now
+	 */
+	const CALENDAR_ROOT = 'calendars';
+
+	/**
+	 * Reference to server object
+	 *
+	 * @var DAV\Server
+	 */
+	protected $server;
+
+	/**
+	 * The default PDO storage uses a MySQL MEDIUMBLOB for iCalendar data,
+	 * which can hold up to 2^24 = 16777216 bytes. This is plenty. We're
+	 * capping it to 10M here.
+	 */
+	protected $maxResourceSize = 10000000;
+
+	/**
+	 * Use this method to tell the server this plugin defines additional
+	 * HTTP methods.
+	 *
+	 * This method is passed a uri. It should only return HTTP methods that are
+	 * available for the specified uri.
+	 *
+	 * @param string $uri
+	 * @return array
+	 */
+	public function getHTTPMethods($uri) {
+
+		// The MKCALENDAR is only available on unmapped uri's, whose
+		// parents extend IExtendedCollection
+		list($parent, $name) = Uri\split($uri);
+
+		$node = $this->server->tree->getNodeForPath($parent);
+
+		if ($node instanceof DAV\IExtendedCollection) {
+			try {
+				$node->getChild($name);
+			} catch (DAV\Exception\NotFound $e) {
+				return ['MKCALENDAR'];
+			}
+		}
+		return [];
+
+	}
+
+	/**
+	 * Returns the path to a principal's calendar home.
+	 *
+	 * The return url must not end with a slash.
+	 * This function should return null in case a principal did not have
+	 * a calendar home.
+	 *
+	 * @param string $principalUrl
+	 * @return string
+	 */
+	public function getCalendarHomeForPrincipal($principalUrl) {
+
+		// The default behavior for most sabre/dav servers is that there is a
+		// principals root node, which contains users directly under it.
+		//
+		// This function assumes that there are two components in a principal
+		// path. If there's more, we don't return a calendar home. This
+		// excludes things like the calendar-proxy-read principal (which it
+		// should).
+		$parts = explode('/', trim($principalUrl, '/'));
+		if (count($parts) !== 2) return;
+		if ($parts[0] !== 'principals') return;
+
+		return self::CALENDAR_ROOT . '/' . $parts[1];
+
+	}
+
+	/**
+	 * Returns a list of features for the DAV: HTTP header.
+	 *
+	 * @return array
+	 */
+	public function getFeatures() {
+
+		return ['calendar-access', 'calendar-proxy'];
+
+	}
+
+	/**
+	 * Returns a plugin name.
+	 *
+	 * Using this name other plugins will be able to access other plugins
+	 * using DAV\Server::getPlugin
+	 *
+	 * @return string
+	 */
+	public function getPluginName() {
+
+		return 'caldav';
+
+	}
+
+	/**
+	 * Returns a list of reports this plugin supports.
+	 *
+	 * This will be used in the {DAV:}supported-report-set property.
+	 * Note that you still need to subscribe to the 'report' event to actually
+	 * implement them
+	 *
+	 * @param string $uri
+	 * @return array
+	 */
+	public function getSupportedReportSet($uri) {
+
+		$node = $this->server->tree->getNodeForPath($uri);
+
+		$reports = [];
+		if ($node instanceof ICalendarObjectContainer || $node instanceof ICalendarObject) {
+			$reports[] = '{' . self::NS_CALDAV . '}calendar-multiget';
+			$reports[] = '{' . self::NS_CALDAV . '}calendar-query';
+		}
+		if ($node instanceof ICalendar) {
+			$reports[] = '{' . self::NS_CALDAV . '}free-busy-query';
+		}
+		// iCal has a bug where it assumes that sync support is enabled, only
+		// if we say we support it on the calendar-home, even though this is
+		// not actually the case.
+		if ($node instanceof CalendarHome && $this->server->getPlugin('sync')) {
+			$reports[] = '{DAV:}sync-collection';
+		}
+		return $reports;
+
+	}
+
+	/**
+	 * Initializes the plugin
+	 *
+	 * @param DAV\Server $server
+	 * @return void
+	 */
+	public function initialize(DAV\Server $server) {
+
+		$this->server = $server;
+
+		$server->on('method:MKCALENDAR',   [$this, 'httpMkCalendar']);
+		$server->on('report',              [$this, 'report']);
+		$server->on('propFind',            [$this, 'propFind']);
+		$server->on('onHTMLActionsPanel',  [$this, 'htmlActionsPanel']);
+		$server->on('beforeCreateFile',    [$this, 'beforeCreateFile']);
+		$server->on('beforeWriteContent',  [$this, 'beforeWriteContent']);
+		$server->on('afterMethod:GET',     [$this, 'httpAfterGET']);
+
+		$server->xml->namespaceMap[self::NS_CALDAV] = 'cal';
+		$server->xml->namespaceMap[self::NS_CALENDARSERVER] = 'cs';
+
+		$server->xml->elementMap['{' . self::NS_CALDAV . '}supported-calendar-component-set'] = 'Sabre\\CalDAV\\Xml\\Property\\SupportedCalendarComponentSet';
+		$server->xml->elementMap['{' . self::NS_CALDAV . '}calendar-query'] = 'Sabre\\CalDAV\\Xml\\Request\\CalendarQueryReport';
+		$server->xml->elementMap['{' . self::NS_CALDAV . '}calendar-multiget'] = 'Sabre\\CalDAV\\Xml\\Request\\CalendarMultiGetReport';
+		$server->xml->elementMap['{' . self::NS_CALDAV . '}free-busy-query'] = 'Sabre\\CalDAV\\Xml\\Request\\FreeBusyQueryReport';
+		$server->xml->elementMap['{' . self::NS_CALDAV . '}mkcalendar'] = 'Sabre\\CalDAV\\Xml\\Request\\MkCalendar';
+		$server->xml->elementMap['{' . self::NS_CALDAV . '}schedule-calendar-transp'] = 'Sabre\\CalDAV\\Xml\\Property\\ScheduleCalendarTransp';
+		$server->xml->elementMap['{' . self::NS_CALDAV . '}supported-calendar-component-set'] = 'Sabre\\CalDAV\\Xml\\Property\\SupportedCalendarComponentSet';
+
+		$server->resourceTypeMapping['\\Sabre\\CalDAV\\ICalendar'] = '{urn:ietf:params:xml:ns:caldav}calendar';
+
+		$server->resourceTypeMapping['\\Sabre\\CalDAV\\Principal\\IProxyRead'] = '{http://calendarserver.org/ns/}calendar-proxy-read';
+		$server->resourceTypeMapping['\\Sabre\\CalDAV\\Principal\\IProxyWrite'] = '{http://calendarserver.org/ns/}calendar-proxy-write';
+
+		array_push($server->protectedProperties,
+
+			'{' . self::NS_CALDAV . '}supported-calendar-component-set',
+			'{' . self::NS_CALDAV . '}supported-calendar-data',
+			'{' . self::NS_CALDAV . '}max-resource-size',
+			'{' . self::NS_CALDAV . '}min-date-time',
+			'{' . self::NS_CALDAV . '}max-date-time',
+			'{' . self::NS_CALDAV . '}max-instances',
+			'{' . self::NS_CALDAV . '}max-attendees-per-instance',
+			'{' . self::NS_CALDAV . '}calendar-home-set',
+			'{' . self::NS_CALDAV . '}supported-collation-set',
+			'{' . self::NS_CALDAV . '}calendar-data',
+
+			// CalendarServer extensions
+			'{' . self::NS_CALENDARSERVER . '}getctag',
+			'{' . self::NS_CALENDARSERVER . '}calendar-proxy-read-for',
+			'{' . self::NS_CALENDARSERVER . '}calendar-proxy-write-for'
+
+		);
+
+		if ($aclPlugin = $server->getPlugin('acl')) {
+			$aclPlugin->principalSearchPropertySet['{' . self::NS_CALDAV . '}calendar-user-address-set'] = 'Calendar address';
+		}
+	}
+
+	/**
+	 * This functions handles REPORT requests specific to CalDAV
+	 *
+	 * @param string $reportName
+	 * @param mixed $report
+	 * @return bool
+	 */
+	public function report($reportName, $report) {
+
+		switch ($reportName) {
+			case '{' . self::NS_CALDAV . '}calendar-multiget' :
+				$this->server->transactionType = 'report-calendar-multiget';
+				$this->calendarMultiGetReport($report);
+				return false;
+			case '{' . self::NS_CALDAV . '}calendar-query' :
+				$this->server->transactionType = 'report-calendar-query';
+				$this->calendarQueryReport($report);
+				return false;
+			case '{' . self::NS_CALDAV . '}free-busy-query' :
+				$this->server->transactionType = 'report-free-busy-query';
+				$this->freeBusyQueryReport($report);
+				return false;
+
+		}
+
+
+	}
+
+	/**
+	 * This function handles the MKCALENDAR HTTP method, which creates
+	 * a new calendar.
+	 *
+	 * @param RequestInterface $request
+	 * @param ResponseInterface $response
+	 * @return bool
+	 */
+	public function httpMkCalendar(RequestInterface $request, ResponseInterface $response) {
+
+		$body = $request->getBodyAsString();
+		$path = $request->getPath();
+
+		$properties = [];
+
+		if ($body) {
+
+			try {
+				$mkcalendar = $this->server->xml->expect(
+					'{urn:ietf:params:xml:ns:caldav}mkcalendar',
+					$body
+				);
+			} catch (\Sabre\Xml\ParseException $e) {
+				throw new BadRequest($e->getMessage(), null, $e);
+			}
+			$properties = $mkcalendar->getProperties();
+
+		}
+
+		// iCal abuses MKCALENDAR since iCal 10.9.2 to create server-stored
+		// subscriptions. Before that it used MKCOL which was the correct way
+		// to do this.
+		//
+		// If the body had a {DAV:}resourcetype, it means we stumbled upon this
+		// request, and we simply use it instead of the pre-defined list.
+		if (isset($properties['{DAV:}resourcetype'])) {
+			$resourceType = $properties['{DAV:}resourcetype']->getValue();
+		} else {
+			$resourceType = ['{DAV:}collection','{urn:ietf:params:xml:ns:caldav}calendar'];
+		}
+
+		$this->server->createCollection($path, new MkCol($resourceType, $properties));
+
+		$this->server->httpResponse->setStatus(201);
+		$this->server->httpResponse->setHeader('Content-Length', 0);
+
+		// This breaks the method chain.
+		return false;
+	}
+
+	/**
+	 * PropFind
+	 *
+	 * This method handler is invoked before any after properties for a
+	 * resource are fetched. This allows us to add in any CalDAV specific
+	 * properties.
+	 *
+	 * @param DAV\PropFind $propFind
+	 * @param DAV\INode $node
+	 * @return void
+	 */
+	public function propFind(DAV\PropFind $propFind, DAV\INode $node) {
+
+		$ns = '{' . self::NS_CALDAV . '}';
+
+		if ($node instanceof ICalendarObjectContainer) {
+
+			$propFind->handle($ns . 'max-resource-size', $this->maxResourceSize);
+			$propFind->handle($ns . 'supported-calendar-data', function() {
+				return new Xml\Property\SupportedCalendarData();
+			});
+			$propFind->handle($ns . 'supported-collation-set', function() {
+				return new Xml\Property\SupportedCollationSet();
+			});
+
+		}
+
+		if ($node instanceof DAVACL\IPrincipal) {
+
+			$principalUrl = $node->getPrincipalUrl();
+
+			$propFind->handle('{' . self::NS_CALDAV . '}calendar-home-set', function() use ($principalUrl) {
+
+				$calendarHomePath = $this->getCalendarHomeForPrincipal($principalUrl);
+				if (is_null($calendarHomePath)) return null;
+				return new Href($calendarHomePath . '/');
+
+			});
+			// The calendar-user-address-set property is basically mapped to
+			// the {DAV:}alternate-URI-set property.
+			$propFind->handle('{' . self::NS_CALDAV . '}calendar-user-address-set', function() use ($node) {
+				$addresses = $node->getAlternateUriSet();
+				$addresses[] = $this->server->getBaseUri() . $node->getPrincipalUrl() . '/';
+				return new Href($addresses, false);
+			});
+			// For some reason somebody thought it was a good idea to add
+			// another one of these properties. We're supporting it too.
+			$propFind->handle('{' . self::NS_CALENDARSERVER . '}email-address-set', function() use ($node) {
+				$addresses = $node->getAlternateUriSet();
+				$emails = [];
+				foreach ($addresses as $address) {
+					if (substr($address, 0, 7) === 'mailto:') {
+						$emails[] = substr($address, 7);
+					}
+				}
+				return new Xml\Property\EmailAddressSet($emails);
+			});
+
+			// These two properties are shortcuts for ical to easily find
+			// other principals this principal has access to.
+			$propRead = '{' . self::NS_CALENDARSERVER . '}calendar-proxy-read-for';
+			$propWrite = '{' . self::NS_CALENDARSERVER . '}calendar-proxy-write-for';
+
+			if ($propFind->getStatus($propRead) === 404 || $propFind->getStatus($propWrite) === 404) {
+
+				$aclPlugin = $this->server->getPlugin('acl');
+				$membership = $aclPlugin->getPrincipalMembership($propFind->getPath());
+				$readList = [];
+				$writeList = [];
+
+				foreach ($membership as $group) {
+
+					$groupNode = $this->server->tree->getNodeForPath($group);
+
+					$listItem = Uri\split($group)[0] . '/';
+
+					// If the node is either ap proxy-read or proxy-write
+					// group, we grab the parent principal and add it to the
+					// list.
+					if ($groupNode instanceof Principal\IProxyRead) {
+						$readList[] = $listItem;
+					}
+					if ($groupNode instanceof Principal\IProxyWrite) {
+						$writeList[] = $listItem;
+					}
+
+				}
+
+				$propFind->set($propRead, new Href($readList));
+				$propFind->set($propWrite, new Href($writeList));
+
+			}
+
+		} // instanceof IPrincipal
+
+		if ($node instanceof ICalendarObject) {
+
+			// The calendar-data property is not supposed to be a 'real'
+			// property, but in large chunks of the spec it does act as such.
+			// Therefore we simply expose it as a property.
+			$propFind->handle('{' . self::NS_CALDAV . '}calendar-data', function() use ($node) {
+				$val = $node->get();
+				if (is_resource($val))
+					$val = stream_get_contents($val);
+
+				// Taking out \r to not screw up the xml output
+				return str_replace("\r", "", $val);
+
+			});
+
+		}
+
+	}
+
+	/**
+	 * This function handles the calendar-multiget REPORT.
+	 *
+	 * This report is used by the client to fetch the content of a series
+	 * of urls. Effectively avoiding a lot of redundant requests.
+	 *
+	 * @param CalendarMultiGetReport $report
+	 * @return void
+	 */
+	public function calendarMultiGetReport($report) {
+
+		$needsJson = $report->contentType === 'application/calendar+json';
+
+		$timeZones = [];
+		$propertyList = [];
+
+		$paths = array_map(
+			[$this->server, 'calculateUri'],
+			$report->hrefs
+		);
+
+		foreach ($this->server->getPropertiesForMultiplePaths($paths, $report->properties) as $uri => $objProps) {
+
+			if (($needsJson || $report->expand) && isset($objProps[200]['{' . self::NS_CALDAV . '}calendar-data'])) {
+				$vObject = VObject\Reader::read($objProps[200]['{' . self::NS_CALDAV . '}calendar-data']);
+
+				if ($report->expand) {
+					// We're expanding, and for that we need to figure out the
+					// calendar's timezone.
+					list($calendarPath) = Uri\split($uri);
+					if (!isset($timeZones[$calendarPath])) {
+						// Checking the calendar-timezone property.
+						$tzProp = '{' . self::NS_CALDAV . '}calendar-timezone';
+						$tzResult = $this->server->getProperties($calendarPath, [$tzProp]);
+						if (isset($tzResult[$tzProp])) {
+							// This property contains a VCALENDAR with a single
+							// VTIMEZONE.
+							$vtimezoneObj = VObject\Reader::read($tzResult[$tzProp]);
+							$timeZone = $vtimezoneObj->VTIMEZONE->getTimeZone();
+						} else {
+							// Defaulting to UTC.
+							$timeZone = new DateTimeZone('UTC');
+						}
+						$timeZones[$calendarPath] = $timeZone;
+					}
+
+					$vObject = $vObject->expand($report->expand['start'], $report->expand['end'], $timeZones[$calendarPath]);
+				}
+				if ($needsJson) {
+					$objProps[200]['{' . self::NS_CALDAV . '}calendar-data'] = json_encode($vObject->jsonSerialize());
+				} else {
+					$objProps[200]['{' . self::NS_CALDAV . '}calendar-data'] = $vObject->serialize();
+				}
+				// Destroy circular references so PHP will garbage collect the
+				// object.
+				$vObject->destroy();
+			}
+
+			$propertyList[] = $objProps;
+
+		}
+
+		$prefer = $this->server->getHTTPPrefer();
+
+		$this->server->httpResponse->setStatus(207);
+		$this->server->httpResponse->setHeader('Content-Type', 'application/xml; charset=utf-8');
+		$this->server->httpResponse->setHeader('Vary', 'Brief,Prefer');
+		$this->server->httpResponse->setBody($this->server->generateMultiStatus($propertyList, $prefer['return'] === 'minimal'));
+
+	}
+
+	/**
+	 * This function handles the calendar-query REPORT
+	 *
+	 * This report is used by clients to request calendar objects based on
+	 * complex conditions.
+	 *
+	 * @param Xml\Request\CalendarQueryReport $report
+	 * @return void
+	 */
+	public function calendarQueryReport($report) {
+
+		$path = $this->server->getRequestUri();
+
+		$needsJson = $report->contentType === 'application/calendar+json';
+
+		$node = $this->server->tree->getNodeForPath($this->server->getRequestUri());
+		$depth = $this->server->getHTTPDepth(0);
+
+		// The default result is an empty array
+		$result = [];
+
+		$calendarTimeZone = null;
+		if ($report->expand) {
+			// We're expanding, and for that we need to figure out the
+			// calendar's timezone.
+			$tzProp = '{' . self::NS_CALDAV . '}calendar-timezone';
+			$tzResult = $this->server->getProperties($path, [$tzProp]);
+			if (isset($tzResult[$tzProp])) {
+				// This property contains a VCALENDAR with a single
+				// VTIMEZONE.
+				$vtimezoneObj = VObject\Reader::read($tzResult[$tzProp]);
+				$calendarTimeZone = $vtimezoneObj->VTIMEZONE->getTimeZone();
+
+				// Destroy circular references so PHP will garbage collect the
+				// object.
+				$vtimezoneObj->destroy();
+			} else {
+				// Defaulting to UTC.
+				$calendarTimeZone = new DateTimeZone('UTC');
+			}
+		}
+
+		// The calendarobject was requested directly. In this case we handle
+		// this locally.
+		if ($depth == 0 && $node instanceof ICalendarObject) {
+
+			$requestedCalendarData = true;
+			$requestedProperties = $report->properties;
+
+			if (!in_array('{urn:ietf:params:xml:ns:caldav}calendar-data', $requestedProperties)) {
+
+				// We always retrieve calendar-data, as we need it for filtering.
+				$requestedProperties[] = '{urn:ietf:params:xml:ns:caldav}calendar-data';
+
+				// If calendar-data wasn't explicitly requested, we need to remove
+				// it after processing.
+				$requestedCalendarData = false;
+			}
+
+			$properties = $this->server->getPropertiesForPath(
+				$path,
+				$requestedProperties,
+				0
+			);
+
+			// This array should have only 1 element, the first calendar
+			// object.
+			$properties = current($properties);
+
+			// If there wasn't any calendar-data returned somehow, we ignore
+			// this.
+			if (isset($properties[200]['{urn:ietf:params:xml:ns:caldav}calendar-data'])) {
+
+				$validator = new CalendarQueryValidator();
+
+				$vObject = VObject\Reader::read($properties[200]['{urn:ietf:params:xml:ns:caldav}calendar-data']);
+				if ($validator->validate($vObject, $report->filters)) {
+
+					// If the client didn't require the calendar-data property,
+					// we won't give it back.
+					if (!$requestedCalendarData) {
+						unset($properties[200]['{urn:ietf:params:xml:ns:caldav}calendar-data']);
+					} else {
+
+
+						if ($report->expand) {
+							$vObject = $vObject->expand($report->expand['start'], $report->expand['end'], $calendarTimeZone);
+						}
+						if ($needsJson) {
+							$properties[200]['{' . self::NS_CALDAV . '}calendar-data'] = json_encode($vObject->jsonSerialize());
+						} elseif ($report->expand) {
+							$properties[200]['{' . self::NS_CALDAV . '}calendar-data'] = $vObject->serialize();
+						}
+					}
+
+					$result = [$properties];
+
+				}
+				// Destroy circular references so PHP will garbage collect the
+				// object.
+				$vObject->destroy();
+
+			}
+
+		}
+
+		if ($node instanceof ICalendarObjectContainer && $depth === 0) {
+
+			if (strpos($this->server->httpRequest->getHeader('User-Agent'), 'MSFT-') === 0) {
+				// Microsoft clients incorrectly supplied depth as 0, when it actually
+				// should have set depth to 1. We're implementing a workaround here
+				// to deal with this.
+				//
+				// This targets at least the following clients:
+				//   Windows 10
+				//   Windows Phone 8, 10
+				$depth = 1;
+			} else {
+				throw new BadRequest('A calendar-query REPORT on a calendar with a Depth: 0 is undefined. Set Depth to 1');
+			}
+
+		}
+
+		// If we're dealing with a calendar, the calendar itself is responsible
+		// for the calendar-query.
+		if ($node instanceof ICalendarObjectContainer && $depth == 1) {
+
+			$nodePaths = $node->calendarQuery($report->filters);
+
+			foreach ($nodePaths as $path) {
+
+				list($properties) =
+					$this->server->getPropertiesForPath($this->server->getRequestUri() . '/' . $path, $report->properties);
+
+				if (($needsJson || $report->expand)) {
+					$vObject = VObject\Reader::read($properties[200]['{' . self::NS_CALDAV . '}calendar-data']);
+
+					if ($report->expand) {
+						$vObject = $vObject->expand($report->expand['start'], $report->expand['end'], $calendarTimeZone);
+					}
+
+					if ($needsJson) {
+						$properties[200]['{' . self::NS_CALDAV . '}calendar-data'] = json_encode($vObject->jsonSerialize());
+					} else {
+						$properties[200]['{' . self::NS_CALDAV . '}calendar-data'] = $vObject->serialize();
+					}
+
+					// Destroy circular references so PHP will garbage collect the
+					// object.
+					$vObject->destroy();
+				}
+				$result[] = $properties;
+
+			}
+
+		}
+
+		$prefer = $this->server->getHTTPPrefer();
+
+		$this->server->httpResponse->setStatus(207);
+		$this->server->httpResponse->setHeader('Content-Type', 'application/xml; charset=utf-8');
+		$this->server->httpResponse->setHeader('Vary', 'Brief,Prefer');
+		$this->server->httpResponse->setBody($this->server->generateMultiStatus($result, $prefer['return'] === 'minimal'));
+
+	}
+
+	/**
+	 * This method is responsible for parsing the request and generating the
+	 * response for the CALDAV:free-busy-query REPORT.
+	 *
+	 * @param Xml\Request\FreeBusyQueryReport $report
+	 * @return void
+	 */
+	protected function freeBusyQueryReport(Xml\Request\FreeBusyQueryReport $report) {
+
+		$uri = $this->server->getRequestUri();
+
+		$acl = $this->server->getPlugin('acl');
+		if ($acl) {
+			$acl->checkPrivileges($uri, '{' . self::NS_CALDAV . '}read-free-busy');
+		}
+
+		$calendar = $this->server->tree->getNodeForPath($uri);
+		if (!$calendar instanceof ICalendar) {
+			throw new DAV\Exception\NotImplemented('The free-busy-query REPORT is only implemented on calendars');
+		}
+
+		$tzProp = '{' . self::NS_CALDAV . '}calendar-timezone';
+
+		// Figuring out the default timezone for the calendar, for floating
+		// times.
+		$calendarProps = $this->server->getProperties($uri, [$tzProp]);
+
+		if (isset($calendarProps[$tzProp])) {
+			$vtimezoneObj = VObject\Reader::read($calendarProps[$tzProp]);
+			$calendarTimeZone = $vtimezoneObj->VTIMEZONE->getTimeZone();
+			// Destroy circular references so PHP will garbage collect the object.
+			$vtimezoneObj->destroy();
+		} else {
+			$calendarTimeZone = new DateTimeZone('UTC');
+		}
+
+		// Doing a calendar-query first, to make sure we get the most
+		// performance.
+		$urls = $calendar->calendarQuery([
+			'name'         => 'VCALENDAR',
+			'comp-filters' => [
+				[
+					'name'           => 'VEVENT',
+					'comp-filters'   => [],
+					'prop-filters'   => [],
+					'is-not-defined' => false,
+					'time-range'     => [
+						'start' => $report->start,
+						'end'   => $report->end,
+					],
+				],
+			],
+			'prop-filters'   => [],
+			'is-not-defined' => false,
+			'time-range'     => null,
+		]);
+
+		$objects = array_map(function($url) use ($calendar) {
+			$obj = $calendar->getChild($url)->get();
+			return $obj;
+		}, $urls);
+
+		$generator = new VObject\FreeBusyGenerator();
+		$generator->setObjects($objects);
+		$generator->setTimeRange($report->start, $report->end);
+		$generator->setTimeZone($calendarTimeZone);
+		$result = $generator->getResult();
+		$result = $result->serialize();
+
+		$this->server->httpResponse->setStatus(200);
+		$this->server->httpResponse->setHeader('Content-Type', 'text/calendar');
+		$this->server->httpResponse->setHeader('Content-Length', strlen($result));
+		$this->server->httpResponse->setBody($result);
+
+	}
+
+	/**
+	 * This method is triggered before a file gets updated with new content.
+	 *
+	 * This plugin uses this method to ensure that CalDAV objects receive
+	 * valid calendar data.
+	 *
+	 * @param string $path
+	 * @param DAV\IFile $node
+	 * @param resource $data
+	 * @param bool $modified Should be set to true, if this event handler
+	 *                       changed &$data.
+	 * @return void
+	 */
+	public function beforeWriteContent($path, DAV\IFile $node, &$data, &$modified) {
+
+		if (!$node instanceof ICalendarObject)
+			return;
+
+		// We're onyl interested in ICalendarObject nodes that are inside of a
+		// real calendar. This is to avoid triggering validation and scheduling
+		// for non-calendars (such as an inbox).
+		list($parent) = Uri\split($path);
+		$parentNode = $this->server->tree->getNodeForPath($parent);
+
+		if (!$parentNode instanceof ICalendar)
+			return;
+
+		$this->validateICalendar(
+			$data,
+			$path,
+			$modified,
+			$this->server->httpRequest,
+			$this->server->httpResponse,
+			false
+		);
+
+	}
+
+	/**
+	 * This method is triggered before a new file is created.
+	 *
+	 * This plugin uses this method to ensure that newly created calendar
+	 * objects contain valid calendar data.
+	 *
+	 * @param string $path
+	 * @param resource $data
+	 * @param DAV\ICollection $parentNode
+	 * @param bool $modified Should be set to true, if this event handler
+	 *                       changed &$data.
+	 * @return void
+	 */
+	public function beforeCreateFile($path, &$data, DAV\ICollection $parentNode, &$modified) {
+
+		if (!$parentNode instanceof ICalendar)
+			return;
+
+		$this->validateICalendar(
+			$data,
+			$path,
+			$modified,
+			$this->server->httpRequest,
+			$this->server->httpResponse,
+			true
+		);
+
+	}
+
+	/**
+	 * Checks if the submitted iCalendar data is in fact, valid.
+	 *
+	 * An exception is thrown if it's not.
+	 *
+	 * @param resource|string $data
+	 * @param string $path
+	 * @param bool $modified Should be set to true, if this event handler
+	 *                       changed &$data.
+	 * @param RequestInterface $request The http request.
+	 * @param ResponseInterface $response The http response.
+	 * @param bool $isNew Is the item a new one, or an update.
+	 * @return void
+	 */
+	protected function validateICalendar(&$data, $path, &$modified, RequestInterface $request, ResponseInterface $response, $isNew) {
+
+		// If it's a stream, we convert it to a string first.
+		if (is_resource($data)) {
+			$data = stream_get_contents($data);
+		}
+
+		$before = md5($data);
+		// Converting the data to unicode, if needed.
+		$data = DAV\StringUtil::ensureUTF8($data);
+
+		if ($before !== md5($data)) $modified = true;
+
+		try {
+
+			// If the data starts with a [, we can reasonably assume we're dealing
+			// with a jCal object.
+			if (substr($data, 0, 1) === '[') {
+				$vobj = VObject\Reader::readJson($data);
+
+				// Converting $data back to iCalendar, as that's what we
+				// technically support everywhere.
+				$data = $vobj->serialize();
+				$modified = true;
+			} else {
+				$vobj = VObject\Reader::read($data);
+			}
+
+		} catch (VObject\ParseException $e) {
+
+			throw new DAV\Exception\UnsupportedMediaType('This resource only supports valid iCalendar 2.0 data. Parse error: ' . $e->getMessage());
+
+		}
+
+		if ($vobj->name !== 'VCALENDAR') {
+			throw new DAV\Exception\UnsupportedMediaType('This collection can only support iCalendar objects.');
+		}
+
+		$sCCS = '{urn:ietf:params:xml:ns:caldav}supported-calendar-component-set';
+
+		// Get the Supported Components for the target calendar
+		list($parentPath) = Uri\split($path);
+		$calendarProperties = $this->server->getProperties($parentPath, [$sCCS]);
+
+		if (isset($calendarProperties[$sCCS])) {
+			$supportedComponents = $calendarProperties[$sCCS]->getValue();
+		} else {
+			$supportedComponents = ['VJOURNAL', 'VTODO', 'VEVENT'];
+		}
+
+		$foundType = null;
+		$foundUID = null;
+		foreach ($vobj->getComponents() as $component) {
+			switch ($component->name) {
+				case 'VTIMEZONE' :
+					continue 2;
+				case 'VEVENT' :
+				case 'VTODO' :
+				case 'VJOURNAL' :
+					if (is_null($foundType)) {
+						$foundType = $component->name;
+						if (!in_array($foundType, $supportedComponents)) {
+							throw new Exception\InvalidComponentType('This calendar only supports ' . implode(', ', $supportedComponents) . '. We found a ' . $foundType);
+						}
+						if (!isset($component->UID)) {
+							throw new DAV\Exception\BadRequest('Every ' . $component->name . ' component must have an UID');
+						}
+						$foundUID = (string)$component->UID;
+					} else {
+						if ($foundType !== $component->name) {
+							throw new DAV\Exception\BadRequest('A calendar object must only contain 1 component. We found a ' . $component->name . ' as well as a ' . $foundType);
+						}
+						if ($foundUID !== (string)$component->UID) {
+							throw new DAV\Exception\BadRequest('Every ' . $component->name . ' in this object must have identical UIDs');
+						}
+					}
+					break;
+				default :
+					throw new DAV\Exception\BadRequest('You are not allowed to create components of type: ' . $component->name . ' here');
+
+			}
+		}
+		if (!$foundType)
+			throw new DAV\Exception\BadRequest('iCalendar object must contain at least 1 of VEVENT, VTODO or VJOURNAL');
+
+		// We use an extra variable to allow event handles to tell us wether
+		// the object was modified or not.
+		//
+		// This helps us determine if we need to re-serialize the object.
+		$subModified = false;
+
+		$this->server->emit(
+			'calendarObjectChange',
+			[
+				$request,
+				$response,
+				$vobj,
+				$parentPath,
+				&$subModified,
+				$isNew
+			]
+		);
+
+		if ($subModified) {
+			// An event handler told us that it modified the object.
+			$data = $vobj->serialize();
+
+			// Using md5 to figure out if there was an *actual* change.
+			if (!$modified && $before !== md5($data)) {
+				$modified = true;
+			}
+
+		}
+
+		// Destroy circular references so PHP will garbage collect the object.
+		$vobj->destroy();
+
+	}
+
+
+	/**
+	 * This method is used to generate HTML output for the
+	 * DAV\Browser\Plugin. This allows us to generate an interface users
+	 * can use to create new calendars.
+	 *
+	 * @param DAV\INode $node
+	 * @param string $output
+	 * @return bool
+	 */
+	public function htmlActionsPanel(DAV\INode $node, &$output) {
+
+		if (!$node instanceof CalendarHome)
+			return;
+
+		$output .= '<tr><td colspan="2"><form method="post" action="">
             <h3>Create new calendar</h3>
             <input type="hidden" name="sabreAction" value="mkcol" />
             <input type="hidden" name="resourceType" value="{DAV:}collection,{' . self::NS_CALDAV . '}calendar" />
@@ -958,68 +958,68 @@ 
             </form>
             </td></tr>';
 
-        return false;
-
-    }
-
-    /**
-     * This event is triggered after GET requests.
-     *
-     * This is used to transform data into jCal, if this was requested.
-     *
-     * @param RequestInterface $request
-     * @param ResponseInterface $response
-     * @return void
-     */
-    public function httpAfterGet(RequestInterface $request, ResponseInterface $response) {
-
-        if (strpos($response->getHeader('Content-Type'), 'text/calendar') === false) {
-            return;
-        }
-
-        $result = HTTP\Util::negotiate(
-            $request->getHeader('Accept'),
-            ['text/calendar', 'application/calendar+json']
-        );
-
-        if ($result !== 'application/calendar+json') {
-            // Do nothing
-            return;
-        }
-
-        // Transforming.
-        $vobj = VObject\Reader::read($response->getBody());
-
-        $jsonBody = json_encode($vobj->jsonSerialize());
-        $response->setBody($jsonBody);
-
-        // Destroy circular references so PHP will garbage collect the object.
-        $vobj->destroy();
-
-        $response->setHeader('Content-Type', 'application/calendar+json');
-        $response->setHeader('Content-Length', strlen($jsonBody));
-
-    }
-
-    /**
-     * Returns a bunch of meta-data about the plugin.
-     *
-     * Providing this information is optional, and is mainly displayed by the
-     * Browser plugin.
-     *
-     * The description key in the returned array may contain html and will not
-     * be sanitized.
-     *
-     * @return array
-     */
-    public function getPluginInfo() {
-
-        return [
-            'name'        => $this->getPluginName(),
-            'description' => 'Adds support for CalDAV (rfc4791)',
-            'link'        => 'http://sabre.io/dav/caldav/',
-        ];
-
-    }
+		return false;
+
+	}
+
+	/**
+	 * This event is triggered after GET requests.
+	 *
+	 * This is used to transform data into jCal, if this was requested.
+	 *
+	 * @param RequestInterface $request
+	 * @param ResponseInterface $response
+	 * @return void
+	 */
+	public function httpAfterGet(RequestInterface $request, ResponseInterface $response) {
+
+		if (strpos($response->getHeader('Content-Type'), 'text/calendar') === false) {
+			return;
+		}
+
+		$result = HTTP\Util::negotiate(
+			$request->getHeader('Accept'),
+			['text/calendar', 'application/calendar+json']
+		);
+
+		if ($result !== 'application/calendar+json') {
+			// Do nothing
+			return;
+		}
+
+		// Transforming.
+		$vobj = VObject\Reader::read($response->getBody());
+
+		$jsonBody = json_encode($vobj->jsonSerialize());
+		$response->setBody($jsonBody);
+
+		// Destroy circular references so PHP will garbage collect the object.
+		$vobj->destroy();
+
+		$response->setHeader('Content-Type', 'application/calendar+json');
+		$response->setHeader('Content-Length', strlen($jsonBody));
+
+	}
+
+	/**
+	 * Returns a bunch of meta-data about the plugin.
+	 *
+	 * Providing this information is optional, and is mainly displayed by the
+	 * Browser plugin.
+	 *
+	 * The description key in the returned array may contain html and will not
+	 * be sanitized.
+	 *
+	 * @return array
+	 */
+	public function getPluginInfo() {
+
+		return [
+			'name'        => $this->getPluginName(),
+			'description' => 'Adds support for CalDAV (rfc4791)',
+			'link'        => 'http://sabre.io/dav/caldav/',
+		];
+
+	}
 
 }

Doc Comments +4 added lines, -4 removed lines

@@ -115,7 +115,7 @@ 
     /**
      * Returns a list of features for the DAV: HTTP header.
      *
-     * @return array
+     * @return string[]
      */
     public function getFeatures() {
 
@@ -233,7 +233,7 @@ 
      *
      * @param string $reportName
      * @param mixed $report
-     * @return bool
+     * @return false|null
      */
     public function report($reportName, $report) {
 
@@ -805,7 +805,7 @@ 
      *
      * An exception is thrown if it's not.
      *
-     * @param resource|string $data
+     * @param resource $data
      * @param string $path
      * @param bool $modified Should be set to true, if this event handler
      *                       changed &$data.
@@ -941,7 +941,7 @@ 
      *
      * @param DAV\INode $node
      * @param string $output
-     * @return bool
+     * @return null|false
      */
     public function htmlActionsPanel(DAV\INode $node, &$output) {
 

Braces +30 added lines, -16 removed lines

@@ -105,8 +105,12 @@ 
         // excludes things like the calendar-proxy-read principal (which it
         // should).
         $parts = explode('/', trim($principalUrl, '/'));
-        if (count($parts) !== 2) return;
-        if ($parts[0] !== 'principals') return;
+        if (count($parts) !== 2) {
+        	return;
+        }
+        if ($parts[0] !== 'principals') {
+        	return;
+        }
 
         return self::CALENDAR_ROOT . '/' . $parts[1];
 
@@ -340,7 +344,9 @@ 
             $propFind->handle('{' . self::NS_CALDAV . '}calendar-home-set', function() use ($principalUrl) {
 
                 $calendarHomePath = $this->getCalendarHomeForPrincipal($principalUrl);
-                if (is_null($calendarHomePath)) return null;
+                if (is_null($calendarHomePath)) {
+                	return null;
+                }
                 return new Href($calendarHomePath . '/');
 
             });
@@ -408,8 +414,9 @@ 
             // Therefore we simply expose it as a property.
             $propFind->handle('{' . self::NS_CALDAV . '}calendar-data', function() use ($node) {
                 $val = $node->get();
-                if (is_resource($val))
-                    $val = stream_get_contents($val);
+                if (is_resource($val)) {
+                                    $val = stream_get_contents($val);
+                }
 
                 // Taking out \r to not screw up the xml output
                 return str_replace("\r", "", $val);
@@ -748,8 +755,9 @@ 
      */
     public function beforeWriteContent($path, DAV\IFile $node, &$data, &$modified) {
 
-        if (!$node instanceof ICalendarObject)
-            return;
+        if (!$node instanceof ICalendarObject) {
+                    return;
+        }
 
         // We're onyl interested in ICalendarObject nodes that are inside of a
         // real calendar. This is to avoid triggering validation and scheduling
@@ -757,8 +765,9 @@ 
         list($parent) = Uri\split($path);
         $parentNode = $this->server->tree->getNodeForPath($parent);
 
-        if (!$parentNode instanceof ICalendar)
-            return;
+        if (!$parentNode instanceof ICalendar) {
+                    return;
+        }
 
         $this->validateICalendar(
             $data,
@@ -786,8 +795,9 @@ 
      */
     public function beforeCreateFile($path, &$data, DAV\ICollection $parentNode, &$modified) {
 
-        if (!$parentNode instanceof ICalendar)
-            return;
+        if (!$parentNode instanceof ICalendar) {
+                    return;
+        }
 
         $this->validateICalendar(
             $data,
@@ -825,7 +835,9 @@ 
         // Converting the data to unicode, if needed.
         $data = DAV\StringUtil::ensureUTF8($data);
 
-        if ($before !== md5($data)) $modified = true;
+        if ($before !== md5($data)) {
+        	$modified = true;
+        }
 
         try {
 
@@ -896,8 +908,9 @@ 
 
             }
         }
-        if (!$foundType)
-            throw new DAV\Exception\BadRequest('iCalendar object must contain at least 1 of VEVENT, VTODO or VJOURNAL');
+        if (!$foundType) {
+                    throw new DAV\Exception\BadRequest('iCalendar object must contain at least 1 of VEVENT, VTODO or VJOURNAL');
+        }
 
         // We use an extra variable to allow event handles to tell us wether
         // the object was modified or not.
@@ -945,8 +958,9 @@ 
      */
     public function htmlActionsPanel(DAV\INode $node, &$output) {
 
-        if (!$node instanceof CalendarHome)
-            return;
+        if (!$node instanceof CalendarHome) {
+                    return;
+        }
 
         $output .= '<tr><td colspan="2"><form method="post" action="">
             <h3>Create new calendar</h3>

libraries/SabreDAV/CalDAV/Principal/ProxyRead.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,8 +2,8 @@
 
 namespace Sabre\CalDAV\Principal;
 
-use Sabre\DAVACL;
 use Sabre\DAV;
+use Sabre\DAVACL;
 
 /**
  * ProxyRead principal

Indentation +159 added lines, -159 removed lines

@@ -18,164 +18,164 @@
  */
 class ProxyRead implements IProxyRead {
 
-    /**
-     * Principal information from the parent principal.
-     *
-     * @var array
-     */
-    protected $principalInfo;
-
-    /**
-     * Principal backend
-     *
-     * @var DAVACL\PrincipalBackend\BackendInterface
-     */
-    protected $principalBackend;
-
-    /**
-     * Creates the object.
-     *
-     * Note that you MUST supply the parent principal information.
-     *
-     * @param DAVACL\PrincipalBackend\BackendInterface $principalBackend
-     * @param array $principalInfo
-     */
-    public function __construct(DAVACL\PrincipalBackend\BackendInterface $principalBackend, array $principalInfo) {
-
-        $this->principalInfo = $principalInfo;
-        $this->principalBackend = $principalBackend;
-
-    }
-
-    /**
-     * Returns this principals name.
-     *
-     * @return string
-     */
-    public function getName() {
-
-        return 'calendar-proxy-read';
-
-    }
-
-    /**
-     * Returns the last modification time
-     *
-     * @return null
-     */
-    public function getLastModified() {
-
-        return null;
-
-    }
-
-    /**
-     * Deletes the current node
-     *
-     * @throws DAV\Exception\Forbidden
-     * @return void
-     */
-    public function delete() {
-
-        throw new DAV\Exception\Forbidden('Permission denied to delete node');
-
-    }
-
-    /**
-     * Renames the node
-     *
-     * @throws DAV\Exception\Forbidden
-     * @param string $name The new name
-     * @return void
-     */
-    public function setName($name) {
-
-        throw new DAV\Exception\Forbidden('Permission denied to rename file');
-
-    }
-
-
-    /**
-     * Returns a list of alternative urls for a principal
-     *
-     * This can for example be an email address, or ldap url.
-     *
-     * @return array
-     */
-    public function getAlternateUriSet() {
-
-        return [];
-
-    }
-
-    /**
-     * Returns the full principal url
-     *
-     * @return string
-     */
-    public function getPrincipalUrl() {
-
-        return $this->principalInfo['uri'] . '/' . $this->getName();
-
-    }
-
-    /**
-     * Returns the list of group members
-     *
-     * If this principal is a group, this function should return
-     * all member principal uri's for the group.
-     *
-     * @return array
-     */
-    public function getGroupMemberSet() {
-
-        return $this->principalBackend->getGroupMemberSet($this->getPrincipalUrl());
-
-    }
-
-    /**
-     * Returns the list of groups this principal is member of
-     *
-     * If this principal is a member of a (list of) groups, this function
-     * should return a list of principal uri's for it's members.
-     *
-     * @return array
-     */
-    public function getGroupMembership() {
-
-        return $this->principalBackend->getGroupMembership($this->getPrincipalUrl());
-
-    }
-
-    /**
-     * Sets a list of group members
-     *
-     * If this principal is a group, this method sets all the group members.
-     * The list of members is always overwritten, never appended to.
-     *
-     * This method should throw an exception if the members could not be set.
-     *
-     * @param array $principals
-     * @return void
-     */
-    public function setGroupMemberSet(array $principals) {
-
-        $this->principalBackend->setGroupMemberSet($this->getPrincipalUrl(), $principals);
-
-    }
-
-    /**
-     * Returns the displayname
-     *
-     * This should be a human readable name for the principal.
-     * If none is available, return the nodename.
-     *
-     * @return string
-     */
-    public function getDisplayName() {
-
-        return $this->getName();
-
-    }
+	/**
+	 * Principal information from the parent principal.
+	 *
+	 * @var array
+	 */
+	protected $principalInfo;
+
+	/**
+	 * Principal backend
+	 *
+	 * @var DAVACL\PrincipalBackend\BackendInterface
+	 */
+	protected $principalBackend;
+
+	/**
+	 * Creates the object.
+	 *
+	 * Note that you MUST supply the parent principal information.
+	 *
+	 * @param DAVACL\PrincipalBackend\BackendInterface $principalBackend
+	 * @param array $principalInfo
+	 */
+	public function __construct(DAVACL\PrincipalBackend\BackendInterface $principalBackend, array $principalInfo) {
+
+		$this->principalInfo = $principalInfo;
+		$this->principalBackend = $principalBackend;
+
+	}
+
+	/**
+	 * Returns this principals name.
+	 *
+	 * @return string
+	 */
+	public function getName() {
+
+		return 'calendar-proxy-read';
+
+	}
+
+	/**
+	 * Returns the last modification time
+	 *
+	 * @return null
+	 */
+	public function getLastModified() {
+
+		return null;
+
+	}
+
+	/**
+	 * Deletes the current node
+	 *
+	 * @throws DAV\Exception\Forbidden
+	 * @return void
+	 */
+	public function delete() {
+
+		throw new DAV\Exception\Forbidden('Permission denied to delete node');
+
+	}
+
+	/**
+	 * Renames the node
+	 *
+	 * @throws DAV\Exception\Forbidden
+	 * @param string $name The new name
+	 * @return void
+	 */
+	public function setName($name) {
+
+		throw new DAV\Exception\Forbidden('Permission denied to rename file');
+
+	}
+
+
+	/**
+	 * Returns a list of alternative urls for a principal
+	 *
+	 * This can for example be an email address, or ldap url.
+	 *
+	 * @return array
+	 */
+	public function getAlternateUriSet() {
+
+		return [];
+
+	}
+
+	/**
+	 * Returns the full principal url
+	 *
+	 * @return string
+	 */
+	public function getPrincipalUrl() {
+
+		return $this->principalInfo['uri'] . '/' . $this->getName();
+
+	}
+
+	/**
+	 * Returns the list of group members
+	 *
+	 * If this principal is a group, this function should return
+	 * all member principal uri's for the group.
+	 *
+	 * @return array
+	 */
+	public function getGroupMemberSet() {
+
+		return $this->principalBackend->getGroupMemberSet($this->getPrincipalUrl());
+
+	}
+
+	/**
+	 * Returns the list of groups this principal is member of
+	 *
+	 * If this principal is a member of a (list of) groups, this function
+	 * should return a list of principal uri's for it's members.
+	 *
+	 * @return array
+	 */
+	public function getGroupMembership() {
+
+		return $this->principalBackend->getGroupMembership($this->getPrincipalUrl());
+
+	}
+
+	/**
+	 * Sets a list of group members
+	 *
+	 * If this principal is a group, this method sets all the group members.
+	 * The list of members is always overwritten, never appended to.
+	 *
+	 * This method should throw an exception if the members could not be set.
+	 *
+	 * @param array $principals
+	 * @return void
+	 */
+	public function setGroupMemberSet(array $principals) {
+
+		$this->principalBackend->setGroupMemberSet($this->getPrincipalUrl(), $principals);
+
+	}
+
+	/**
+	 * Returns the displayname
+	 *
+	 * This should be a human readable name for the principal.
+	 * If none is available, return the nodename.
+	 *
+	 * @return string
+	 */
+	public function getDisplayName() {
+
+		return $this->getName();
+
+	}
 
 }

libraries/SabreDAV/CalDAV/Principal/ProxyWrite.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,8 +2,8 @@
 
 namespace Sabre\CalDAV\Principal;
 
-use Sabre\DAVACL;
 use Sabre\DAV;
+use Sabre\DAVACL;
 
 /**
  * ProxyWrite principal

Indentation +159 added lines, -159 removed lines

@@ -18,164 +18,164 @@
  */
 class ProxyWrite implements IProxyWrite {
 
-    /**
-     * Parent principal information
-     *
-     * @var array
-     */
-    protected $principalInfo;
-
-    /**
-     * Principal Backend
-     *
-     * @var DAVACL\PrincipalBackend\BackendInterface
-     */
-    protected $principalBackend;
-
-    /**
-     * Creates the object
-     *
-     * Note that you MUST supply the parent principal information.
-     *
-     * @param DAVACL\PrincipalBackend\BackendInterface $principalBackend
-     * @param array $principalInfo
-     */
-    public function __construct(DAVACL\PrincipalBackend\BackendInterface $principalBackend, array $principalInfo) {
-
-        $this->principalInfo = $principalInfo;
-        $this->principalBackend = $principalBackend;
-
-    }
-
-    /**
-     * Returns this principals name.
-     *
-     * @return string
-     */
-    public function getName() {
-
-        return 'calendar-proxy-write';
-
-    }
-
-    /**
-     * Returns the last modification time
-     *
-     * @return null
-     */
-    public function getLastModified() {
-
-        return null;
-
-    }
-
-    /**
-     * Deletes the current node
-     *
-     * @throws DAV\Exception\Forbidden
-     * @return void
-     */
-    public function delete() {
-
-        throw new DAV\Exception\Forbidden('Permission denied to delete node');
-
-    }
-
-    /**
-     * Renames the node
-     *
-     * @throws DAV\Exception\Forbidden
-     * @param string $name The new name
-     * @return void
-     */
-    public function setName($name) {
-
-        throw new DAV\Exception\Forbidden('Permission denied to rename file');
-
-    }
-
-
-    /**
-     * Returns a list of alternative urls for a principal
-     *
-     * This can for example be an email address, or ldap url.
-     *
-     * @return array
-     */
-    public function getAlternateUriSet() {
-
-        return [];
-
-    }
-
-    /**
-     * Returns the full principal url
-     *
-     * @return string
-     */
-    public function getPrincipalUrl() {
-
-        return $this->principalInfo['uri'] . '/' . $this->getName();
-
-    }
-
-    /**
-     * Returns the list of group members
-     *
-     * If this principal is a group, this function should return
-     * all member principal uri's for the group.
-     *
-     * @return array
-     */
-    public function getGroupMemberSet() {
-
-        return $this->principalBackend->getGroupMemberSet($this->getPrincipalUrl());
-
-    }
-
-    /**
-     * Returns the list of groups this principal is member of
-     *
-     * If this principal is a member of a (list of) groups, this function
-     * should return a list of principal uri's for it's members.
-     *
-     * @return array
-     */
-    public function getGroupMembership() {
-
-        return $this->principalBackend->getGroupMembership($this->getPrincipalUrl());
-
-    }
-
-    /**
-     * Sets a list of group members
-     *
-     * If this principal is a group, this method sets all the group members.
-     * The list of members is always overwritten, never appended to.
-     *
-     * This method should throw an exception if the members could not be set.
-     *
-     * @param array $principals
-     * @return void
-     */
-    public function setGroupMemberSet(array $principals) {
-
-        $this->principalBackend->setGroupMemberSet($this->getPrincipalUrl(), $principals);
-
-    }
-
-    /**
-     * Returns the displayname
-     *
-     * This should be a human readable name for the principal.
-     * If none is available, return the nodename.
-     *
-     * @return string
-     */
-    public function getDisplayName() {
-
-        return $this->getName();
-
-    }
+	/**
+	 * Parent principal information
+	 *
+	 * @var array
+	 */
+	protected $principalInfo;
+
+	/**
+	 * Principal Backend
+	 *
+	 * @var DAVACL\PrincipalBackend\BackendInterface
+	 */
+	protected $principalBackend;
+
+	/**
+	 * Creates the object
+	 *
+	 * Note that you MUST supply the parent principal information.
+	 *
+	 * @param DAVACL\PrincipalBackend\BackendInterface $principalBackend
+	 * @param array $principalInfo
+	 */
+	public function __construct(DAVACL\PrincipalBackend\BackendInterface $principalBackend, array $principalInfo) {
+
+		$this->principalInfo = $principalInfo;
+		$this->principalBackend = $principalBackend;
+
+	}
+
+	/**
+	 * Returns this principals name.
+	 *
+	 * @return string
+	 */
+	public function getName() {
+
+		return 'calendar-proxy-write';
+
+	}
+
+	/**
+	 * Returns the last modification time
+	 *
+	 * @return null
+	 */
+	public function getLastModified() {
+
+		return null;
+
+	}
+
+	/**
+	 * Deletes the current node
+	 *
+	 * @throws DAV\Exception\Forbidden
+	 * @return void
+	 */
+	public function delete() {
+
+		throw new DAV\Exception\Forbidden('Permission denied to delete node');
+
+	}
+
+	/**
+	 * Renames the node
+	 *
+	 * @throws DAV\Exception\Forbidden
+	 * @param string $name The new name
+	 * @return void
+	 */
+	public function setName($name) {
+
+		throw new DAV\Exception\Forbidden('Permission denied to rename file');
+
+	}
+
+
+	/**
+	 * Returns a list of alternative urls for a principal
+	 *
+	 * This can for example be an email address, or ldap url.
+	 *
+	 * @return array
+	 */
+	public function getAlternateUriSet() {
+
+		return [];
+
+	}
+
+	/**
+	 * Returns the full principal url
+	 *
+	 * @return string
+	 */
+	public function getPrincipalUrl() {
+
+		return $this->principalInfo['uri'] . '/' . $this->getName();
+
+	}
+
+	/**
+	 * Returns the list of group members
+	 *
+	 * If this principal is a group, this function should return
+	 * all member principal uri's for the group.
+	 *
+	 * @return array
+	 */
+	public function getGroupMemberSet() {
+
+		return $this->principalBackend->getGroupMemberSet($this->getPrincipalUrl());
+
+	}
+
+	/**
+	 * Returns the list of groups this principal is member of
+	 *
+	 * If this principal is a member of a (list of) groups, this function
+	 * should return a list of principal uri's for it's members.
+	 *
+	 * @return array
+	 */
+	public function getGroupMembership() {
+
+		return $this->principalBackend->getGroupMembership($this->getPrincipalUrl());
+
+	}
+
+	/**
+	 * Sets a list of group members
+	 *
+	 * If this principal is a group, this method sets all the group members.
+	 * The list of members is always overwritten, never appended to.
+	 *
+	 * This method should throw an exception if the members could not be set.
+	 *
+	 * @param array $principals
+	 * @return void
+	 */
+	public function setGroupMemberSet(array $principals) {
+
+		$this->principalBackend->setGroupMemberSet($this->getPrincipalUrl(), $principals);
+
+	}
+
+	/**
+	 * Returns the displayname
+	 *
+	 * This should be a human readable name for the principal.
+	 * If none is available, return the nodename.
+	 *
+	 * @return string
+	 */
+	public function getDisplayName() {
+
+		return $this->getName();
+
+	}
 
 }

libraries/SabreDAV/CalDAV/Schedule/IMipPlugin.php

Doc Comments +1 added lines, -1 removed lines

@@ -155,7 +155,7 @@
      * @param string $to Recipient email address
      * @param string $subject Subject of the email
      * @param string $body iCalendar body
-     * @param array $headers List of headers
+     * @param string[] $headers List of headers
      * @return void
      */
     protected function mail($to, $subject, $body, array $headers) {

Braces +6 added lines, -4 removed lines

@@ -99,11 +99,13 @@
 
         $summary = $iTipMessage->message->VEVENT->SUMMARY;
 
-        if (parse_url($iTipMessage->sender, PHP_URL_SCHEME) !== 'mailto')
-            return;
+        if (parse_url($iTipMessage->sender, PHP_URL_SCHEME) !== 'mailto') {
+                    return;
+        }
 
-        if (parse_url($iTipMessage->recipient, PHP_URL_SCHEME) !== 'mailto')
-            return;
+        if (parse_url($iTipMessage->recipient, PHP_URL_SCHEME) !== 'mailto') {
+                    return;
+        }
 
         $sender = substr($iTipMessage->sender, 7);
         $recipient = substr($iTipMessage->recipient, 7);

Indentation +155 added lines, -155 removed lines

@@ -21,35 +21,35 @@ 
  */
 class IMipPlugin extends DAV\ServerPlugin {
 
-    /**
-     * Email address used in From: header.
-     *
-     * @var string
-     */
-    protected $senderEmail;
-
-    /**
-     * ITipMessage
-     *
-     * @var ITip\Message
-     */
-    protected $itipMessage;
-
-    /**
-     * Creates the email handler.
-     *
-     * @param string $senderEmail. The 'senderEmail' is the email that shows up
-     *                             in the 'From:' address. This should
-     *                             generally be some kind of no-reply email
-     *                             address you own.
-     */
-    public function __construct($senderEmail) {
-
-        $this->senderEmail = $senderEmail;
-
-    }
-
-    /*
+	/**
+	 * Email address used in From: header.
+	 *
+	 * @var string
+	 */
+	protected $senderEmail;
+
+	/**
+	 * ITipMessage
+	 *
+	 * @var ITip\Message
+	 */
+	protected $itipMessage;
+
+	/**
+	 * Creates the email handler.
+	 *
+	 * @param string $senderEmail. The 'senderEmail' is the email that shows up
+	 *                             in the 'From:' address. This should
+	 *                             generally be some kind of no-reply email
+	 *                             address you own.
+	 */
+	public function __construct($senderEmail) {
+
+		$this->senderEmail = $senderEmail;
+
+	}
+
+	/*
      * This initializes the plugin.
      *
      * This function is called by Sabre\DAV\Server, after
@@ -60,131 +60,131 @@ 
      * @param DAV\Server $server
      * @return void
      */
-    public function initialize(DAV\Server $server) {
-
-        $server->on('schedule', [$this, 'schedule'], 120);
-
-    }
-
-    /**
-     * Returns a plugin name.
-     *
-     * Using this name other plugins will be able to access other plugins
-     * using \Sabre\DAV\Server::getPlugin
-     *
-     * @return string
-     */
-    public function getPluginName() {
-
-        return 'imip';
-
-    }
-
-    /**
-     * Event handler for the 'schedule' event.
-     *
-     * @param ITip\Message $iTipMessage
-     * @return void
-     */
-    public function schedule(ITip\Message $iTipMessage) {
-
-        // Not sending any emails if the system considers the update
-        // insignificant.
-        if (!$iTipMessage->significantChange) {
-            if (!$iTipMessage->scheduleStatus) {
-                $iTipMessage->scheduleStatus = '1.0;We got the message, but it\'s not significant enough to warrant an email';
-            }
-            return;
-        }
-
-        $summary = $iTipMessage->message->VEVENT->SUMMARY;
-
-        if (parse_url($iTipMessage->sender, PHP_URL_SCHEME) !== 'mailto')
-            return;
-
-        if (parse_url($iTipMessage->recipient, PHP_URL_SCHEME) !== 'mailto')
-            return;
-
-        $sender = substr($iTipMessage->sender, 7);
-        $recipient = substr($iTipMessage->recipient, 7);
-
-        if ($iTipMessage->senderName) {
-            $sender = $iTipMessage->senderName . ' <' . $sender . '>';
-        }
-        if ($iTipMessage->recipientName) {
-            $recipient = $iTipMessage->recipientName . ' <' . $recipient . '>';
-        }
-
-        $subject = 'SabreDAV iTIP message';
-        switch (strtoupper($iTipMessage->method)) {
-            case 'REPLY' :
-                $subject = 'Re: ' . $summary;
-                break;
-            case 'REQUEST' :
-                $subject = $summary;
-                break;
-            case 'CANCEL' :
-                $subject = 'Cancelled: ' . $summary;
-                break;
-        }
-
-        $headers = [
-            'Reply-To: ' . $sender,
-            'From: ' . $this->senderEmail,
-            'Content-Type: text/calendar; charset=UTF-8; method=' . $iTipMessage->method,
-        ];
-        if (DAV\Server::$exposeVersion) {
-            $headers[] = 'X-Sabre-Version: ' . DAV\Version::VERSION;
-        }
-        $this->mail(
-            $recipient,
-            $subject,
-            $iTipMessage->message->serialize(),
-            $headers
-        );
-        $iTipMessage->scheduleStatus = '1.1; Scheduling message is sent via iMip';
-
-    }
-
-    // @codeCoverageIgnoreStart
-    // This is deemed untestable in a reasonable manner
-
-    /**
-     * This function is responsible for sending the actual email.
-     *
-     * @param string $to Recipient email address
-     * @param string $subject Subject of the email
-     * @param string $body iCalendar body
-     * @param array $headers List of headers
-     * @return void
-     */
-    protected function mail($to, $subject, $body, array $headers) {
-
-        mail($to, $subject, $body, implode("\r\n", $headers));
-
-    }
-
-    // @codeCoverageIgnoreEnd
-
-    /**
-     * Returns a bunch of meta-data about the plugin.
-     *
-     * Providing this information is optional, and is mainly displayed by the
-     * Browser plugin.
-     *
-     * The description key in the returned array may contain html and will not
-     * be sanitized.
-     *
-     * @return array
-     */
-    public function getPluginInfo() {
-
-        return [
-            'name'        => $this->getPluginName(),
-            'description' => 'Email delivery (rfc6037) for CalDAV scheduling',
-            'link'        => 'http://sabre.io/dav/scheduling/',
-        ];
-
-    }
+	public function initialize(DAV\Server $server) {
+
+		$server->on('schedule', [$this, 'schedule'], 120);
+
+	}
+
+	/**
+	 * Returns a plugin name.
+	 *
+	 * Using this name other plugins will be able to access other plugins
+	 * using \Sabre\DAV\Server::getPlugin
+	 *
+	 * @return string
+	 */
+	public function getPluginName() {
+
+		return 'imip';
+
+	}
+
+	/**
+	 * Event handler for the 'schedule' event.
+	 *
+	 * @param ITip\Message $iTipMessage
+	 * @return void
+	 */
+	public function schedule(ITip\Message $iTipMessage) {
+
+		// Not sending any emails if the system considers the update
+		// insignificant.
+		if (!$iTipMessage->significantChange) {
+			if (!$iTipMessage->scheduleStatus) {
+				$iTipMessage->scheduleStatus = '1.0;We got the message, but it\'s not significant enough to warrant an email';
+			}
+			return;
+		}
+
+		$summary = $iTipMessage->message->VEVENT->SUMMARY;
+
+		if (parse_url($iTipMessage->sender, PHP_URL_SCHEME) !== 'mailto')
+			return;
+
+		if (parse_url($iTipMessage->recipient, PHP_URL_SCHEME) !== 'mailto')
+			return;
+
+		$sender = substr($iTipMessage->sender, 7);
+		$recipient = substr($iTipMessage->recipient, 7);
+
+		if ($iTipMessage->senderName) {
+			$sender = $iTipMessage->senderName . ' <' . $sender . '>';
+		}
+		if ($iTipMessage->recipientName) {
+			$recipient = $iTipMessage->recipientName . ' <' . $recipient . '>';
+		}
+
+		$subject = 'SabreDAV iTIP message';
+		switch (strtoupper($iTipMessage->method)) {
+			case 'REPLY' :
+				$subject = 'Re: ' . $summary;
+				break;
+			case 'REQUEST' :
+				$subject = $summary;
+				break;
+			case 'CANCEL' :
+				$subject = 'Cancelled: ' . $summary;
+				break;
+		}
+
+		$headers = [
+			'Reply-To: ' . $sender,
+			'From: ' . $this->senderEmail,
+			'Content-Type: text/calendar; charset=UTF-8; method=' . $iTipMessage->method,
+		];
+		if (DAV\Server::$exposeVersion) {
+			$headers[] = 'X-Sabre-Version: ' . DAV\Version::VERSION;
+		}
+		$this->mail(
+			$recipient,
+			$subject,
+			$iTipMessage->message->serialize(),
+			$headers
+		);
+		$iTipMessage->scheduleStatus = '1.1; Scheduling message is sent via iMip';
+
+	}
+
+	// @codeCoverageIgnoreStart
+	// This is deemed untestable in a reasonable manner
+
+	/**
+	 * This function is responsible for sending the actual email.
+	 *
+	 * @param string $to Recipient email address
+	 * @param string $subject Subject of the email
+	 * @param string $body iCalendar body
+	 * @param array $headers List of headers
+	 * @return void
+	 */
+	protected function mail($to, $subject, $body, array $headers) {
+
+		mail($to, $subject, $body, implode("\r\n", $headers));
+
+	}
+
+	// @codeCoverageIgnoreEnd
+
+	/**
+	 * Returns a bunch of meta-data about the plugin.
+	 *
+	 * Providing this information is optional, and is mainly displayed by the
+	 * Browser plugin.
+	 *
+	 * The description key in the returned array may contain html and will not
+	 * be sanitized.
+	 *
+	 * @return array
+	 */
+	public function getPluginInfo() {
+
+		return [
+			'name'        => $this->getPluginName(),
+			'description' => 'Email delivery (rfc6037) for CalDAV scheduling',
+			'link'        => 'http://sabre.io/dav/scheduling/',
+		];
+
+	}
 
 }