Skip to content
Snippets Groups Projects
Commit 050ff7f7 authored by Dorian Merz's avatar Dorian Merz Committed by Robert Lange
Browse files

refs #19711 [fid] change FidPermissionProvider behaviour to OR by default

* from now on, if a set of fidis permissions is configured,
  users having ANY of those will be granted the respective VuFind permission
* potentially BREAKING CHANGE
parent c4476fd2
No related merge requests found
......@@ -67,12 +67,12 @@ class FidApiPermission implements PermissionProviderInterface
*/
public function getPermissions($options)
{
$authorized = TRUE;
$authorized = FALSE;
foreach ((array)$options as $option) {
if (!$this->client->isAuthorized($option)) {
// a user missing any role
// will not be authorized
$authorized = FALSE;
if ($this->client->isAuthorized($option)) {
// a user having any of the provided roles
// will be authorized
$authorized = TRUE;
break;
}
}
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment