Merge pull request #56498 from nextcloud/fix/fix-deleting-broken-shares

sorbaugh · web-flow · commit a7dd4e8c0553 · 2025-11-25T11:46:29.000+01:00
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
@@ -545,6 +545,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
`@@ -545,6 +545,8 @@ public function deleteShare(string $id): DataResponse {`
`545`	`545`	`$this->lock($share->getNode());`
`546`	`546`	`} catch (LockedException $e) {`
`547`	`547`	`throw new OCSNotFoundException($this->l->t('Could not delete share'));`
	`548`	`+ } catch (NotFoundException $e) {`
	`549`	`+ $this->logger->debug('File of to be deleted share was not found, skip locking', ['exception' => $e]);`
`548`	`550`	`}`
`549`	`551`
`550`	`552`	`if (!$this->canAccessShare($share)) {`