Browse Source

Merge pull request #66 from jonathangreen/6.x

Fixed a warning in SecurityClass.inc
pull/65/merge
Jonathan Green 13 years ago
parent
commit
011d874553
  1. 12
      SecurityClass.inc

12
SecurityClass.inc

@ -81,11 +81,15 @@ class SecurityClass {
$roles = $xml->xpath('//default:SubjectAttributeDesignator[@AttributeId="fedoraRole"]/../default:Apply/default:AttributeValue'); $roles = $xml->xpath('//default:SubjectAttributeDesignator[@AttributeId="fedoraRole"]/../default:Apply/default:AttributeValue');
$users = $xml->xpath('//default:SubjectAttributeDesignator[@AttributeId="urn:fedora:names:fedora:2.1:subject:loginId"]/../default:Apply/default:AttributeValue'); $users = $xml->xpath('//default:SubjectAttributeDesignator[@AttributeId="urn:fedora:names:fedora:2.1:subject:loginId"]/../default:Apply/default:AttributeValue');
foreach($roles as $role) { if($roles) {
$allowedRoles[] = (string)$role; foreach($roles as $role) {
$allowedRoles[] = (string)$role;
}
} }
foreach($users as $user) { if($users) {
$allowedUsers[] = (string)$user; foreach($users as $user) {
$allowedUsers[] = (string)$user;
}
} }
$usersAndRoles['users'] = $allowedUsers; $usersAndRoles['users'] = $allowedUsers;

Loading…
Cancel
Save