| @@ -4,8 +4,9 @@ | ||
| 4 | 4 | |
| 5 | 5 | namespace UserAccessManager\ObjectMembership; |
| 6 | 6 | |
| 7 | 7 | use Exception; |
| 8 | +use UserAccessManager\ObjectMembership\Exception\MissingObjectTypeException; | |
| 8 | 9 | use UserAccessManager\UserGroup\AbstractUserGroup; |
| 9 | 10 | use UserAccessManager\UserGroup\AssignmentInformation; |
| 10 | 11 | use UserAccessManager\UserGroup\AssignmentInformationFactory; |
| 11 | 12 | |
| @@ -37,10 +38,9 @@ | ||
| 37 | 38 | } |
| 38 | 39 | |
| 39 | 40 | public function handlesObject(mixed $objectType): bool |
| 40 | 41 | { |
| 41 | - $objectTypes = $this->getHandledObjects(); | |
| 42 | - return isset($objectTypes[$objectType]); | |
| 42 | + return isset($this->getHandledObjects()[$objectType]); | |
| 43 | 43 | } |
| 44 | 44 | |
| 45 | 45 | protected function assignRecursiveMembership( |
| 46 | 46 | ?AssignmentInformation &$assignmentInformation, |
| @@ -67,15 +67,11 @@ | ||
| 67 | 67 | } |
| 68 | 68 | |
| 69 | 69 | protected function getSimpleAssignedObjects(AbstractUserGroup $userGroup, string $objectType): array |
| 70 | 70 | { |
| 71 | - $objects = $userGroup->getAssignedObjects($objectType); | |
| 72 | - | |
| 73 | 71 | return array_map( |
| 74 | - function (AssignmentInformation $element) { | |
| 75 | - return $element->getType(); | |
| 76 | - }, | |
| 77 | - $objects | |
| 72 | + fn(AssignmentInformation $assignmentInformation) => $assignmentInformation->getType(), | |
| 73 | + $userGroup->getAssignedObjects($objectType) | |
| 78 | 74 | ); |
| 79 | 75 | } |
| 80 | 76 | |
| 81 | 77 | abstract public function isMember( |
