fix: Throw exception when copy failed

Signed-off-by: Louis Chemineau <louis@chmn.me>
nextcloud · Dec 2, 2024 · bafc1e5 · bafc1e5
1 parent 4310615
commit bafc1e5
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/apps/dav/lib/Connector/Sabre/Directory.php b/apps/dav/lib/Connector/Sabre/Directory.php
@@ -447,7 +447,13 @@ 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('Copy did not proceed');
+				}
+
+				return true;
 			} catch (StorageNotAvailableException $e) {
 				throw new ServiceUnavailable($e->getMessage(), $e->getCode(), $e);
 			} catch (ForbiddenException $ex) {