Merge pull request #7741 from nextcloud/backport/7727/stable29

grnd-alt · web-flow · commit 14c10f059954 · 2026-03-11T11:27:46.000+01:00
[stable29] fix: check cardId for attachment
diff --git a/lib/Service/AttachmentService.php b/lib/Service/AttachmentService.php
@@ -289,6 +289,7 @@ public function update($cardId, $attachmentId, $data, $type = 'deck_file') {
 		}
 
 		if ($service instanceof ICustomAttachmentService) {
+			$this->permissionService->checkPermission($this->cardMapper, $cardId, Acl::PERMISSION_EDIT);
 			try {
 				$attachment = new Attachment();
 				$attachment->setId($attachmentId);

Original file line number	Diff line number	Diff line change
`@@ -289,6 +289,7 @@ public function update($cardId, $attachmentId, $data, $type = 'deck_file') {`
`289`	`289`	`}`
`290`	`290`
`291`	`291`	`if ($service instanceof ICustomAttachmentService) {`
	`292`	`+ $this->permissionService->checkPermission($this->cardMapper, $cardId, Acl::PERMISSION_EDIT);`
`292`	`293`	`try {`
`293`	`294`	`$attachment = new Attachment();`
`294`	`295`	`$attachment->setId($attachmentId);`