fix: Properly return copy status in DAV

Signed-off-by: Louis Chemineau <[email protected]>
nextcloud · Nov 26, 2024 · 43ed165 · 43ed165
1 parent 5b01166
commit 43ed165
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/apps/dav/lib/Connector/Sabre/Directory.php b/apps/dav/lib/Connector/Sabre/Directory.php
@@ -447,7 +447,10 @@ public function copyInto($targetName, $sourcePath, INode $sourceNode) {
 					throw new InvalidPath($ex->getMessage());
 				}
 
-				return $this->fileView->copy($sourcePath, $destinationPath);
+				$copyOkay = $this->fileView->copy($sourcePath, $destinationPath);
+				if (!$copyOkay) {
+					throw new \Sabre\DAV\Exception\Forbidden('');
+				}
 			} catch (StorageNotAvailableException $e) {
 				throw new ServiceUnavailable($e->getMessage(), $e->getCode(), $e);
 			} catch (ForbiddenException $ex) {
@@ -457,7 +460,7 @@ public function copyInto($targetName, $sourcePath, INode $sourceNode) {
 			}
 		}
 
-		return false;
+		return true;
 	}
 
 	public function getNode(): Folder {