Merge pull request #3699 from nextcloud/fix/acl-plugin/test-permissions-path

provokateurin · web-flow · commit 65ba1eb21bd2 · 2025-03-18T12:08:49.000+01:00
fix(ACLPlugin): Use correct path to test new permissions
diff --git a/lib/DAV/ACLPlugin.php b/lib/DAV/ACLPlugin.php
@@ -227,7 +227,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 cannot remove your own read permission.'));
 			}

Original file line number	Diff line number	Diff line change
`@@ -227,7 +227,7 @@ public function propPatch(string $path, PropPatch $propPatch): void {`
`227`	`227`	`}`
`228`	`228`
`229`	`229`	`$aclManager = $this->aclManagerFactory->getACLManager($this->user);`
`230`		`- $newPermissions = $aclManager->testACLPermissionsForPath($fileInfo->getPath(), $rules);`
	`230`	`+ $newPermissions = $aclManager->testACLPermissionsForPath($path, $rules);`
`231`	`231`	`if (!($newPermissions & Constants::PERMISSION_READ)) {`
`232`	`232`	`throw new BadRequest($this->l10n->t('You cannot remove your own read permission.'));`
`233`	`233`	`}`