Merge pull request #15146 from nextcloud/fix/customer_properties_boom

Prefetching blows up if there are a lot of files.

2 files changed, 0 insertions, 11 deletions

diff --git a/apps/dav/lib/Connector/Sabre/CustomPropertiesBackend.php b/apps/dav/lib/Connector/Sabre/CustomPropertiesBackend.php
index c77a4d6be48..f3f685a8b9e 100644
--- a/apps/dav/lib/Connector/Sabre/CustomPropertiesBackend.php
+++ b/apps/dav/lib/Connector/Sabre/CustomPropertiesBackend.php
@@ -129,13 +129,6 @@ class CustomPropertiesBackend implements BackendInterface {
 			return;
 		}
 
-		if ($node instanceof Directory
-			&& $propFind->getDepth() !== 0
-		) {
-			// note: pre-fetching only supported for depth <= 1
-			$this->loadChildrenProperties($node, $requestedProps);
-		}
-
 		$props = $this->getProperties($node, $requestedProps);
 		foreach ($props as $propName => $propValue) {
 			$propFind->set($propName, $propValue);
diff --git a/apps/dav/tests/unit/Connector/Sabre/CustomPropertiesBackendTest.php b/apps/dav/tests/unit/Connector/Sabre/CustomPropertiesBackendTest.php
index 91973eeff5d..3b7bd859b9f 100644
--- a/apps/dav/tests/unit/Connector/Sabre/CustomPropertiesBackendTest.php
+++ b/apps/dav/tests/unit/Connector/Sabre/CustomPropertiesBackendTest.php
@@ -226,10 +226,6 @@ class CustomPropertiesBackendTest extends \Test\TestCase {
 			->method('getPath')
 			->will($this->returnValue('/dummypath/test.txt'));
 
-		$rootNode->expects($this->once())
-			->method('getChildren')
-			->will($this->returnValue(array($nodeSub)));
-
 		$this->tree->expects($this->at(0))
 			->method('getNodeForPath')
 			->with('/dummypath')