Dolibarr / dolibarr

htdocs/includes/sabre/sabre/dav/lib/DAV/CorePlugin.php

Indentation +853 added lines, -853 removed lines

@@ -18,433 +18,433 @@ 
  */
 class CorePlugin extends ServerPlugin
 {
-    /**
-     * Reference to server object.
-     *
-     * @var Server
-     */
-    protected $server;
-
-    /**
-     * Sets up the plugin.
-     */
-    public function initialize(Server $server)
-    {
-        $this->server = $server;
-        $server->on('method:GET', [$this, 'httpGet']);
-        $server->on('method:OPTIONS', [$this, 'httpOptions']);
-        $server->on('method:HEAD', [$this, 'httpHead']);
-        $server->on('method:DELETE', [$this, 'httpDelete']);
-        $server->on('method:PROPFIND', [$this, 'httpPropFind']);
-        $server->on('method:PROPPATCH', [$this, 'httpPropPatch']);
-        $server->on('method:PUT', [$this, 'httpPut']);
-        $server->on('method:MKCOL', [$this, 'httpMkcol']);
-        $server->on('method:MOVE', [$this, 'httpMove']);
-        $server->on('method:COPY', [$this, 'httpCopy']);
-        $server->on('method:REPORT', [$this, 'httpReport']);
-
-        $server->on('propPatch', [$this, 'propPatchProtectedPropertyCheck'], 90);
-        $server->on('propPatch', [$this, 'propPatchNodeUpdate'], 200);
-        $server->on('propFind', [$this, 'propFind']);
-        $server->on('propFind', [$this, 'propFindNode'], 120);
-        $server->on('propFind', [$this, 'propFindLate'], 200);
-
-        $server->on('exception', [$this, 'exception']);
-    }
-
-    /**
-     * 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 'core';
-    }
-
-    /**
-     * This is the default implementation for the GET method.
-     *
-     * @return bool
-     */
-    public function httpGet(RequestInterface $request, ResponseInterface $response)
-    {
-        $path = $request->getPath();
-        $node = $this->server->tree->getNodeForPath($path);
-
-        if (!$node instanceof IFile) {
-            return;
-        }
-
-        if ('HEAD' === $request->getHeader('X-Sabre-Original-Method')) {
-            $body = '';
-        } else {
-            $body = $node->get();
-
-            // Converting string into stream, if needed.
-            if (is_string($body)) {
-                $stream = fopen('php://temp', 'r+');
-                fwrite($stream, $body);
-                rewind($stream);
-                $body = $stream;
-            }
-        }
-
-        /*
+	/**
+	 * Reference to server object.
+	 *
+	 * @var Server
+	 */
+	protected $server;
+
+	/**
+	 * Sets up the plugin.
+	 */
+	public function initialize(Server $server)
+	{
+		$this->server = $server;
+		$server->on('method:GET', [$this, 'httpGet']);
+		$server->on('method:OPTIONS', [$this, 'httpOptions']);
+		$server->on('method:HEAD', [$this, 'httpHead']);
+		$server->on('method:DELETE', [$this, 'httpDelete']);
+		$server->on('method:PROPFIND', [$this, 'httpPropFind']);
+		$server->on('method:PROPPATCH', [$this, 'httpPropPatch']);
+		$server->on('method:PUT', [$this, 'httpPut']);
+		$server->on('method:MKCOL', [$this, 'httpMkcol']);
+		$server->on('method:MOVE', [$this, 'httpMove']);
+		$server->on('method:COPY', [$this, 'httpCopy']);
+		$server->on('method:REPORT', [$this, 'httpReport']);
+
+		$server->on('propPatch', [$this, 'propPatchProtectedPropertyCheck'], 90);
+		$server->on('propPatch', [$this, 'propPatchNodeUpdate'], 200);
+		$server->on('propFind', [$this, 'propFind']);
+		$server->on('propFind', [$this, 'propFindNode'], 120);
+		$server->on('propFind', [$this, 'propFindLate'], 200);
+
+		$server->on('exception', [$this, 'exception']);
+	}
+
+	/**
+	 * 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 'core';
+	}
+
+	/**
+	 * This is the default implementation for the GET method.
+	 *
+	 * @return bool
+	 */
+	public function httpGet(RequestInterface $request, ResponseInterface $response)
+	{
+		$path = $request->getPath();
+		$node = $this->server->tree->getNodeForPath($path);
+
+		if (!$node instanceof IFile) {
+			return;
+		}
+
+		if ('HEAD' === $request->getHeader('X-Sabre-Original-Method')) {
+			$body = '';
+		} else {
+			$body = $node->get();
+
+			// Converting string into stream, if needed.
+			if (is_string($body)) {
+				$stream = fopen('php://temp', 'r+');
+				fwrite($stream, $body);
+				rewind($stream);
+				$body = $stream;
+			}
+		}
+
+		/*
          * TODO: getetag, getlastmodified, getsize should also be used using
          * this method
          */
-        $httpHeaders = $this->server->getHTTPHeaders($path);
+		$httpHeaders = $this->server->getHTTPHeaders($path);
 
-        /* ContentType needs to get a default, because many webservers will otherwise
+		/* ContentType needs to get a default, because many webservers will otherwise
          * default to text/html, and we don't want this for security reasons.
          */
-        if (!isset($httpHeaders['Content-Type'])) {
-            $httpHeaders['Content-Type'] = 'application/octet-stream';
-        }
-
-        if (isset($httpHeaders['Content-Length'])) {
-            $nodeSize = $httpHeaders['Content-Length'];
-
-            // Need to unset Content-Length, because we'll handle that during figuring out the range
-            unset($httpHeaders['Content-Length']);
-        } else {
-            $nodeSize = null;
-        }
-
-        $response->addHeaders($httpHeaders);
-
-        $range = $this->server->getHTTPRange();
-        $ifRange = $request->getHeader('If-Range');
-        $ignoreRangeHeader = false;
-
-        // If ifRange is set, and range is specified, we first need to check
-        // the precondition.
-        if ($nodeSize && $range && $ifRange) {
-            // if IfRange is parsable as a date we'll treat it as a DateTime
-            // otherwise, we must treat it as an etag.
-            try {
-                $ifRangeDate = new \DateTime($ifRange);
-
-                // It's a date. We must check if the entity is modified since
-                // the specified date.
-                if (!isset($httpHeaders['Last-Modified'])) {
-                    $ignoreRangeHeader = true;
-                } else {
-                    $modified = new \DateTime($httpHeaders['Last-Modified']);
-                    if ($modified > $ifRangeDate) {
-                        $ignoreRangeHeader = true;
-                    }
-                }
-            } catch (\Exception $e) {
-                // It's an entity. We can do a simple comparison.
-                if (!isset($httpHeaders['ETag'])) {
-                    $ignoreRangeHeader = true;
-                } elseif ($httpHeaders['ETag'] !== $ifRange) {
-                    $ignoreRangeHeader = true;
-                }
-            }
-        }
-
-        // We're only going to support HTTP ranges if the backend provided a filesize
-        if (!$ignoreRangeHeader && $nodeSize && $range) {
-            // Determining the exact byte offsets
-            if (!is_null($range[0])) {
-                $start = $range[0];
-                $end = $range[1] ? $range[1] : $nodeSize - 1;
-                if ($start >= $nodeSize) {
-                    throw new Exception\RequestedRangeNotSatisfiable('The start offset ('.$range[0].') exceeded the size of the entity ('.$nodeSize.')');
-                }
-                if ($end < $start) {
-                    throw new Exception\RequestedRangeNotSatisfiable('The end offset ('.$range[1].') is lower than the start offset ('.$range[0].')');
-                }
-                if ($end >= $nodeSize) {
-                    $end = $nodeSize - 1;
-                }
-            } else {
-                $start = $nodeSize - $range[1];
-                $end = $nodeSize - 1;
-
-                if ($start < 0) {
-                    $start = 0;
-                }
-            }
-
-            // Streams may advertise themselves as seekable, but still not
-            // actually allow fseek.  We'll manually go forward in the stream
-            // if fseek failed.
-            if (!stream_get_meta_data($body)['seekable'] || -1 === fseek($body, $start, SEEK_SET)) {
-                $consumeBlock = 8192;
-                for ($consumed = 0; $start - $consumed > 0;) {
-                    if (feof($body)) {
-                        throw new Exception\RequestedRangeNotSatisfiable('The start offset ('.$start.') exceeded the size of the entity ('.$consumed.')');
-                    }
-                    $consumed += strlen(fread($body, min($start - $consumed, $consumeBlock)));
-                }
-            }
-
-            $response->setHeader('Content-Length', $end - $start + 1);
-            $response->setHeader('Content-Range', 'bytes '.$start.'-'.$end.'/'.$nodeSize);
-            $response->setStatus(206);
-            $response->setBody($body);
-        } else {
-            if ($nodeSize) {
-                $response->setHeader('Content-Length', $nodeSize);
-            }
-            $response->setStatus(200);
-            $response->setBody($body);
-        }
-        // Sending back false will interrupt the event chain and tell the server
-        // we've handled this method.
-        return false;
-    }
-
-    /**
-     * HTTP OPTIONS.
-     *
-     * @return bool
-     */
-    public function httpOptions(RequestInterface $request, ResponseInterface $response)
-    {
-        $methods = $this->server->getAllowedMethods($request->getPath());
-
-        $response->setHeader('Allow', strtoupper(implode(', ', $methods)));
-        $features = ['1', '3', 'extended-mkcol'];
-
-        foreach ($this->server->getPlugins() as $plugin) {
-            $features = array_merge($features, $plugin->getFeatures());
-        }
-
-        $response->setHeader('DAV', implode(', ', $features));
-        $response->setHeader('MS-Author-Via', 'DAV');
-        $response->setHeader('Accept-Ranges', 'bytes');
-        $response->setHeader('Content-Length', '0');
-        $response->setStatus(200);
-
-        // Sending back false will interrupt the event chain and tell the server
-        // we've handled this method.
-        return false;
-    }
-
-    /**
-     * HTTP HEAD.
-     *
-     * This method is normally used to take a peak at a url, and only get the
-     * HTTP response headers, without the body. This is used by clients to
-     * determine if a remote file was changed, so they can use a local cached
-     * version, instead of downloading it again
-     *
-     * @return bool
-     */
-    public function httpHead(RequestInterface $request, ResponseInterface $response)
-    {
-        // This is implemented by changing the HEAD request to a GET request,
-        // and telling the request handler that is doesn't need to create the body.
-        $subRequest = clone $request;
-        $subRequest->setMethod('GET');
-        $subRequest->setHeader('X-Sabre-Original-Method', 'HEAD');
-
-        try {
-            $this->server->invokeMethod($subRequest, $response, false);
-        } catch (Exception\NotImplemented $e) {
-            // Some clients may do HEAD requests on collections, however, GET
-            // requests and HEAD requests _may_ not be defined on a collection,
-            // which would trigger a 501.
-            // This breaks some clients though, so we're transforming these
-            // 501s into 200s.
-            $response->setStatus(200);
-            $response->setBody('');
-            $response->setHeader('Content-Type', 'text/plain');
-            $response->setHeader('X-Sabre-Real-Status', $e->getHTTPCode());
-        }
-
-        // Sending back false will interrupt the event chain and tell the server
-        // we've handled this method.
-        return false;
-    }
-
-    /**
-     * HTTP Delete.
-     *
-     * The HTTP delete method, deletes a given uri
-     */
-    public function httpDelete(RequestInterface $request, ResponseInterface $response)
-    {
-        $path = $request->getPath();
-
-        if (!$this->server->emit('beforeUnbind', [$path])) {
-            return false;
-        }
-        $this->server->tree->delete($path);
-        $this->server->emit('afterUnbind', [$path]);
-
-        $response->setStatus(204);
-        $response->setHeader('Content-Length', '0');
-
-        // Sending back false will interrupt the event chain and tell the server
-        // we've handled this method.
-        return false;
-    }
-
-    /**
-     * WebDAV PROPFIND.
-     *
-     * This WebDAV method requests information about an uri resource, or a list of resources
-     * If a client wants to receive the properties for a single resource it will add an HTTP Depth: header with a 0 value
-     * If the value is 1, it means that it also expects a list of sub-resources (e.g.: files in a directory)
-     *
-     * The request body contains an XML data structure that has a list of properties the client understands
-     * The response body is also an xml document, containing information about every uri resource and the requested properties
-     *
-     * It has to return a HTTP 207 Multi-status status code
-     */
-    public function httpPropFind(RequestInterface $request, ResponseInterface $response)
-    {
-        $path = $request->getPath();
-
-        $requestBody = $request->getBodyAsString();
-        if (strlen($requestBody)) {
-            try {
-                $propFindXml = $this->server->xml->expect('{DAV:}propfind', $requestBody);
-            } catch (ParseException $e) {
-                throw new BadRequest($e->getMessage(), 0, $e);
-            }
-        } else {
-            $propFindXml = new Xml\Request\PropFind();
-            $propFindXml->allProp = true;
-            $propFindXml->properties = [];
-        }
-
-        $depth = $this->server->getHTTPDepth(1);
-        // The only two options for the depth of a propfind is 0 or 1 - as long as depth infinity is not enabled
-        if (!$this->server->enablePropfindDepthInfinity && 0 != $depth) {
-            $depth = 1;
-        }
-
-        $newProperties = $this->server->getPropertiesIteratorForPath($path, $propFindXml->properties, $depth);
-
-        // This is a multi-status response
-        $response->setStatus(207);
-        $response->setHeader('Content-Type', 'application/xml; charset=utf-8');
-        $response->setHeader('Vary', 'Brief,Prefer');
-
-        // Normally this header is only needed for OPTIONS responses, however..
-        // iCal seems to also depend on these being set for PROPFIND. Since
-        // this is not harmful, we'll add it.
-        $features = ['1', '3', 'extended-mkcol'];
-        foreach ($this->server->getPlugins() as $plugin) {
-            $features = array_merge($features, $plugin->getFeatures());
-        }
-        $response->setHeader('DAV', implode(', ', $features));
-
-        $prefer = $this->server->getHTTPPrefer();
-        $minimal = 'minimal' === $prefer['return'];
-
-        $data = $this->server->generateMultiStatus($newProperties, $minimal);
-        $response->setBody($data);
-
-        // Sending back false will interrupt the event chain and tell the server
-        // we've handled this method.
-        return false;
-    }
-
-    /**
-     * WebDAV PROPPATCH.
-     *
-     * This method is called to update properties on a Node. The request is an XML body with all the mutations.
-     * In this XML body it is specified which properties should be set/updated and/or deleted
-     *
-     * @return bool
-     */
-    public function httpPropPatch(RequestInterface $request, ResponseInterface $response)
-    {
-        $path = $request->getPath();
-
-        try {
-            $propPatch = $this->server->xml->expect('{DAV:}propertyupdate', $request->getBody());
-        } catch (ParseException $e) {
-            throw new BadRequest($e->getMessage(), 0, $e);
-        }
-        $newProperties = $propPatch->properties;
-
-        $result = $this->server->updateProperties($path, $newProperties);
-
-        $prefer = $this->server->getHTTPPrefer();
-        $response->setHeader('Vary', 'Brief,Prefer');
-
-        if ('minimal' === $prefer['return']) {
-            // If return-minimal is specified, we only have to check if the
-            // request was successful, and don't need to return the
-            // multi-status.
-            $ok = true;
-            foreach ($result as $prop => $code) {
-                if ((int) $code > 299) {
-                    $ok = false;
-                }
-            }
-
-            if ($ok) {
-                $response->setStatus(204);
-
-                return false;
-            }
-        }
-
-        $response->setStatus(207);
-        $response->setHeader('Content-Type', 'application/xml; charset=utf-8');
-
-        // Reorganizing the result for generateMultiStatus
-        $multiStatus = [];
-        foreach ($result as $propertyName => $code) {
-            if (isset($multiStatus[$code])) {
-                $multiStatus[$code][$propertyName] = null;
-            } else {
-                $multiStatus[$code] = [$propertyName => null];
-            }
-        }
-        $multiStatus['href'] = $path;
-
-        $response->setBody(
-            $this->server->generateMultiStatus([$multiStatus])
-        );
-
-        // Sending back false will interrupt the event chain and tell the server
-        // we've handled this method.
-        return false;
-    }
-
-    /**
-     * HTTP PUT method.
-     *
-     * This HTTP method updates a file, or creates a new one.
-     *
-     * If a new resource was created, a 201 Created status code should be returned. If an existing resource is updated, it's a 204 No Content
-     *
-     * @return bool
-     */
-    public function httpPut(RequestInterface $request, ResponseInterface $response)
-    {
-        $body = $request->getBodyAsStream();
-        $path = $request->getPath();
-
-        // Intercepting Content-Range
-        if ($request->getHeader('Content-Range')) {
-            /*
+		if (!isset($httpHeaders['Content-Type'])) {
+			$httpHeaders['Content-Type'] = 'application/octet-stream';
+		}
+
+		if (isset($httpHeaders['Content-Length'])) {
+			$nodeSize = $httpHeaders['Content-Length'];
+
+			// Need to unset Content-Length, because we'll handle that during figuring out the range
+			unset($httpHeaders['Content-Length']);
+		} else {
+			$nodeSize = null;
+		}
+
+		$response->addHeaders($httpHeaders);
+
+		$range = $this->server->getHTTPRange();
+		$ifRange = $request->getHeader('If-Range');
+		$ignoreRangeHeader = false;
+
+		// If ifRange is set, and range is specified, we first need to check
+		// the precondition.
+		if ($nodeSize && $range && $ifRange) {
+			// if IfRange is parsable as a date we'll treat it as a DateTime
+			// otherwise, we must treat it as an etag.
+			try {
+				$ifRangeDate = new \DateTime($ifRange);
+
+				// It's a date. We must check if the entity is modified since
+				// the specified date.
+				if (!isset($httpHeaders['Last-Modified'])) {
+					$ignoreRangeHeader = true;
+				} else {
+					$modified = new \DateTime($httpHeaders['Last-Modified']);
+					if ($modified > $ifRangeDate) {
+						$ignoreRangeHeader = true;
+					}
+				}
+			} catch (\Exception $e) {
+				// It's an entity. We can do a simple comparison.
+				if (!isset($httpHeaders['ETag'])) {
+					$ignoreRangeHeader = true;
+				} elseif ($httpHeaders['ETag'] !== $ifRange) {
+					$ignoreRangeHeader = true;
+				}
+			}
+		}
+
+		// We're only going to support HTTP ranges if the backend provided a filesize
+		if (!$ignoreRangeHeader && $nodeSize && $range) {
+			// Determining the exact byte offsets
+			if (!is_null($range[0])) {
+				$start = $range[0];
+				$end = $range[1] ? $range[1] : $nodeSize - 1;
+				if ($start >= $nodeSize) {
+					throw new Exception\RequestedRangeNotSatisfiable('The start offset ('.$range[0].') exceeded the size of the entity ('.$nodeSize.')');
+				}
+				if ($end < $start) {
+					throw new Exception\RequestedRangeNotSatisfiable('The end offset ('.$range[1].') is lower than the start offset ('.$range[0].')');
+				}
+				if ($end >= $nodeSize) {
+					$end = $nodeSize - 1;
+				}
+			} else {
+				$start = $nodeSize - $range[1];
+				$end = $nodeSize - 1;
+
+				if ($start < 0) {
+					$start = 0;
+				}
+			}
+
+			// Streams may advertise themselves as seekable, but still not
+			// actually allow fseek.  We'll manually go forward in the stream
+			// if fseek failed.
+			if (!stream_get_meta_data($body)['seekable'] || -1 === fseek($body, $start, SEEK_SET)) {
+				$consumeBlock = 8192;
+				for ($consumed = 0; $start - $consumed > 0;) {
+					if (feof($body)) {
+						throw new Exception\RequestedRangeNotSatisfiable('The start offset ('.$start.') exceeded the size of the entity ('.$consumed.')');
+					}
+					$consumed += strlen(fread($body, min($start - $consumed, $consumeBlock)));
+				}
+			}
+
+			$response->setHeader('Content-Length', $end - $start + 1);
+			$response->setHeader('Content-Range', 'bytes '.$start.'-'.$end.'/'.$nodeSize);
+			$response->setStatus(206);
+			$response->setBody($body);
+		} else {
+			if ($nodeSize) {
+				$response->setHeader('Content-Length', $nodeSize);
+			}
+			$response->setStatus(200);
+			$response->setBody($body);
+		}
+		// Sending back false will interrupt the event chain and tell the server
+		// we've handled this method.
+		return false;
+	}
+
+	/**
+	 * HTTP OPTIONS.
+	 *
+	 * @return bool
+	 */
+	public function httpOptions(RequestInterface $request, ResponseInterface $response)
+	{
+		$methods = $this->server->getAllowedMethods($request->getPath());
+
+		$response->setHeader('Allow', strtoupper(implode(', ', $methods)));
+		$features = ['1', '3', 'extended-mkcol'];
+
+		foreach ($this->server->getPlugins() as $plugin) {
+			$features = array_merge($features, $plugin->getFeatures());
+		}
+
+		$response->setHeader('DAV', implode(', ', $features));
+		$response->setHeader('MS-Author-Via', 'DAV');
+		$response->setHeader('Accept-Ranges', 'bytes');
+		$response->setHeader('Content-Length', '0');
+		$response->setStatus(200);
+
+		// Sending back false will interrupt the event chain and tell the server
+		// we've handled this method.
+		return false;
+	}
+
+	/**
+	 * HTTP HEAD.
+	 *
+	 * This method is normally used to take a peak at a url, and only get the
+	 * HTTP response headers, without the body. This is used by clients to
+	 * determine if a remote file was changed, so they can use a local cached
+	 * version, instead of downloading it again
+	 *
+	 * @return bool
+	 */
+	public function httpHead(RequestInterface $request, ResponseInterface $response)
+	{
+		// This is implemented by changing the HEAD request to a GET request,
+		// and telling the request handler that is doesn't need to create the body.
+		$subRequest = clone $request;
+		$subRequest->setMethod('GET');
+		$subRequest->setHeader('X-Sabre-Original-Method', 'HEAD');
+
+		try {
+			$this->server->invokeMethod($subRequest, $response, false);
+		} catch (Exception\NotImplemented $e) {
+			// Some clients may do HEAD requests on collections, however, GET
+			// requests and HEAD requests _may_ not be defined on a collection,
+			// which would trigger a 501.
+			// This breaks some clients though, so we're transforming these
+			// 501s into 200s.
+			$response->setStatus(200);
+			$response->setBody('');
+			$response->setHeader('Content-Type', 'text/plain');
+			$response->setHeader('X-Sabre-Real-Status', $e->getHTTPCode());
+		}
+
+		// Sending back false will interrupt the event chain and tell the server
+		// we've handled this method.
+		return false;
+	}
+
+	/**
+	 * HTTP Delete.
+	 *
+	 * The HTTP delete method, deletes a given uri
+	 */
+	public function httpDelete(RequestInterface $request, ResponseInterface $response)
+	{
+		$path = $request->getPath();
+
+		if (!$this->server->emit('beforeUnbind', [$path])) {
+			return false;
+		}
+		$this->server->tree->delete($path);
+		$this->server->emit('afterUnbind', [$path]);
+
+		$response->setStatus(204);
+		$response->setHeader('Content-Length', '0');
+
+		// Sending back false will interrupt the event chain and tell the server
+		// we've handled this method.
+		return false;
+	}
+
+	/**
+	 * WebDAV PROPFIND.
+	 *
+	 * This WebDAV method requests information about an uri resource, or a list of resources
+	 * If a client wants to receive the properties for a single resource it will add an HTTP Depth: header with a 0 value
+	 * If the value is 1, it means that it also expects a list of sub-resources (e.g.: files in a directory)
+	 *
+	 * The request body contains an XML data structure that has a list of properties the client understands
+	 * The response body is also an xml document, containing information about every uri resource and the requested properties
+	 *
+	 * It has to return a HTTP 207 Multi-status status code
+	 */
+	public function httpPropFind(RequestInterface $request, ResponseInterface $response)
+	{
+		$path = $request->getPath();
+
+		$requestBody = $request->getBodyAsString();
+		if (strlen($requestBody)) {
+			try {
+				$propFindXml = $this->server->xml->expect('{DAV:}propfind', $requestBody);
+			} catch (ParseException $e) {
+				throw new BadRequest($e->getMessage(), 0, $e);
+			}
+		} else {
+			$propFindXml = new Xml\Request\PropFind();
+			$propFindXml->allProp = true;
+			$propFindXml->properties = [];
+		}
+
+		$depth = $this->server->getHTTPDepth(1);
+		// The only two options for the depth of a propfind is 0 or 1 - as long as depth infinity is not enabled
+		if (!$this->server->enablePropfindDepthInfinity && 0 != $depth) {
+			$depth = 1;
+		}
+
+		$newProperties = $this->server->getPropertiesIteratorForPath($path, $propFindXml->properties, $depth);
+
+		// This is a multi-status response
+		$response->setStatus(207);
+		$response->setHeader('Content-Type', 'application/xml; charset=utf-8');
+		$response->setHeader('Vary', 'Brief,Prefer');
+
+		// Normally this header is only needed for OPTIONS responses, however..
+		// iCal seems to also depend on these being set for PROPFIND. Since
+		// this is not harmful, we'll add it.
+		$features = ['1', '3', 'extended-mkcol'];
+		foreach ($this->server->getPlugins() as $plugin) {
+			$features = array_merge($features, $plugin->getFeatures());
+		}
+		$response->setHeader('DAV', implode(', ', $features));
+
+		$prefer = $this->server->getHTTPPrefer();
+		$minimal = 'minimal' === $prefer['return'];
+
+		$data = $this->server->generateMultiStatus($newProperties, $minimal);
+		$response->setBody($data);
+
+		// Sending back false will interrupt the event chain and tell the server
+		// we've handled this method.
+		return false;
+	}
+
+	/**
+	 * WebDAV PROPPATCH.
+	 *
+	 * This method is called to update properties on a Node. The request is an XML body with all the mutations.
+	 * In this XML body it is specified which properties should be set/updated and/or deleted
+	 *
+	 * @return bool
+	 */
+	public function httpPropPatch(RequestInterface $request, ResponseInterface $response)
+	{
+		$path = $request->getPath();
+
+		try {
+			$propPatch = $this->server->xml->expect('{DAV:}propertyupdate', $request->getBody());
+		} catch (ParseException $e) {
+			throw new BadRequest($e->getMessage(), 0, $e);
+		}
+		$newProperties = $propPatch->properties;
+
+		$result = $this->server->updateProperties($path, $newProperties);
+
+		$prefer = $this->server->getHTTPPrefer();
+		$response->setHeader('Vary', 'Brief,Prefer');
+
+		if ('minimal' === $prefer['return']) {
+			// If return-minimal is specified, we only have to check if the
+			// request was successful, and don't need to return the
+			// multi-status.
+			$ok = true;
+			foreach ($result as $prop => $code) {
+				if ((int) $code > 299) {
+					$ok = false;
+				}
+			}
+
+			if ($ok) {
+				$response->setStatus(204);
+
+				return false;
+			}
+		}
+
+		$response->setStatus(207);
+		$response->setHeader('Content-Type', 'application/xml; charset=utf-8');
+
+		// Reorganizing the result for generateMultiStatus
+		$multiStatus = [];
+		foreach ($result as $propertyName => $code) {
+			if (isset($multiStatus[$code])) {
+				$multiStatus[$code][$propertyName] = null;
+			} else {
+				$multiStatus[$code] = [$propertyName => null];
+			}
+		}
+		$multiStatus['href'] = $path;
+
+		$response->setBody(
+			$this->server->generateMultiStatus([$multiStatus])
+		);
+
+		// Sending back false will interrupt the event chain and tell the server
+		// we've handled this method.
+		return false;
+	}
+
+	/**
+	 * HTTP PUT method.
+	 *
+	 * This HTTP method updates a file, or creates a new one.
+	 *
+	 * If a new resource was created, a 201 Created status code should be returned. If an existing resource is updated, it's a 204 No Content
+	 *
+	 * @return bool
+	 */
+	public function httpPut(RequestInterface $request, ResponseInterface $response)
+	{
+		$body = $request->getBodyAsStream();
+		$path = $request->getPath();
+
+		// Intercepting Content-Range
+		if ($request->getHeader('Content-Range')) {
+			/*
                An origin server that allows PUT on a given target resource MUST send
                a 400 (Bad Request) response to a PUT request that contains a
                Content-Range header field.
 
                Reference: http://tools.ietf.org/html/rfc7231#section-4.3.4
             */
-            throw new Exception\BadRequest('Content-Range on PUT requests are forbidden.');
-        }
+			throw new Exception\BadRequest('Content-Range on PUT requests are forbidden.');
+		}
 
-        // Intercepting the Finder problem
-        if (($expected = $request->getHeader('X-Expected-Entity-Length')) && $expected > 0) {
-            /*
+		// Intercepting the Finder problem
+		if (($expected = $request->getHeader('X-Expected-Entity-Length')) && $expected > 0) {
+			/*
             Many webservers will not cooperate well with Finder PUT requests,
             because it uses 'Chunked' transfer encoding for the request body.
 
@@ -465,443 +465,443 @@ 
             protect the end-user.
             */
 
-            // Only reading first byte
-            $firstByte = fread($body, 1);
-            if (1 !== strlen($firstByte)) {
-                throw new Exception\Forbidden('This server is not compatible with OS/X finder. Consider using a different WebDAV client or webserver.');
-            }
-
-            // The body needs to stay intact, so we copy everything to a
-            // temporary stream.
-
-            $newBody = fopen('php://temp', 'r+');
-            fwrite($newBody, $firstByte);
-            stream_copy_to_stream($body, $newBody);
-            rewind($newBody);
-
-            $body = $newBody;
-        }
-
-        if ($this->server->tree->nodeExists($path)) {
-            $node = $this->server->tree->getNodeForPath($path);
-
-            // If the node is a collection, we'll deny it
-            if (!($node instanceof IFile)) {
-                throw new Exception\Conflict('PUT is not allowed on non-files.');
-            }
-            if (!$this->server->updateFile($path, $body, $etag)) {
-                return false;
-            }
-
-            $response->setHeader('Content-Length', '0');
-            if ($etag) {
-                $response->setHeader('ETag', $etag);
-            }
-            $response->setStatus(204);
-        } else {
-            $etag = null;
-            // If we got here, the resource didn't exist yet.
-            if (!$this->server->createFile($path, $body, $etag)) {
-                // For one reason or another the file was not created.
-                return false;
-            }
-
-            $response->setHeader('Content-Length', '0');
-            if ($etag) {
-                $response->setHeader('ETag', $etag);
-            }
-            $response->setStatus(201);
-        }
-
-        // Sending back false will interrupt the event chain and tell the server
-        // we've handled this method.
-        return false;
-    }
-
-    /**
-     * WebDAV MKCOL.
-     *
-     * The MKCOL method is used to create a new collection (directory) on the server
-     *
-     * @return bool
-     */
-    public function httpMkcol(RequestInterface $request, ResponseInterface $response)
-    {
-        $requestBody = $request->getBodyAsString();
-        $path = $request->getPath();
-
-        if ($requestBody) {
-            $contentType = $request->getHeader('Content-Type');
-            if (null === $contentType || (0 !== strpos($contentType, 'application/xml') && 0 !== strpos($contentType, 'text/xml'))) {
-                // We must throw 415 for unsupported mkcol bodies
-                throw new Exception\UnsupportedMediaType('The request body for the MKCOL request must have an xml Content-Type');
-            }
-
-            try {
-                $mkcol = $this->server->xml->expect('{DAV:}mkcol', $requestBody);
-            } catch (\Sabre\Xml\ParseException $e) {
-                throw new Exception\BadRequest($e->getMessage(), 0, $e);
-            }
-
-            $properties = $mkcol->getProperties();
-
-            if (!isset($properties['{DAV:}resourcetype'])) {
-                throw new Exception\BadRequest('The mkcol request must include a {DAV:}resourcetype property');
-            }
-            $resourceType = $properties['{DAV:}resourcetype']->getValue();
-            unset($properties['{DAV:}resourcetype']);
-        } else {
-            $properties = [];
-            $resourceType = ['{DAV:}collection'];
-        }
-
-        $mkcol = new MkCol($resourceType, $properties);
-
-        $result = $this->server->createCollection($path, $mkcol);
-
-        if (is_array($result)) {
-            $response->setStatus(207);
-            $response->setHeader('Content-Type', 'application/xml; charset=utf-8');
-
-            $response->setBody(
-                $this->server->generateMultiStatus([$result])
-            );
-        } else {
-            $response->setHeader('Content-Length', '0');
-            $response->setStatus(201);
-        }
-
-        // Sending back false will interrupt the event chain and tell the server
-        // we've handled this method.
-        return false;
-    }
-
-    /**
-     * WebDAV HTTP MOVE method.
-     *
-     * This method moves one uri to a different uri. A lot of the actual request processing is done in getCopyMoveInfo
-     *
-     * @return bool
-     */
-    public function httpMove(RequestInterface $request, ResponseInterface $response)
-    {
-        $path = $request->getPath();
-
-        $moveInfo = $this->server->getCopyAndMoveInfo($request);
-
-        if ($moveInfo['destinationExists']) {
-            if (!$this->server->emit('beforeUnbind', [$moveInfo['destination']])) {
-                return false;
-            }
-        }
-        if (!$this->server->emit('beforeUnbind', [$path])) {
-            return false;
-        }
-        if (!$this->server->emit('beforeBind', [$moveInfo['destination']])) {
-            return false;
-        }
-        if (!$this->server->emit('beforeMove', [$path, $moveInfo['destination']])) {
-            return false;
-        }
-
-        if ($moveInfo['destinationExists']) {
-            $this->server->tree->delete($moveInfo['destination']);
-            $this->server->emit('afterUnbind', [$moveInfo['destination']]);
-        }
-
-        $this->server->tree->move($path, $moveInfo['destination']);
-
-        // Its important afterMove is called before afterUnbind, because it
-        // allows systems to transfer data from one path to another.
-        // PropertyStorage uses this. If afterUnbind was first, it would clean
-        // up all the properties before it has a chance.
-        $this->server->emit('afterMove', [$path, $moveInfo['destination']]);
-        $this->server->emit('afterUnbind', [$path]);
-        $this->server->emit('afterBind', [$moveInfo['destination']]);
-
-        // If a resource was overwritten we should send a 204, otherwise a 201
-        $response->setHeader('Content-Length', '0');
-        $response->setStatus($moveInfo['destinationExists'] ? 204 : 201);
-
-        // Sending back false will interrupt the event chain and tell the server
-        // we've handled this method.
-        return false;
-    }
-
-    /**
-     * WebDAV HTTP COPY method.
-     *
-     * This method copies one uri to a different uri, and works much like the MOVE request
-     * A lot of the actual request processing is done in getCopyMoveInfo
-     *
-     * @return bool
-     */
-    public function httpCopy(RequestInterface $request, ResponseInterface $response)
-    {
-        $path = $request->getPath();
-
-        $copyInfo = $this->server->getCopyAndMoveInfo($request);
-
-        if (!$this->server->emit('beforeBind', [$copyInfo['destination']])) {
-            return false;
-        }
-        if (!$this->server->emit('beforeCopy', [$path, $copyInfo['destination']])) {
-            return false;
-        }
-
-        if ($copyInfo['destinationExists']) {
-            if (!$this->server->emit('beforeUnbind', [$copyInfo['destination']])) {
-                return false;
-            }
-            $this->server->tree->delete($copyInfo['destination']);
-        }
-
-        $this->server->tree->copy($path, $copyInfo['destination']);
-        $this->server->emit('afterCopy', [$path, $copyInfo['destination']]);
-        $this->server->emit('afterBind', [$copyInfo['destination']]);
-
-        // If a resource was overwritten we should send a 204, otherwise a 201
-        $response->setHeader('Content-Length', '0');
-        $response->setStatus($copyInfo['destinationExists'] ? 204 : 201);
-
-        // Sending back false will interrupt the event chain and tell the server
-        // we've handled this method.
-        return false;
-    }
-
-    /**
-     * HTTP REPORT method implementation.
-     *
-     * Although the REPORT method is not part of the standard WebDAV spec (it's from rfc3253)
-     * It's used in a lot of extensions, so it made sense to implement it into the core.
-     *
-     * @return bool
-     */
-    public function httpReport(RequestInterface $request, ResponseInterface $response)
-    {
-        $path = $request->getPath();
-
-        $result = $this->server->xml->parse(
-            $request->getBody(),
-            $request->getUrl(),
-            $rootElementName
-        );
-
-        if ($this->server->emit('report', [$rootElementName, $result, $path])) {
-            // If emit returned true, it means the report was not supported
-            throw new Exception\ReportNotSupported();
-        }
-
-        // Sending back false will interrupt the event chain and tell the server
-        // we've handled this method.
-        return false;
-    }
-
-    /**
-     * This method is called during property updates.
-     *
-     * Here we check if a user attempted to update a protected property and
-     * ensure that the process fails if this is the case.
-     *
-     * @param string $path
-     */
-    public function propPatchProtectedPropertyCheck($path, PropPatch $propPatch)
-    {
-        // Comparing the mutation list to the list of protected properties.
-        $mutations = $propPatch->getMutations();
-
-        $protected = array_intersect(
-            $this->server->protectedProperties,
-            array_keys($mutations)
-        );
-
-        if ($protected) {
-            $propPatch->setResultCode($protected, 403);
-        }
-    }
-
-    /**
-     * This method is called during property updates.
-     *
-     * Here we check if a node implements IProperties and let the node handle
-     * updating of (some) properties.
-     *
-     * @param string $path
-     */
-    public function propPatchNodeUpdate($path, PropPatch $propPatch)
-    {
-        // This should trigger a 404 if the node doesn't exist.
-        $node = $this->server->tree->getNodeForPath($path);
-
-        if ($node instanceof IProperties) {
-            $node->propPatch($propPatch);
-        }
-    }
-
-    /**
-     * This method is called when properties are retrieved.
-     *
-     * Here we add all the default properties.
-     */
-    public function propFind(PropFind $propFind, INode $node)
-    {
-        $propFind->handle('{DAV:}getlastmodified', function () use ($node) {
-            $lm = $node->getLastModified();
-            if ($lm) {
-                return new Xml\Property\GetLastModified($lm);
-            }
-        });
-
-        if ($node instanceof IFile) {
-            $propFind->handle('{DAV:}getcontentlength', [$node, 'getSize']);
-            $propFind->handle('{DAV:}getetag', [$node, 'getETag']);
-            $propFind->handle('{DAV:}getcontenttype', [$node, 'getContentType']);
-        }
-
-        if ($node instanceof IQuota) {
-            $quotaInfo = null;
-            $propFind->handle('{DAV:}quota-used-bytes', function () use (&$quotaInfo, $node) {
-                $quotaInfo = $node->getQuotaInfo();
-
-                return $quotaInfo[0];
-            });
-            $propFind->handle('{DAV:}quota-available-bytes', function () use (&$quotaInfo, $node) {
-                if (!$quotaInfo) {
-                    $quotaInfo = $node->getQuotaInfo();
-                }
-
-                return $quotaInfo[1];
-            });
-        }
-
-        $propFind->handle('{DAV:}supported-report-set', function () use ($propFind) {
-            $reports = [];
-            foreach ($this->server->getPlugins() as $plugin) {
-                $reports = array_merge($reports, $plugin->getSupportedReportSet($propFind->getPath()));
-            }
-
-            return new Xml\Property\SupportedReportSet($reports);
-        });
-        $propFind->handle('{DAV:}resourcetype', function () use ($node) {
-            return new Xml\Property\ResourceType($this->server->getResourceTypeForNode($node));
-        });
-        $propFind->handle('{DAV:}supported-method-set', function () use ($propFind) {
-            return new Xml\Property\SupportedMethodSet(
-                $this->server->getAllowedMethods($propFind->getPath())
-            );
-        });
-    }
-
-    /**
-     * Fetches properties for a node.
-     *
-     * This event is called a bit later, so plugins have a chance first to
-     * populate the result.
-     */
-    public function propFindNode(PropFind $propFind, INode $node)
-    {
-        if ($node instanceof IProperties && $propertyNames = $propFind->get404Properties()) {
-            $nodeProperties = $node->getProperties($propertyNames);
-            foreach ($nodeProperties as $propertyName => $propertyValue) {
-                $propFind->set($propertyName, $propertyValue, 200);
-            }
-        }
-    }
-
-    /**
-     * This method is called when properties are retrieved.
-     *
-     * This specific handler is called very late in the process, because we
-     * want other systems to first have a chance to handle the properties.
-     */
-    public function propFindLate(PropFind $propFind, INode $node)
-    {
-        $propFind->handle('{http://calendarserver.org/ns/}getctag', function () use ($propFind) {
-            // If we already have a sync-token from the current propFind
-            // request, we can re-use that.
-            $val = $propFind->get('{http://sabredav.org/ns}sync-token');
-            if ($val) {
-                return $val;
-            }
-
-            $val = $propFind->get('{DAV:}sync-token');
-            if ($val && is_scalar($val)) {
-                return $val;
-            }
-            if ($val && $val instanceof Xml\Property\Href) {
-                return substr($val->getHref(), strlen(Sync\Plugin::SYNCTOKEN_PREFIX));
-            }
-
-            // If we got here, the earlier two properties may simply not have
-            // been part of the earlier request. We're going to fetch them.
-            $result = $this->server->getProperties($propFind->getPath(), [
-                '{http://sabredav.org/ns}sync-token',
-                '{DAV:}sync-token',
-            ]);
-
-            if (isset($result['{http://sabredav.org/ns}sync-token'])) {
-                return $result['{http://sabredav.org/ns}sync-token'];
-            }
-            if (isset($result['{DAV:}sync-token'])) {
-                $val = $result['{DAV:}sync-token'];
-                if (is_scalar($val)) {
-                    return $val;
-                } elseif ($val instanceof Xml\Property\Href) {
-                    return substr($val->getHref(), strlen(Sync\Plugin::SYNCTOKEN_PREFIX));
-                }
-            }
-        });
-    }
-
-    /**
-     * Listens for exception events, and automatically logs them.
-     *
-     * @param Exception $e
-     */
-    public function exception($e)
-    {
-        $logLevel = \Psr\Log\LogLevel::CRITICAL;
-        if ($e instanceof \Sabre\DAV\Exception) {
-            // If it's a standard sabre/dav exception, it means we have a http
-            // status code available.
-            $code = $e->getHTTPCode();
-
-            if ($code >= 400 && $code < 500) {
-                // user error
-                $logLevel = \Psr\Log\LogLevel::INFO;
-            } else {
-                // Server-side error. We mark it's as an error, but it's not
-                // critical.
-                $logLevel = \Psr\Log\LogLevel::ERROR;
-            }
-        }
-
-        $this->server->getLogger()->log(
-            $logLevel,
-            'Uncaught exception',
-            [
-                'exception' => $e,
-            ]
-        );
-    }
-
-    /**
-     * 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' => 'The Core plugin provides a lot of the basic functionality required by WebDAV, such as a default implementation for all HTTP and WebDAV methods.',
-            'link' => null,
-        ];
-    }
+			// Only reading first byte
+			$firstByte = fread($body, 1);
+			if (1 !== strlen($firstByte)) {
+				throw new Exception\Forbidden('This server is not compatible with OS/X finder. Consider using a different WebDAV client or webserver.');
+			}
+
+			// The body needs to stay intact, so we copy everything to a
+			// temporary stream.
+
+			$newBody = fopen('php://temp', 'r+');
+			fwrite($newBody, $firstByte);
+			stream_copy_to_stream($body, $newBody);
+			rewind($newBody);
+
+			$body = $newBody;
+		}
+
+		if ($this->server->tree->nodeExists($path)) {
+			$node = $this->server->tree->getNodeForPath($path);
+
+			// If the node is a collection, we'll deny it
+			if (!($node instanceof IFile)) {
+				throw new Exception\Conflict('PUT is not allowed on non-files.');
+			}
+			if (!$this->server->updateFile($path, $body, $etag)) {
+				return false;
+			}
+
+			$response->setHeader('Content-Length', '0');
+			if ($etag) {
+				$response->setHeader('ETag', $etag);
+			}
+			$response->setStatus(204);
+		} else {
+			$etag = null;
+			// If we got here, the resource didn't exist yet.
+			if (!$this->server->createFile($path, $body, $etag)) {
+				// For one reason or another the file was not created.
+				return false;
+			}
+
+			$response->setHeader('Content-Length', '0');
+			if ($etag) {
+				$response->setHeader('ETag', $etag);
+			}
+			$response->setStatus(201);
+		}
+
+		// Sending back false will interrupt the event chain and tell the server
+		// we've handled this method.
+		return false;
+	}
+
+	/**
+	 * WebDAV MKCOL.
+	 *
+	 * The MKCOL method is used to create a new collection (directory) on the server
+	 *
+	 * @return bool
+	 */
+	public function httpMkcol(RequestInterface $request, ResponseInterface $response)
+	{
+		$requestBody = $request->getBodyAsString();
+		$path = $request->getPath();
+
+		if ($requestBody) {
+			$contentType = $request->getHeader('Content-Type');
+			if (null === $contentType || (0 !== strpos($contentType, 'application/xml') && 0 !== strpos($contentType, 'text/xml'))) {
+				// We must throw 415 for unsupported mkcol bodies
+				throw new Exception\UnsupportedMediaType('The request body for the MKCOL request must have an xml Content-Type');
+			}
+
+			try {
+				$mkcol = $this->server->xml->expect('{DAV:}mkcol', $requestBody);
+			} catch (\Sabre\Xml\ParseException $e) {
+				throw new Exception\BadRequest($e->getMessage(), 0, $e);
+			}
+
+			$properties = $mkcol->getProperties();
+
+			if (!isset($properties['{DAV:}resourcetype'])) {
+				throw new Exception\BadRequest('The mkcol request must include a {DAV:}resourcetype property');
+			}
+			$resourceType = $properties['{DAV:}resourcetype']->getValue();
+			unset($properties['{DAV:}resourcetype']);
+		} else {
+			$properties = [];
+			$resourceType = ['{DAV:}collection'];
+		}
+
+		$mkcol = new MkCol($resourceType, $properties);
+
+		$result = $this->server->createCollection($path, $mkcol);
+
+		if (is_array($result)) {
+			$response->setStatus(207);
+			$response->setHeader('Content-Type', 'application/xml; charset=utf-8');
+
+			$response->setBody(
+				$this->server->generateMultiStatus([$result])
+			);
+		} else {
+			$response->setHeader('Content-Length', '0');
+			$response->setStatus(201);
+		}
+
+		// Sending back false will interrupt the event chain and tell the server
+		// we've handled this method.
+		return false;
+	}
+
+	/**
+	 * WebDAV HTTP MOVE method.
+	 *
+	 * This method moves one uri to a different uri. A lot of the actual request processing is done in getCopyMoveInfo
+	 *
+	 * @return bool
+	 */
+	public function httpMove(RequestInterface $request, ResponseInterface $response)
+	{
+		$path = $request->getPath();
+
+		$moveInfo = $this->server->getCopyAndMoveInfo($request);
+
+		if ($moveInfo['destinationExists']) {
+			if (!$this->server->emit('beforeUnbind', [$moveInfo['destination']])) {
+				return false;
+			}
+		}
+		if (!$this->server->emit('beforeUnbind', [$path])) {
+			return false;
+		}
+		if (!$this->server->emit('beforeBind', [$moveInfo['destination']])) {
+			return false;
+		}
+		if (!$this->server->emit('beforeMove', [$path, $moveInfo['destination']])) {
+			return false;
+		}
+
+		if ($moveInfo['destinationExists']) {
+			$this->server->tree->delete($moveInfo['destination']);
+			$this->server->emit('afterUnbind', [$moveInfo['destination']]);
+		}
+
+		$this->server->tree->move($path, $moveInfo['destination']);
+
+		// Its important afterMove is called before afterUnbind, because it
+		// allows systems to transfer data from one path to another.
+		// PropertyStorage uses this. If afterUnbind was first, it would clean
+		// up all the properties before it has a chance.
+		$this->server->emit('afterMove', [$path, $moveInfo['destination']]);
+		$this->server->emit('afterUnbind', [$path]);
+		$this->server->emit('afterBind', [$moveInfo['destination']]);
+
+		// If a resource was overwritten we should send a 204, otherwise a 201
+		$response->setHeader('Content-Length', '0');
+		$response->setStatus($moveInfo['destinationExists'] ? 204 : 201);
+
+		// Sending back false will interrupt the event chain and tell the server
+		// we've handled this method.
+		return false;
+	}
+
+	/**
+	 * WebDAV HTTP COPY method.
+	 *
+	 * This method copies one uri to a different uri, and works much like the MOVE request
+	 * A lot of the actual request processing is done in getCopyMoveInfo
+	 *
+	 * @return bool
+	 */
+	public function httpCopy(RequestInterface $request, ResponseInterface $response)
+	{
+		$path = $request->getPath();
+
+		$copyInfo = $this->server->getCopyAndMoveInfo($request);
+
+		if (!$this->server->emit('beforeBind', [$copyInfo['destination']])) {
+			return false;
+		}
+		if (!$this->server->emit('beforeCopy', [$path, $copyInfo['destination']])) {
+			return false;
+		}
+
+		if ($copyInfo['destinationExists']) {
+			if (!$this->server->emit('beforeUnbind', [$copyInfo['destination']])) {
+				return false;
+			}
+			$this->server->tree->delete($copyInfo['destination']);
+		}
+
+		$this->server->tree->copy($path, $copyInfo['destination']);
+		$this->server->emit('afterCopy', [$path, $copyInfo['destination']]);
+		$this->server->emit('afterBind', [$copyInfo['destination']]);
+
+		// If a resource was overwritten we should send a 204, otherwise a 201
+		$response->setHeader('Content-Length', '0');
+		$response->setStatus($copyInfo['destinationExists'] ? 204 : 201);
+
+		// Sending back false will interrupt the event chain and tell the server
+		// we've handled this method.
+		return false;
+	}
+
+	/**
+	 * HTTP REPORT method implementation.
+	 *
+	 * Although the REPORT method is not part of the standard WebDAV spec (it's from rfc3253)
+	 * It's used in a lot of extensions, so it made sense to implement it into the core.
+	 *
+	 * @return bool
+	 */
+	public function httpReport(RequestInterface $request, ResponseInterface $response)
+	{
+		$path = $request->getPath();
+
+		$result = $this->server->xml->parse(
+			$request->getBody(),
+			$request->getUrl(),
+			$rootElementName
+		);
+
+		if ($this->server->emit('report', [$rootElementName, $result, $path])) {
+			// If emit returned true, it means the report was not supported
+			throw new Exception\ReportNotSupported();
+		}
+
+		// Sending back false will interrupt the event chain and tell the server
+		// we've handled this method.
+		return false;
+	}
+
+	/**
+	 * This method is called during property updates.
+	 *
+	 * Here we check if a user attempted to update a protected property and
+	 * ensure that the process fails if this is the case.
+	 *
+	 * @param string $path
+	 */
+	public function propPatchProtectedPropertyCheck($path, PropPatch $propPatch)
+	{
+		// Comparing the mutation list to the list of protected properties.
+		$mutations = $propPatch->getMutations();
+
+		$protected = array_intersect(
+			$this->server->protectedProperties,
+			array_keys($mutations)
+		);
+
+		if ($protected) {
+			$propPatch->setResultCode($protected, 403);
+		}
+	}
+
+	/**
+	 * This method is called during property updates.
+	 *
+	 * Here we check if a node implements IProperties and let the node handle
+	 * updating of (some) properties.
+	 *
+	 * @param string $path
+	 */
+	public function propPatchNodeUpdate($path, PropPatch $propPatch)
+	{
+		// This should trigger a 404 if the node doesn't exist.
+		$node = $this->server->tree->getNodeForPath($path);
+
+		if ($node instanceof IProperties) {
+			$node->propPatch($propPatch);
+		}
+	}
+
+	/**
+	 * This method is called when properties are retrieved.
+	 *
+	 * Here we add all the default properties.
+	 */
+	public function propFind(PropFind $propFind, INode $node)
+	{
+		$propFind->handle('{DAV:}getlastmodified', function () use ($node) {
+			$lm = $node->getLastModified();
+			if ($lm) {
+				return new Xml\Property\GetLastModified($lm);
+			}
+		});
+
+		if ($node instanceof IFile) {
+			$propFind->handle('{DAV:}getcontentlength', [$node, 'getSize']);
+			$propFind->handle('{DAV:}getetag', [$node, 'getETag']);
+			$propFind->handle('{DAV:}getcontenttype', [$node, 'getContentType']);
+		}
+
+		if ($node instanceof IQuota) {
+			$quotaInfo = null;
+			$propFind->handle('{DAV:}quota-used-bytes', function () use (&$quotaInfo, $node) {
+				$quotaInfo = $node->getQuotaInfo();
+
+				return $quotaInfo[0];
+			});
+			$propFind->handle('{DAV:}quota-available-bytes', function () use (&$quotaInfo, $node) {
+				if (!$quotaInfo) {
+					$quotaInfo = $node->getQuotaInfo();
+				}
+
+				return $quotaInfo[1];
+			});
+		}
+
+		$propFind->handle('{DAV:}supported-report-set', function () use ($propFind) {
+			$reports = [];
+			foreach ($this->server->getPlugins() as $plugin) {
+				$reports = array_merge($reports, $plugin->getSupportedReportSet($propFind->getPath()));
+			}
+
+			return new Xml\Property\SupportedReportSet($reports);
+		});
+		$propFind->handle('{DAV:}resourcetype', function () use ($node) {
+			return new Xml\Property\ResourceType($this->server->getResourceTypeForNode($node));
+		});
+		$propFind->handle('{DAV:}supported-method-set', function () use ($propFind) {
+			return new Xml\Property\SupportedMethodSet(
+				$this->server->getAllowedMethods($propFind->getPath())
+			);
+		});
+	}
+
+	/**
+	 * Fetches properties for a node.
+	 *
+	 * This event is called a bit later, so plugins have a chance first to
+	 * populate the result.
+	 */
+	public function propFindNode(PropFind $propFind, INode $node)
+	{
+		if ($node instanceof IProperties && $propertyNames = $propFind->get404Properties()) {
+			$nodeProperties = $node->getProperties($propertyNames);
+			foreach ($nodeProperties as $propertyName => $propertyValue) {
+				$propFind->set($propertyName, $propertyValue, 200);
+			}
+		}
+	}
+
+	/**
+	 * This method is called when properties are retrieved.
+	 *
+	 * This specific handler is called very late in the process, because we
+	 * want other systems to first have a chance to handle the properties.
+	 */
+	public function propFindLate(PropFind $propFind, INode $node)
+	{
+		$propFind->handle('{http://calendarserver.org/ns/}getctag', function () use ($propFind) {
+			// If we already have a sync-token from the current propFind
+			// request, we can re-use that.
+			$val = $propFind->get('{http://sabredav.org/ns}sync-token');
+			if ($val) {
+				return $val;
+			}
+
+			$val = $propFind->get('{DAV:}sync-token');
+			if ($val && is_scalar($val)) {
+				return $val;
+			}
+			if ($val && $val instanceof Xml\Property\Href) {
+				return substr($val->getHref(), strlen(Sync\Plugin::SYNCTOKEN_PREFIX));
+			}
+
+			// If we got here, the earlier two properties may simply not have
+			// been part of the earlier request. We're going to fetch them.
+			$result = $this->server->getProperties($propFind->getPath(), [
+				'{http://sabredav.org/ns}sync-token',
+				'{DAV:}sync-token',
+			]);
+
+			if (isset($result['{http://sabredav.org/ns}sync-token'])) {
+				return $result['{http://sabredav.org/ns}sync-token'];
+			}
+			if (isset($result['{DAV:}sync-token'])) {
+				$val = $result['{DAV:}sync-token'];
+				if (is_scalar($val)) {
+					return $val;
+				} elseif ($val instanceof Xml\Property\Href) {
+					return substr($val->getHref(), strlen(Sync\Plugin::SYNCTOKEN_PREFIX));
+				}
+			}
+		});
+	}
+
+	/**
+	 * Listens for exception events, and automatically logs them.
+	 *
+	 * @param Exception $e
+	 */
+	public function exception($e)
+	{
+		$logLevel = \Psr\Log\LogLevel::CRITICAL;
+		if ($e instanceof \Sabre\DAV\Exception) {
+			// If it's a standard sabre/dav exception, it means we have a http
+			// status code available.
+			$code = $e->getHTTPCode();
+
+			if ($code >= 400 && $code < 500) {
+				// user error
+				$logLevel = \Psr\Log\LogLevel::INFO;
+			} else {
+				// Server-side error. We mark it's as an error, but it's not
+				// critical.
+				$logLevel = \Psr\Log\LogLevel::ERROR;
+			}
+		}
+
+		$this->server->getLogger()->log(
+			$logLevel,
+			'Uncaught exception',
+			[
+				'exception' => $e,
+			]
+		);
+	}
+
+	/**
+	 * 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' => 'The Core plugin provides a lot of the basic functionality required by WebDAV, such as a default implementation for all HTTP and WebDAV methods.',
+			'link' => null,
+		];
+	}
 }

Spacing +7 added lines, -7 removed lines

@@ -745,7 +745,7 @@ 
      */
     public function propFind(PropFind $propFind, INode $node)
     {
-        $propFind->handle('{DAV:}getlastmodified', function () use ($node) {
+        $propFind->handle('{DAV:}getlastmodified', function() use ($node) {
             $lm = $node->getLastModified();
             if ($lm) {
                 return new Xml\Property\GetLastModified($lm);
@@ -760,12 +760,12 @@ 
 
         if ($node instanceof IQuota) {
             $quotaInfo = null;
-            $propFind->handle('{DAV:}quota-used-bytes', function () use (&$quotaInfo, $node) {
+            $propFind->handle('{DAV:}quota-used-bytes', function() use (&$quotaInfo, $node) {
                 $quotaInfo = $node->getQuotaInfo();
 
                 return $quotaInfo[0];
             });
-            $propFind->handle('{DAV:}quota-available-bytes', function () use (&$quotaInfo, $node) {
+            $propFind->handle('{DAV:}quota-available-bytes', function() use (&$quotaInfo, $node) {
                 if (!$quotaInfo) {
                     $quotaInfo = $node->getQuotaInfo();
                 }
@@ -774,7 +774,7 @@ 
             });
         }
 
-        $propFind->handle('{DAV:}supported-report-set', function () use ($propFind) {
+        $propFind->handle('{DAV:}supported-report-set', function() use ($propFind) {
             $reports = [];
             foreach ($this->server->getPlugins() as $plugin) {
                 $reports = array_merge($reports, $plugin->getSupportedReportSet($propFind->getPath()));
@@ -782,10 +782,10 @@ 
 
             return new Xml\Property\SupportedReportSet($reports);
         });
-        $propFind->handle('{DAV:}resourcetype', function () use ($node) {
+        $propFind->handle('{DAV:}resourcetype', function() use ($node) {
             return new Xml\Property\ResourceType($this->server->getResourceTypeForNode($node));
         });
-        $propFind->handle('{DAV:}supported-method-set', function () use ($propFind) {
+        $propFind->handle('{DAV:}supported-method-set', function() use ($propFind) {
             return new Xml\Property\SupportedMethodSet(
                 $this->server->getAllowedMethods($propFind->getPath())
             );
@@ -816,7 +816,7 @@ 
      */
     public function propFindLate(PropFind $propFind, INode $node)
     {
-        $propFind->handle('{http://calendarserver.org/ns/}getctag', function () use ($propFind) {
+        $propFind->handle('{http://calendarserver.org/ns/}getctag', function() use ($propFind) {
             // If we already have a sync-token from the current propFind
             // request, we can re-use that.
             $val = $propFind->get('{http://sabredav.org/ns}sync-token');

htdocs/includes/sabre/sabre/dav/lib/DAV/PropertyStorage/Plugin.php

Indentation +136 added lines, -136 removed lines

@@ -27,150 +27,150 @@
  */
 class Plugin extends ServerPlugin
 {
-    /**
-     * If you only want this plugin to store properties for a limited set of
-     * paths, you can use a pathFilter to do this.
-     *
-     * The pathFilter should be a callable. The callable retrieves a path as
-     * its argument, and should return true or false whether it allows
-     * properties to be stored.
-     *
-     * @var callable
-     */
-    public $pathFilter;
+	/**
+	 * If you only want this plugin to store properties for a limited set of
+	 * paths, you can use a pathFilter to do this.
+	 *
+	 * The pathFilter should be a callable. The callable retrieves a path as
+	 * its argument, and should return true or false whether it allows
+	 * properties to be stored.
+	 *
+	 * @var callable
+	 */
+	public $pathFilter;
 
-    /**
-     * @var Backend\BackendInterface
-     */
-    public $backend;
+	/**
+	 * @var Backend\BackendInterface
+	 */
+	public $backend;
 
-    /**
-     * Creates the plugin.
-     */
-    public function __construct(Backend\BackendInterface $backend)
-    {
-        $this->backend = $backend;
-    }
+	/**
+	 * Creates the plugin.
+	 */
+	public function __construct(Backend\BackendInterface $backend)
+	{
+		$this->backend = $backend;
+	}
 
-    /**
-     * 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.
-     */
-    public function initialize(Server $server)
-    {
-        $server->on('propFind', [$this, 'propFind'], 130);
-        $server->on('propPatch', [$this, 'propPatch'], 300);
-        $server->on('afterMove', [$this, 'afterMove']);
-        $server->on('afterUnbind', [$this, 'afterUnbind']);
-    }
+	/**
+	 * 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.
+	 */
+	public function initialize(Server $server)
+	{
+		$server->on('propFind', [$this, 'propFind'], 130);
+		$server->on('propPatch', [$this, 'propPatch'], 300);
+		$server->on('afterMove', [$this, 'afterMove']);
+		$server->on('afterUnbind', [$this, 'afterUnbind']);
+	}
 
-    /**
-     * Called during PROPFIND operations.
-     *
-     * If there's any requested properties that don't have a value yet, this
-     * plugin will look in the property storage backend to find them.
-     */
-    public function propFind(PropFind $propFind, INode $node)
-    {
-        $path = $propFind->getPath();
-        $pathFilter = $this->pathFilter;
-        if ($pathFilter && !$pathFilter($path)) {
-            return;
-        }
-        $this->backend->propFind($propFind->getPath(), $propFind);
-    }
+	/**
+	 * Called during PROPFIND operations.
+	 *
+	 * If there's any requested properties that don't have a value yet, this
+	 * plugin will look in the property storage backend to find them.
+	 */
+	public function propFind(PropFind $propFind, INode $node)
+	{
+		$path = $propFind->getPath();
+		$pathFilter = $this->pathFilter;
+		if ($pathFilter && !$pathFilter($path)) {
+			return;
+		}
+		$this->backend->propFind($propFind->getPath(), $propFind);
+	}
 
-    /**
-     * Called during PROPPATCH operations.
-     *
-     * If there's any updated properties that haven't been stored, the
-     * propertystorage backend can handle it.
-     *
-     * @param string $path
-     */
-    public function propPatch($path, PropPatch $propPatch)
-    {
-        $pathFilter = $this->pathFilter;
-        if ($pathFilter && !$pathFilter($path)) {
-            return;
-        }
-        $this->backend->propPatch($path, $propPatch);
-    }
+	/**
+	 * Called during PROPPATCH operations.
+	 *
+	 * If there's any updated properties that haven't been stored, the
+	 * propertystorage backend can handle it.
+	 *
+	 * @param string $path
+	 */
+	public function propPatch($path, PropPatch $propPatch)
+	{
+		$pathFilter = $this->pathFilter;
+		if ($pathFilter && !$pathFilter($path)) {
+			return;
+		}
+		$this->backend->propPatch($path, $propPatch);
+	}
 
-    /**
-     * Called after a node is deleted.
-     *
-     * This allows the backend to clean up any properties still in the
-     * database.
-     *
-     * @param string $path
-     */
-    public function afterUnbind($path)
-    {
-        $pathFilter = $this->pathFilter;
-        if ($pathFilter && !$pathFilter($path)) {
-            return;
-        }
-        $this->backend->delete($path);
-    }
+	/**
+	 * Called after a node is deleted.
+	 *
+	 * This allows the backend to clean up any properties still in the
+	 * database.
+	 *
+	 * @param string $path
+	 */
+	public function afterUnbind($path)
+	{
+		$pathFilter = $this->pathFilter;
+		if ($pathFilter && !$pathFilter($path)) {
+			return;
+		}
+		$this->backend->delete($path);
+	}
 
-    /**
-     * Called after a node is moved.
-     *
-     * This allows the backend to move all the associated properties.
-     *
-     * @param string $source
-     * @param string $destination
-     */
-    public function afterMove($source, $destination)
-    {
-        $pathFilter = $this->pathFilter;
-        if ($pathFilter && !$pathFilter($source)) {
-            return;
-        }
-        // If the destination is filtered, afterUnbind will handle cleaning up
-        // the properties.
-        if ($pathFilter && !$pathFilter($destination)) {
-            return;
-        }
+	/**
+	 * Called after a node is moved.
+	 *
+	 * This allows the backend to move all the associated properties.
+	 *
+	 * @param string $source
+	 * @param string $destination
+	 */
+	public function afterMove($source, $destination)
+	{
+		$pathFilter = $this->pathFilter;
+		if ($pathFilter && !$pathFilter($source)) {
+			return;
+		}
+		// If the destination is filtered, afterUnbind will handle cleaning up
+		// the properties.
+		if ($pathFilter && !$pathFilter($destination)) {
+			return;
+		}
 
-        $this->backend->move($source, $destination);
-    }
+		$this->backend->move($source, $destination);
+	}
 
-    /**
-     * 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 'property-storage';
-    }
+	/**
+	 * 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 'property-storage';
+	}
 
-    /**
-     * 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' => 'This plugin allows any arbitrary WebDAV property to be set on any resource.',
-            'link' => 'http://sabre.io/dav/property-storage/',
-        ];
-    }
+	/**
+	 * 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' => 'This plugin allows any arbitrary WebDAV property to be set on any resource.',
+			'link' => 'http://sabre.io/dav/property-storage/',
+		];
+	}
 }

htdocs/includes/sabre/sabre/dav/lib/DAV/PropertyStorage/Backend/BackendInterface.php

Indentation +50 added lines, -50 removed lines

@@ -19,57 +19,57 @@
  */
 interface BackendInterface
 {
-    /**
-     * Fetches properties for a path.
-     *
-     * This method received a PropFind object, which contains all the
-     * information about the properties that need to be fetched.
-     *
-     * Usually you would just want to call 'get404Properties' on this object,
-     * as this will give you the _exact_ list of properties that need to be
-     * fetched, and haven't yet.
-     *
-     * However, you can also support the 'allprops' property here. In that
-     * case, you should check for $propFind->isAllProps().
-     *
-     * @param string $path
-     */
-    public function propFind($path, PropFind $propFind);
+	/**
+	 * Fetches properties for a path.
+	 *
+	 * This method received a PropFind object, which contains all the
+	 * information about the properties that need to be fetched.
+	 *
+	 * Usually you would just want to call 'get404Properties' on this object,
+	 * as this will give you the _exact_ list of properties that need to be
+	 * fetched, and haven't yet.
+	 *
+	 * However, you can also support the 'allprops' property here. In that
+	 * case, you should check for $propFind->isAllProps().
+	 *
+	 * @param string $path
+	 */
+	public function propFind($path, PropFind $propFind);
 
-    /**
-     * Updates properties for a path.
-     *
-     * This method received a PropPatch object, which contains all the
-     * information about the update.
-     *
-     * Usually you would want to call 'handleRemaining' on this object, to get;
-     * a list of all properties that need to be stored.
-     *
-     * @param string $path
-     */
-    public function propPatch($path, PropPatch $propPatch);
+	/**
+	 * Updates properties for a path.
+	 *
+	 * This method received a PropPatch object, which contains all the
+	 * information about the update.
+	 *
+	 * Usually you would want to call 'handleRemaining' on this object, to get;
+	 * a list of all properties that need to be stored.
+	 *
+	 * @param string $path
+	 */
+	public function propPatch($path, PropPatch $propPatch);
 
-    /**
-     * This method is called after a node is deleted.
-     *
-     * This allows a backend to clean up all associated properties.
-     *
-     * The delete method will get called once for the deletion of an entire
-     * tree.
-     *
-     * @param string $path
-     */
-    public function delete($path);
+	/**
+	 * This method is called after a node is deleted.
+	 *
+	 * This allows a backend to clean up all associated properties.
+	 *
+	 * The delete method will get called once for the deletion of an entire
+	 * tree.
+	 *
+	 * @param string $path
+	 */
+	public function delete($path);
 
-    /**
-     * This method is called after a successful MOVE.
-     *
-     * This should be used to migrate all properties from one path to another.
-     * Note that entire collections may be moved, so ensure that all properties
-     * for children are also moved along.
-     *
-     * @param string $source
-     * @param string $destination
-     */
-    public function move($source, $destination);
+	/**
+	 * This method is called after a successful MOVE.
+	 *
+	 * This should be used to migrate all properties from one path to another.
+	 * Note that entire collections may be moved, so ensure that all properties
+	 * for children are also moved along.
+	 *
+	 * @param string $source
+	 * @param string $destination
+	 */
+	public function move($source, $destination);
 }

htdocs/includes/sabre/sabre/dav/lib/DAV/PropertyStorage/Backend/PDO.php

Indentation +191 added lines, -191 removed lines

@@ -22,203 +22,203 @@
  */
 class PDO implements BackendInterface
 {
-    /**
-     * Value is stored as string.
-     */
-    const VT_STRING = 1;
-
-    /**
-     * Value is stored as XML fragment.
-     */
-    const VT_XML = 2;
-
-    /**
-     * Value is stored as a property object.
-     */
-    const VT_OBJECT = 3;
-
-    /**
-     * PDO.
-     *
-     * @var \PDO
-     */
-    protected $pdo;
-
-    /**
-     * PDO table name we'll be using.
-     *
-     * @var string
-     */
-    public $tableName = 'propertystorage';
-
-    /**
-     * Creates the PDO property storage engine.
-     */
-    public function __construct(\PDO $pdo)
-    {
-        $this->pdo = $pdo;
-    }
-
-    /**
-     * Fetches properties for a path.
-     *
-     * This method received a PropFind object, which contains all the
-     * information about the properties that need to be fetched.
-     *
-     * Usually you would just want to call 'get404Properties' on this object,
-     * as this will give you the _exact_ list of properties that need to be
-     * fetched, and haven't yet.
-     *
-     * However, you can also support the 'allprops' property here. In that
-     * case, you should check for $propFind->isAllProps().
-     *
-     * @param string $path
-     */
-    public function propFind($path, PropFind $propFind)
-    {
-        if (!$propFind->isAllProps() && 0 === count($propFind->get404Properties())) {
-            return;
-        }
-
-        $query = 'SELECT name, value, valuetype FROM '.$this->tableName.' WHERE path = ?';
-        $stmt = $this->pdo->prepare($query);
-        $stmt->execute([$path]);
-
-        while ($row = $stmt->fetch(\PDO::FETCH_ASSOC)) {
-            if ('resource' === gettype($row['value'])) {
-                $row['value'] = stream_get_contents($row['value']);
-            }
-            switch ($row['valuetype']) {
-                case null:
-                case self::VT_STRING:
-                    $propFind->set($row['name'], $row['value']);
-                    break;
-                case self::VT_XML:
-                    $propFind->set($row['name'], new Complex($row['value']));
-                    break;
-                case self::VT_OBJECT:
-                    $propFind->set($row['name'], unserialize($row['value']));
-                    break;
-            }
-        }
-    }
-
-    /**
-     * Updates properties for a path.
-     *
-     * This method received a PropPatch object, which contains all the
-     * information about the update.
-     *
-     * Usually you would want to call 'handleRemaining' on this object, to get;
-     * a list of all properties that need to be stored.
-     *
-     * @param string $path
-     */
-    public function propPatch($path, PropPatch $propPatch)
-    {
-        $propPatch->handleRemaining(function ($properties) use ($path) {
-            if ('pgsql' === $this->pdo->getAttribute(\PDO::ATTR_DRIVER_NAME)) {
-                $updateSql = <<<SQL
+	/**
+	 * Value is stored as string.
+	 */
+	const VT_STRING = 1;
+
+	/**
+	 * Value is stored as XML fragment.
+	 */
+	const VT_XML = 2;
+
+	/**
+	 * Value is stored as a property object.
+	 */
+	const VT_OBJECT = 3;
+
+	/**
+	 * PDO.
+	 *
+	 * @var \PDO
+	 */
+	protected $pdo;
+
+	/**
+	 * PDO table name we'll be using.
+	 *
+	 * @var string
+	 */
+	public $tableName = 'propertystorage';
+
+	/**
+	 * Creates the PDO property storage engine.
+	 */
+	public function __construct(\PDO $pdo)
+	{
+		$this->pdo = $pdo;
+	}
+
+	/**
+	 * Fetches properties for a path.
+	 *
+	 * This method received a PropFind object, which contains all the
+	 * information about the properties that need to be fetched.
+	 *
+	 * Usually you would just want to call 'get404Properties' on this object,
+	 * as this will give you the _exact_ list of properties that need to be
+	 * fetched, and haven't yet.
+	 *
+	 * However, you can also support the 'allprops' property here. In that
+	 * case, you should check for $propFind->isAllProps().
+	 *
+	 * @param string $path
+	 */
+	public function propFind($path, PropFind $propFind)
+	{
+		if (!$propFind->isAllProps() && 0 === count($propFind->get404Properties())) {
+			return;
+		}
+
+		$query = 'SELECT name, value, valuetype FROM '.$this->tableName.' WHERE path = ?';
+		$stmt = $this->pdo->prepare($query);
+		$stmt->execute([$path]);
+
+		while ($row = $stmt->fetch(\PDO::FETCH_ASSOC)) {
+			if ('resource' === gettype($row['value'])) {
+				$row['value'] = stream_get_contents($row['value']);
+			}
+			switch ($row['valuetype']) {
+				case null:
+				case self::VT_STRING:
+					$propFind->set($row['name'], $row['value']);
+					break;
+				case self::VT_XML:
+					$propFind->set($row['name'], new Complex($row['value']));
+					break;
+				case self::VT_OBJECT:
+					$propFind->set($row['name'], unserialize($row['value']));
+					break;
+			}
+		}
+	}
+
+	/**
+	 * Updates properties for a path.
+	 *
+	 * This method received a PropPatch object, which contains all the
+	 * information about the update.
+	 *
+	 * Usually you would want to call 'handleRemaining' on this object, to get;
+	 * a list of all properties that need to be stored.
+	 *
+	 * @param string $path
+	 */
+	public function propPatch($path, PropPatch $propPatch)
+	{
+		$propPatch->handleRemaining(function ($properties) use ($path) {
+			if ('pgsql' === $this->pdo->getAttribute(\PDO::ATTR_DRIVER_NAME)) {
+				$updateSql = <<<SQL
 INSERT INTO {$this->tableName} (path, name, valuetype, value)
 VALUES (:path, :name, :valuetype, :value)
 ON CONFLICT (path, name)
 DO UPDATE SET valuetype = :valuetype, value = :value
 SQL;
-            } else {
-                $updateSql = <<<SQL
+			} else {
+				$updateSql = <<<SQL
 REPLACE INTO {$this->tableName} (path, name, valuetype, value)
 VALUES (:path, :name, :valuetype, :value)
 SQL;
-            }
-
-            $updateStmt = $this->pdo->prepare($updateSql);
-            $deleteStmt = $this->pdo->prepare('DELETE FROM '.$this->tableName.' WHERE path = ? AND name = ?');
-
-            foreach ($properties as $name => $value) {
-                if (!is_null($value)) {
-                    if (is_scalar($value)) {
-                        $valueType = self::VT_STRING;
-                    } elseif ($value instanceof Complex) {
-                        $valueType = self::VT_XML;
-                        $value = $value->getXml();
-                    } else {
-                        $valueType = self::VT_OBJECT;
-                        $value = serialize($value);
-                    }
-
-                    $updateStmt->bindParam('path', $path, \PDO::PARAM_STR);
-                    $updateStmt->bindParam('name', $name, \PDO::PARAM_STR);
-                    $updateStmt->bindParam('valuetype', $valueType, \PDO::PARAM_INT);
-                    $updateStmt->bindParam('value', $value, \PDO::PARAM_LOB);
-
-                    $updateStmt->execute();
-                } else {
-                    $deleteStmt->execute([$path, $name]);
-                }
-            }
-
-            return true;
-        });
-    }
-
-    /**
-     * This method is called after a node is deleted.
-     *
-     * This allows a backend to clean up all associated properties.
-     *
-     * The delete method will get called once for the deletion of an entire
-     * tree.
-     *
-     * @param string $path
-     */
-    public function delete($path)
-    {
-        $stmt = $this->pdo->prepare('DELETE FROM '.$this->tableName."  WHERE path = ? OR path LIKE ? ESCAPE '='");
-        $childPath = strtr(
-            $path,
-            [
-                '=' => '==',
-                '%' => '=%',
-                '_' => '=_',
-            ]
-        ).'/%';
-
-        $stmt->execute([$path, $childPath]);
-    }
-
-    /**
-     * This method is called after a successful MOVE.
-     *
-     * This should be used to migrate all properties from one path to another.
-     * Note that entire collections may be moved, so ensure that all properties
-     * for children are also moved along.
-     *
-     * @param string $source
-     * @param string $destination
-     */
-    public function move($source, $destination)
-    {
-        // I don't know a way to write this all in a single sql query that's
-        // also compatible across db engines, so we're letting PHP do all the
-        // updates. Much slower, but it should still be pretty fast in most
-        // cases.
-        $select = $this->pdo->prepare('SELECT id, path FROM '.$this->tableName.'  WHERE path = ? OR path LIKE ?');
-        $select->execute([$source, $source.'/%']);
-
-        $update = $this->pdo->prepare('UPDATE '.$this->tableName.' SET path = ? WHERE id = ?');
-        while ($row = $select->fetch(\PDO::FETCH_ASSOC)) {
-            // Sanity check. SQL may select too many records, such as records
-            // with different cases.
-            if ($row['path'] !== $source && 0 !== strpos($row['path'], $source.'/')) {
-                continue;
-            }
-
-            $trailingPart = substr($row['path'], strlen($source) + 1);
-            $newPath = $destination;
-            if ($trailingPart) {
-                $newPath .= '/'.$trailingPart;
-            }
-            $update->execute([$newPath, $row['id']]);
-        }
-    }
+			}
+
+			$updateStmt = $this->pdo->prepare($updateSql);
+			$deleteStmt = $this->pdo->prepare('DELETE FROM '.$this->tableName.' WHERE path = ? AND name = ?');
+
+			foreach ($properties as $name => $value) {
+				if (!is_null($value)) {
+					if (is_scalar($value)) {
+						$valueType = self::VT_STRING;
+					} elseif ($value instanceof Complex) {
+						$valueType = self::VT_XML;
+						$value = $value->getXml();
+					} else {
+						$valueType = self::VT_OBJECT;
+						$value = serialize($value);
+					}
+
+					$updateStmt->bindParam('path', $path, \PDO::PARAM_STR);
+					$updateStmt->bindParam('name', $name, \PDO::PARAM_STR);
+					$updateStmt->bindParam('valuetype', $valueType, \PDO::PARAM_INT);
+					$updateStmt->bindParam('value', $value, \PDO::PARAM_LOB);
+
+					$updateStmt->execute();
+				} else {
+					$deleteStmt->execute([$path, $name]);
+				}
+			}
+
+			return true;
+		});
+	}
+
+	/**
+	 * This method is called after a node is deleted.
+	 *
+	 * This allows a backend to clean up all associated properties.
+	 *
+	 * The delete method will get called once for the deletion of an entire
+	 * tree.
+	 *
+	 * @param string $path
+	 */
+	public function delete($path)
+	{
+		$stmt = $this->pdo->prepare('DELETE FROM '.$this->tableName."  WHERE path = ? OR path LIKE ? ESCAPE '='");
+		$childPath = strtr(
+			$path,
+			[
+				'=' => '==',
+				'%' => '=%',
+				'_' => '=_',
+			]
+		).'/%';
+
+		$stmt->execute([$path, $childPath]);
+	}
+
+	/**
+	 * This method is called after a successful MOVE.
+	 *
+	 * This should be used to migrate all properties from one path to another.
+	 * Note that entire collections may be moved, so ensure that all properties
+	 * for children are also moved along.
+	 *
+	 * @param string $source
+	 * @param string $destination
+	 */
+	public function move($source, $destination)
+	{
+		// I don't know a way to write this all in a single sql query that's
+		// also compatible across db engines, so we're letting PHP do all the
+		// updates. Much slower, but it should still be pretty fast in most
+		// cases.
+		$select = $this->pdo->prepare('SELECT id, path FROM '.$this->tableName.'  WHERE path = ? OR path LIKE ?');
+		$select->execute([$source, $source.'/%']);
+
+		$update = $this->pdo->prepare('UPDATE '.$this->tableName.' SET path = ? WHERE id = ?');
+		while ($row = $select->fetch(\PDO::FETCH_ASSOC)) {
+			// Sanity check. SQL may select too many records, such as records
+			// with different cases.
+			if ($row['path'] !== $source && 0 !== strpos($row['path'], $source.'/')) {
+				continue;
+			}
+
+			$trailingPart = substr($row['path'], strlen($source) + 1);
+			$newPath = $destination;
+			if ($trailingPart) {
+				$newPath .= '/'.$trailingPart;
+			}
+			$update->execute([$newPath, $row['id']]);
+		}
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -116,7 +116,7 @@
      */
     public function propPatch($path, PropPatch $propPatch)
     {
-        $propPatch->handleRemaining(function ($properties) use ($path) {
+        $propPatch->handleRemaining(function($properties) use ($path) {
             if ('pgsql' === $this->pdo->getAttribute(\PDO::ATTR_DRIVER_NAME)) {
                 $updateSql = <<<SQL
 INSERT INTO {$this->tableName} (path, name, valuetype, value)

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -123,12 +123,12 @@
 VALUES (:path, :name, :valuetype, :value)
 ON CONFLICT (path, name)
 DO UPDATE SET valuetype = :valuetype, value = :value
-SQL;
+sql;
             } else {
                 $updateSql = <<<SQL
 REPLACE INTO {$this->tableName} (path, name, valuetype, value)
 VALUES (:path, :name, :valuetype, :value)
-SQL;
+sql;
             }
 
             $updateStmt = $this->pdo->prepare($updateSql);

htdocs/includes/sabre/sabre/dav/lib/DAV/ServerPlugin.php

Indentation +82 added lines, -82 removed lines

@@ -15,91 +15,91 @@
  */
 abstract class ServerPlugin
 {
-    /**
-     * 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.
-     */
-    abstract public function initialize(Server $server);
+	/**
+	 * 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.
+	 */
+	abstract public function initialize(Server $server);
 
-    /**
-     * This method should return a list of server-features.
-     *
-     * This is for example 'versioning' and is added to the DAV: header
-     * in an OPTIONS response.
-     *
-     * @return array
-     */
-    public function getFeatures()
-    {
-        return [];
-    }
+	/**
+	 * This method should return a list of server-features.
+	 *
+	 * This is for example 'versioning' and is added to the DAV: header
+	 * in an OPTIONS response.
+	 *
+	 * @return array
+	 */
+	public function getFeatures()
+	{
+		return [];
+	}
 
-    /**
-     * 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 $path
-     *
-     * @return array
-     */
-    public function getHTTPMethods($path)
-    {
-        return [];
-    }
+	/**
+	 * 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 $path
+	 *
+	 * @return array
+	 */
+	public function getHTTPMethods($path)
+	{
+		return [];
+	}
 
-    /**
-     * 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 get_class($this);
-    }
+	/**
+	 * 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 get_class($this);
+	}
 
-    /**
-     * 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)
-    {
-        return [];
-    }
+	/**
+	 * 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)
+	{
+		return [];
+	}
 
-    /**
-     * 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' => null,
-            'link' => null,
-        ];
-    }
+	/**
+	 * 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' => null,
+			'link' => null,
+		];
+	}
 }

htdocs/includes/sabre/sabre/dav/lib/DAV/IExtendedCollection.php

Indentation +24 added lines, -24 removed lines

@@ -16,28 +16,28 @@
  */
 interface IExtendedCollection extends ICollection
 {
-    /**
-     * Creates a new collection.
-     *
-     * This method will receive a MkCol object with all the information about
-     * the new collection that's being created.
-     *
-     * The MkCol object contains information about the resourceType of the new
-     * collection. If you don't support the specified resourceType, you should
-     * throw Exception\InvalidResourceType.
-     *
-     * The object also contains a list of WebDAV properties for the new
-     * collection.
-     *
-     * You should call the handle() method on this object to specify exactly
-     * which properties you are storing. This allows the system to figure out
-     * exactly which properties you didn't store, which in turn allows other
-     * plugins (such as the propertystorage plugin) to handle storing the
-     * property for you.
-     *
-     * @param string $name
-     *
-     * @throws Exception\InvalidResourceType
-     */
-    public function createExtendedCollection($name, MkCol $mkCol);
+	/**
+	 * Creates a new collection.
+	 *
+	 * This method will receive a MkCol object with all the information about
+	 * the new collection that's being created.
+	 *
+	 * The MkCol object contains information about the resourceType of the new
+	 * collection. If you don't support the specified resourceType, you should
+	 * throw Exception\InvalidResourceType.
+	 *
+	 * The object also contains a list of WebDAV properties for the new
+	 * collection.
+	 *
+	 * You should call the handle() method on this object to specify exactly
+	 * which properties you are storing. This allows the system to figure out
+	 * exactly which properties you didn't store, which in turn allows other
+	 * plugins (such as the propertystorage plugin) to handle storing the
+	 * property for you.
+	 *
+	 * @param string $name
+	 *
+	 * @throws Exception\InvalidResourceType
+	 */
+	public function createExtendedCollection($name, MkCol $mkCol);
 }

htdocs/includes/sabre/sabre/dav/lib/DAV/SimpleFile.php

Indentation +90 added lines, -90 removed lines

@@ -17,102 +17,102 @@
  */
 class SimpleFile extends File
 {
-    /**
-     * File contents.
-     *
-     * @var string
-     */
-    protected $contents = [];
+	/**
+	 * File contents.
+	 *
+	 * @var string
+	 */
+	protected $contents = [];
 
-    /**
-     * Name of this resource.
-     *
-     * @var string
-     */
-    protected $name;
+	/**
+	 * Name of this resource.
+	 *
+	 * @var string
+	 */
+	protected $name;
 
-    /**
-     * A mimetype, such as 'text/plain' or 'text/html'.
-     *
-     * @var string
-     */
-    protected $mimeType;
+	/**
+	 * A mimetype, such as 'text/plain' or 'text/html'.
+	 *
+	 * @var string
+	 */
+	protected $mimeType;
 
-    /**
-     * Creates this node.
-     *
-     * The name of the node must be passed, as well as the contents of the
-     * file.
-     *
-     * @param string      $name
-     * @param string      $contents
-     * @param string|null $mimeType
-     */
-    public function __construct($name, $contents, $mimeType = null)
-    {
-        $this->name = $name;
-        $this->contents = $contents;
-        $this->mimeType = $mimeType;
-    }
+	/**
+	 * Creates this node.
+	 *
+	 * The name of the node must be passed, as well as the contents of the
+	 * file.
+	 *
+	 * @param string      $name
+	 * @param string      $contents
+	 * @param string|null $mimeType
+	 */
+	public function __construct($name, $contents, $mimeType = null)
+	{
+		$this->name = $name;
+		$this->contents = $contents;
+		$this->mimeType = $mimeType;
+	}
 
-    /**
-     * Returns the node name for this file.
-     *
-     * This name is used to construct the url.
-     *
-     * @return string
-     */
-    public function getName()
-    {
-        return $this->name;
-    }
+	/**
+	 * Returns the node name for this file.
+	 *
+	 * This name is used to construct the url.
+	 *
+	 * @return string
+	 */
+	public function getName()
+	{
+		return $this->name;
+	}
 
-    /**
-     * Returns the data.
-     *
-     * This method may either return a string or a readable stream resource
-     *
-     * @return mixed
-     */
-    public function get()
-    {
-        return $this->contents;
-    }
+	/**
+	 * Returns the data.
+	 *
+	 * This method may either return a string or a readable stream resource
+	 *
+	 * @return mixed
+	 */
+	public function get()
+	{
+		return $this->contents;
+	}
 
-    /**
-     * Returns the size of the file, in bytes.
-     *
-     * @return int
-     */
-    public function getSize()
-    {
-        return strlen($this->contents);
-    }
+	/**
+	 * Returns the size of the file, in bytes.
+	 *
+	 * @return int
+	 */
+	public function getSize()
+	{
+		return strlen($this->contents);
+	}
 
-    /**
-     * Returns the ETag for a file.
-     *
-     * An ETag is a unique identifier representing the current version of the file. If the file changes, the ETag MUST change.
-     * The ETag is an arbitrary string, but MUST be surrounded by double-quotes.
-     *
-     * Return null if the ETag can not effectively be determined
-     *
-     * @return string
-     */
-    public function getETag()
-    {
-        return '"'.sha1($this->contents).'"';
-    }
+	/**
+	 * Returns the ETag for a file.
+	 *
+	 * An ETag is a unique identifier representing the current version of the file. If the file changes, the ETag MUST change.
+	 * The ETag is an arbitrary string, but MUST be surrounded by double-quotes.
+	 *
+	 * Return null if the ETag can not effectively be determined
+	 *
+	 * @return string
+	 */
+	public function getETag()
+	{
+		return '"'.sha1($this->contents).'"';
+	}
 
-    /**
-     * Returns the mime-type for a file.
-     *
-     * If null is returned, we'll assume application/octet-stream
-     *
-     * @return string
-     */
-    public function getContentType()
-    {
-        return $this->mimeType;
-    }
+	/**
+	 * Returns the mime-type for a file.
+	 *
+	 * If null is returned, we'll assume application/octet-stream
+	 *
+	 * @return string
+	 */
+	public function getContentType()
+	{
+		return $this->mimeType;
+	}
 }

htdocs/includes/sabre/sabre/dav/lib/DAV/IQuota.php

Indentation +7 added lines, -7 removed lines

@@ -17,11 +17,11 @@
  */
 interface IQuota extends ICollection
 {
-    /**
-     * Returns the quota information.
-     *
-     * This method MUST return an array with 2 values, the first being the total used space,
-     * the second the available space (in bytes)
-     */
-    public function getQuotaInfo();
+	/**
+	 * Returns the quota information.
+	 *
+	 * This method MUST return an array with 2 values, the first being the total used space,
+	 * the second the available space (in bytes)
+	 */
+	public function getQuotaInfo();
 }

htdocs/includes/sabre/sabre/dav/lib/DAV/FS/Directory.php

Indentation +119 added lines, -119 removed lines

@@ -15,133 +15,133 @@
  */
 class Directory extends Node implements DAV\ICollection, DAV\IQuota
 {
-    /**
-     * Creates a new file in the directory.
-     *
-     * Data will either be supplied as a stream resource, or in certain cases
-     * as a string. Keep in mind that you may have to support either.
-     *
-     * After successful creation of the file, you may choose to return the ETag
-     * of the new file here.
-     *
-     * The returned ETag must be surrounded by double-quotes (The quotes should
-     * be part of the actual string).
-     *
-     * If you cannot accurately determine the ETag, you should not return it.
-     * If you don't store the file exactly as-is (you're transforming it
-     * somehow) you should also not return an ETag.
-     *
-     * This means that if a subsequent GET to this new file does not exactly
-     * return the same contents of what was submitted here, you are strongly
-     * recommended to omit the ETag.
-     *
-     * @param string          $name Name of the file
-     * @param resource|string $data Initial payload
-     *
-     * @return string|null
-     */
-    public function createFile($name, $data = null)
-    {
-        $newPath = $this->path.'/'.$name;
-        file_put_contents($newPath, $data);
-        clearstatcache(true, $newPath);
-    }
+	/**
+	 * Creates a new file in the directory.
+	 *
+	 * Data will either be supplied as a stream resource, or in certain cases
+	 * as a string. Keep in mind that you may have to support either.
+	 *
+	 * After successful creation of the file, you may choose to return the ETag
+	 * of the new file here.
+	 *
+	 * The returned ETag must be surrounded by double-quotes (The quotes should
+	 * be part of the actual string).
+	 *
+	 * If you cannot accurately determine the ETag, you should not return it.
+	 * If you don't store the file exactly as-is (you're transforming it
+	 * somehow) you should also not return an ETag.
+	 *
+	 * This means that if a subsequent GET to this new file does not exactly
+	 * return the same contents of what was submitted here, you are strongly
+	 * recommended to omit the ETag.
+	 *
+	 * @param string          $name Name of the file
+	 * @param resource|string $data Initial payload
+	 *
+	 * @return string|null
+	 */
+	public function createFile($name, $data = null)
+	{
+		$newPath = $this->path.'/'.$name;
+		file_put_contents($newPath, $data);
+		clearstatcache(true, $newPath);
+	}
 
-    /**
-     * Creates a new subdirectory.
-     *
-     * @param string $name
-     */
-    public function createDirectory($name)
-    {
-        $newPath = $this->path.'/'.$name;
-        mkdir($newPath);
-        clearstatcache(true, $newPath);
-    }
+	/**
+	 * Creates a new subdirectory.
+	 *
+	 * @param string $name
+	 */
+	public function createDirectory($name)
+	{
+		$newPath = $this->path.'/'.$name;
+		mkdir($newPath);
+		clearstatcache(true, $newPath);
+	}
 
-    /**
-     * Returns a specific child node, referenced by its name.
-     *
-     * This method must throw DAV\Exception\NotFound if the node does not
-     * exist.
-     *
-     * @param string $name
-     *
-     * @throws DAV\Exception\NotFound
-     *
-     * @return DAV\INode
-     */
-    public function getChild($name)
-    {
-        $path = $this->path.'/'.$name;
+	/**
+	 * Returns a specific child node, referenced by its name.
+	 *
+	 * This method must throw DAV\Exception\NotFound if the node does not
+	 * exist.
+	 *
+	 * @param string $name
+	 *
+	 * @throws DAV\Exception\NotFound
+	 *
+	 * @return DAV\INode
+	 */
+	public function getChild($name)
+	{
+		$path = $this->path.'/'.$name;
 
-        if (!file_exists($path)) {
-            throw new DAV\Exception\NotFound('File with name '.$path.' could not be located');
-        }
-        if (is_dir($path)) {
-            return new self($path);
-        } else {
-            return new File($path);
-        }
-    }
+		if (!file_exists($path)) {
+			throw new DAV\Exception\NotFound('File with name '.$path.' could not be located');
+		}
+		if (is_dir($path)) {
+			return new self($path);
+		} else {
+			return new File($path);
+		}
+	}
 
-    /**
-     * Returns an array with all the child nodes.
-     *
-     * @return DAV\INode[]
-     */
-    public function getChildren()
-    {
-        $nodes = [];
-        $iterator = new \FilesystemIterator(
-            $this->path,
-            \FilesystemIterator::CURRENT_AS_SELF
-          | \FilesystemIterator::SKIP_DOTS
-        );
-        foreach ($iterator as $entry) {
-            $nodes[] = $this->getChild($entry->getFilename());
-        }
+	/**
+	 * Returns an array with all the child nodes.
+	 *
+	 * @return DAV\INode[]
+	 */
+	public function getChildren()
+	{
+		$nodes = [];
+		$iterator = new \FilesystemIterator(
+			$this->path,
+			\FilesystemIterator::CURRENT_AS_SELF
+		  | \FilesystemIterator::SKIP_DOTS
+		);
+		foreach ($iterator as $entry) {
+			$nodes[] = $this->getChild($entry->getFilename());
+		}
 
-        return $nodes;
-    }
+		return $nodes;
+	}
 
-    /**
-     * Checks if a child exists.
-     *
-     * @param string $name
-     *
-     * @return bool
-     */
-    public function childExists($name)
-    {
-        $path = $this->path.'/'.$name;
+	/**
+	 * Checks if a child exists.
+	 *
+	 * @param string $name
+	 *
+	 * @return bool
+	 */
+	public function childExists($name)
+	{
+		$path = $this->path.'/'.$name;
 
-        return file_exists($path);
-    }
+		return file_exists($path);
+	}
 
-    /**
-     * Deletes all files in this directory, and then itself.
-     */
-    public function delete()
-    {
-        foreach ($this->getChildren() as $child) {
-            $child->delete();
-        }
-        rmdir($this->path);
-    }
+	/**
+	 * Deletes all files in this directory, and then itself.
+	 */
+	public function delete()
+	{
+		foreach ($this->getChildren() as $child) {
+			$child->delete();
+		}
+		rmdir($this->path);
+	}
 
-    /**
-     * Returns available diskspace information.
-     *
-     * @return array
-     */
-    public function getQuotaInfo()
-    {
-        $absolute = realpath($this->path);
+	/**
+	 * Returns available diskspace information.
+	 *
+	 * @return array
+	 */
+	public function getQuotaInfo()
+	{
+		$absolute = realpath($this->path);
 
-        return [
-            disk_total_space($absolute) - disk_free_space($absolute),
-            disk_free_space($absolute),
-        ];
-    }
+		return [
+			disk_total_space($absolute) - disk_free_space($absolute),
+			disk_free_space($absolute),
+		];
+	}
 }