Add check if identifier for all groups/user is set to make looser fil…

…ter possible
mapbender · Jun 28, 2019 · 384b4ba · 384b4ba
1 parent 0fd8ab5
commit 384b4ba
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/Security/Provider/LDAPIdentitiesProvider.php b/Security/Provider/LDAPIdentitiesProvider.php
@@ -25,7 +25,10 @@ public function getAllUsers()
         if ($ldapUserList !== null) {
 
             foreach (array_slice($ldapUserList, 2) as $ldapUser) {
-                $user[] = new User($ldapUser[$nameAttribute][0]);
+                if(isset($ldapUser[$nameAttribute][0])){
+                    $user[] = new User($ldapUser[$nameAttribute][0]);
+                }
+
             }
 
         }
@@ -66,7 +69,11 @@ public function getAllGroups()
         $ldapGroupList = $ldapClient->find($groupDn, $groupFilter);
         if ($ldapGroupList != null) {
             foreach (array_slice($ldapGroupList, 2) as $ldapGroup) {
-                $groups[] = new Group($ldapGroup[$groupIdentifier][0]);
+                if(isset($ldapGroup[$groupIdentifier])){
+                    $groups[] = new Group($ldapGroup[$groupIdentifier][0]);
+                }
+
+
             }
 
         }