Skip to content
This repository was archived by the owner on Sep 19, 2022. It is now read-only.

Commit c6605bd

Browse files
author
Dominik František Bučík
authored
Merge pull request #283 from dBucik/fix_capabilities
fix: 🐛 Fix EntitlementUtils calling capabilities in LDAP
2 parents 081ad75 + e3aedd2 commit c6605bd

File tree

2 files changed

+6
-7
lines changed

2 files changed

+6
-7
lines changed

lib/AdapterLdap.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -632,7 +632,7 @@ public function getResourceCapabilities(string $spEntityId, array $userGroups, s
632632

633633
foreach ($resource[self::ASSIGNED_GROUP_ID] as $groupId) {
634634
if (in_array($groupId, $userGroupsIds, true)) {
635-
$capabilities = array_merge($capabilities, $resources[self::CAPABILITIES]);
635+
$capabilities = array_merge($capabilities, $resource[self::CAPABILITIES]);
636636
break;
637637
}
638638
}

lib/EntitlementUtils.php

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -73,13 +73,12 @@ public static function getCapabilities(&$request, $adapter, $prefix, $authority,
7373
}
7474

7575
try {
76-
$resourceCapabilities = $entityIdAttr === null ? $adapter->getResourceCapabilities(
76+
$resourceCapabilities = $adapter->getResourceCapabilities(
7777
$spEntityId,
78-
$request['perun']['groups']
79-
) : $adapter->getResourceCapabilities($spEntityId, $request['perun']['groups'], $entityIdAttr);
80-
$facilityCapabilities = $entityIdAttr === null ? $adapter->getFacilityCapabilities(
81-
$spEntityId
82-
) : $adapter->getFacilityCapabilities($spEntityId, $entityIdAttr);
78+
$request['perun']['groups'],
79+
$entityIdAttr ?? ''
80+
);
81+
$facilityCapabilities = $adapter->getFacilityCapabilities($spEntityId, $entityIdAttr ?? '');
8382
} catch (Exception $exception) {
8483
Logger::error(
8584
'perun:EntitlementUtils: Exception ' . $exception->getMessage() .

0 commit comments

Comments
 (0)