Merge pull request #56657 from nextcloud/backport/56498/stable32

come-nc · web-flow · commit 117d0ae7b15a · 2025-11-27T15:18:46.000+01:00
[stable32] fix(files_sharing): Allow deleting a share of a missing file
diff --git a/apps/files_sharing/lib/Controller/ShareAPIController.php b/apps/files_sharing/lib/Controller/ShareAPIController.php
@@ -543,6 +543,8 @@ public function deleteShare(string $id): DataResponse {
 			$this->lock($share->getNode());
 		} catch (LockedException $e) {
 			throw new OCSNotFoundException($this->l->t('Could not delete share'));
+		} catch (NotFoundException $e) {
+			$this->logger->debug('File of to be deleted share was not found, skip locking', ['exception' => $e]);
 		}
 
 		if (!$this->canAccessShare($share)) {

Original file line number	Diff line number	Diff line change
`@@ -543,6 +543,8 @@ public function deleteShare(string $id): DataResponse {`
`543`	`543`	`$this->lock($share->getNode());`
`544`	`544`	`} catch (LockedException $e) {`
`545`	`545`	`throw new OCSNotFoundException($this->l->t('Could not delete share'));`
	`546`	`+ } catch (NotFoundException $e) {`
	`547`	`+ $this->logger->debug('File of to be deleted share was not found, skip locking', ['exception' => $e]);`
`546`	`548`	`}`
`547`	`549`
`548`	`550`	`if (!$this->canAccessShare($share)) {`