Merge pull request #33265 from owncloud/emptyPropfindResult

return false if propfind result is empty
owncloud · Oct 23, 2018 · e0871ca · e0871ca
2 parents 5971d62 + 5f35ec8
commit e0871ca
Showing 1 changed file with 12 additions and 10 deletions.
diff --git a/tests/acceptance/features/bootstrap/WebDav.php b/tests/acceptance/features/bootstrap/WebDav.php
@@ -2706,16 +2706,18 @@ public function findFileFromPropfindResponse($user, $fileNameToSearch) {
 			$this->parseResponseIntoXml();
 		}
 		$multistatusResults = $this->responseXml["value"];
-		foreach ($multistatusResults as $multistatusResult) {
-			$filePath = $multistatusResult['value'][0]['value'];
-			$fullWebDavPath = \ltrim(
-				$this->getBasePath() . "/" . $this->getDavFilesPath($user) . "/",
-				"/"
-			);
-			$fileName = \str_replace($fullWebDavPath, "", $filePath);
-			$fileName = \rawurldecode($fileName);
-			if ($fileName === $fileNameToSearch) {
-				return $multistatusResult;
+		if ($multistatusResults !== null) {
+			foreach ($multistatusResults as $multistatusResult) {
+				$filePath = $multistatusResult['value'][0]['value'];
+				$fullWebDavPath = \ltrim(
+					$this->getBasePath() . "/" . $this->getDavFilesPath($user) . "/",
+					"/"
+				);
+				$fileName = \str_replace($fullWebDavPath, "", $filePath);
+				$fileName = \rawurldecode($fileName);
+				if ($fileName === $fileNameToSearch) {
+					return $multistatusResult;
+				}
 			}
 		}
 		return false;