[TASK] Use identifier instead of URL in propFind()
authorAndreas Wolf <andreas.wolf@typo3.org>
Wed, 3 Jun 2015 19:09:17 +0000 (21:09 +0200)
committerAndreas Wolf <andreas.wolf@typo3.org>
Sun, 7 Jun 2015 19:50:15 +0000 (21:50 +0200)
Classes/Dav/WebDavFrontend.php
Classes/Driver/WebDavDriver.php

index 6fc85c0..f9a524e 100644 (file)
@@ -79,14 +79,14 @@ class WebDavFrontend {
        /**
         * Wrapper around the PROPFIND method of the WebDAV client to get proper local error handling.
         *
-        * @param string $url
+        * @param string $path
         * @return array
         * @throws DAV\Exception\NotFound if the given URL does not hold a resource
         *
         * TODO define proper error handling for other cases
         */
-       public function propFind($url) {
-               // TODO check cache
+       public function propFind($path) {
+               $url = $this->baseUrl . $path;
                $encodedUrl = $this->encodeUrl($url);
 
                try {
@@ -149,11 +149,10 @@ class WebDavFrontend {
                if (strlen($path) > 0) {
                        $path .= '/';
                }
-               $url = $this->baseUrl . $path;
-               $urlParts = parse_url($url);
+               $urlParts = parse_url($this->baseUrl . $path);
                $unencodedBasePath = $urlParts['path'];
 
-               $result = $this->propFind($url);
+               $result = $this->propFind($path);
 
                // remove first entry, as it is the folder itself
                array_shift($result);
@@ -187,9 +186,8 @@ class WebDavFrontend {
        public function getFileInfo($path) {
                // the leading slash is already included in baseURL/basePath
                $path = ltrim($path, '/');
-               $url = $this->baseUrl . $path;
 
-               $result = $this->propFind($url);
+               $result = $this->propFind($path);
 
                return $this->extractFileInfo($path, $result[$this->basePath . $this->urlEncodePath($path)]);
        }
index b374d2e..62119ad 100644 (file)
@@ -803,9 +803,7 @@ class WebDavDriver extends AbstractDriver {
         * @return bool TRUE if there are no files and folders within $folder
         */
        public function isFolderEmpty($folderIdentifier) {
-               $folderUrl = $this->getResourceUrl($folderIdentifier);
-
-               $folderContents = $this->frontend->propFind($folderUrl);
+               $folderContents = $this->frontend->propFind($folderIdentifier);
 
                return (count($folderContents) == 1);
        }