Merge pull request #105 from vyskocilpavel/adapterlpda_groups

Fixed problem with getting group without description from LDAP

Author: vyskocilpavel

CHANGELOG.md

@@ -2,6 +2,10 @@
 All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
+- Fixed problem with getting group without description from LDAP 
+  * Before: Exeption
+  * Now: Description is ''
+
 #### Changed
 - Releasing forwardedEduPersonEntitlement is now optional (forwardedEduPersonEntitlement are released by default)
 

lib/AdapterLdap.php

@@ -114,7 +114,7 @@ public function getMemberGroups($user, $vo)
                     $group['perunVoId'][0],
                     $group['cn'][0],
                     $group['perunUniqueGroupName'][0],
-                    $group['description'][0]
+                    $group['description'][0] ?? ''
                 )
             );
         }
@@ -154,7 +154,7 @@ public function getSpGroups($spEntityId)
                             $group['perunVoId'][0],
                             $group['cn'],
                             $group['perunUniqueGroupName'][0],
-                            $group['description'][0]
+                            $group['description'][0] ?? ''
                         )
                     );
                 }
@@ -183,7 +183,7 @@ public function getGroupByName($vo, $name)
             $group['perunVoId'][0],
             $group['cn'][0],
             $group['perunUniqueGroupName'][0],
-            $group['description'][0]
+            $group['description'][0] ?? ''
         );
     }
 
@@ -352,7 +352,7 @@ public function getUsersGroupsOnFacility($spEntityId, $userId)
                     $group['perunVoId'][0],
                     $group['cn'][0],
                     $group['perunUniqueGroupName'][0],
-                    $group['description'][0]
+                    $group['description'][0] ?? ''
                 )
             );
         }