refactor

Author: simonLeary42

resources/lib/UnityLDAP.php

@@ -256,6 +256,19 @@ public function getPIGroupGIDsWithMemberUID(string $uid): array
         );
     }
 
+    public function getPIGroupAttributesWithMemberUID(
+        string $uid,
+        array $attributes,
+        array $default_values = [],
+    ) {
+        return $this->pi_groupOU->getChildrenArrayStrict(
+            $attributes,
+            recursive: false,
+            filter: sprintf("(memberuid=%s)", ldap_escape($uid, "", LDAP_ESCAPE_FILTER)),
+            default_values: $default_values,
+        );
+    }
+
     /**
      * Returns an associative array where keys are UIDs and values are arrays of PI GIDs
      */
@@ -397,17 +410,4 @@ public function getUsersAttributes(
         ksort($output);
         return $output;
     }
-
-    public function getPIGroupsAttributesWithMemberUID(
-        string $uid,
-        array $attributes,
-        array $default_values = [],
-    ) {
-        return $this->pi_groupOU->getChildrenArrayStrict(
-            $attributes,
-            recursive: false,
-            filter: sprintf("(memberuid=%s)", ldap_escape($uid, "", LDAP_ESCAPE_FILTER)),
-            default_values: $default_values,
-        );
-    }
 }

webroot/panel/groups.php

@@ -75,7 +75,7 @@
 
 <?php
 $PIGroupGIDs = [];
-$PIGroupAttributes = $LDAP->getPIGroupsAttributesWithMemberUID(
+$PIGroupAttributes = $LDAP->getPIGroupAttributesWithMemberUID(
     $USER->uid,
     ["cn", "memberuid"],
     default_values: ["memberuid" => []]