Merge pull request #3700 from nextcloud/backport/3699/stable30

provokateurin · web-flow · commit 9534730ac2e3 · 2025-03-18T13:02:27.000+01:00
[stable30] fix(ACLPlugin): Use correct path to test new permissions
diff --git a/lib/DAV/ACLPlugin.php b/lib/DAV/ACLPlugin.php
@@ -220,7 +220,7 @@ public function propPatch(string $path, PropPatch $propPatch): void {
 			}
 
 			$aclManager = $this->aclManagerFactory->getACLManager($this->user);
-			$newPermissions = $aclManager->testACLPermissionsForPath($fileInfo->getPath(), $rules);
+			$newPermissions = $aclManager->testACLPermissionsForPath($path, $rules);
 			if (!($newPermissions & Constants::PERMISSION_READ)) {
 				throw new BadRequest($this->l10n->t('You can not remove your own read permission.'));
 			}

Original file line number	Diff line number	Diff line change
`@@ -220,7 +220,7 @@ public function propPatch(string $path, PropPatch $propPatch): void {`
`220`	`220`	`}`
`221`	`221`
`222`	`222`	`$aclManager = $this->aclManagerFactory->getACLManager($this->user);`
`223`		`- $newPermissions = $aclManager->testACLPermissionsForPath($fileInfo->getPath(), $rules);`
	`223`	`+ $newPermissions = $aclManager->testACLPermissionsForPath($path, $rules);`
`224`	`224`	`if (!($newPermissions & Constants::PERMISSION_READ)) {`
`225`	`225`	`throw new BadRequest($this->l10n->t('You can not remove your own read permission.'));`
`226`	`226`	`}`