"entry" is the wrong word

Author: simonLeary42

resources/lib/UnityLDAP.php

@@ -257,20 +257,20 @@ public function getAllUsers($UnitySQL, $UnityMailer, $UnityRedis, $UnityWebhook,
         }
     }
 
-    public function getAllUsersEntries($attributes = [])
+    public function getAllUsersArrays($attributes = [])
     {
         $include_uids = $this->getAllUsersUIDs();
-        $user_entries = $this->baseOU->getChildrenArray(
+        $user_arrays = $this->baseOU->getChildrenArray(
             $attributes,
             true, // recursive
             "objectClass=posixAccount"
         );
-        foreach ($user_entries as $i => $entry) {
-            if (!in_array($entry["uid"][0], $include_uids)) {
-                unset($user_entries[$i]);
+        foreach ($user_arrays as $i => $array) {
+            if (!in_array($array["uid"][0], $include_uids)) {
+                unset($user_arrays[$i]);
             }
         }
-        return $user_entries;
+        return $user_arrays;
     }
 
     public function getAllPIGroups($UnitySQL, $UnityMailer, $UnityRedis, $UnityWebhook, $ignorecache = false)
@@ -305,7 +305,7 @@ public function getAllPIGroups($UnitySQL, $UnityMailer, $UnityRedis, $UnityWebho
         return $out;
     }
 
-    public function getAllPIGroupsEntries()
+    public function getAllPIGroupsArrays()
     {
         return $this->pi_groupOU->getChildrenArray([], true);
     }
@@ -322,7 +322,7 @@ public function getPIGroupGIDsWithMemberUID($uid)
         );
     }
 
-    public function getAllPIGroupOwnerEntries($attributes = [])
+    public function getAllPIGroupOwnerArrays($attributes = [])
     {
         // get the PI groups, filter for just the GIDs, then map the GIDs to owner UIDs
         $owner_uids = array_map(
@@ -332,13 +332,13 @@ public function getAllPIGroupOwnerEntries($attributes = [])
                 $this->pi_groupOU->getChildrenArray(["cn"]),
             ),
         );
-        $owner_entries = $this->getAllUsersEntries($attributes);
-        foreach ($owner_entries as $i => $entry) {
-            if (!in_array($entry["uid"][0], $owner_uids)) {
-                unset($owner_entries[$i]);
+        $owner_arrays = $this->getAllUsersArrays($attributes);
+        foreach ($owner_arrays as $i => $array) {
+            if (!in_array($array["uid"][0], $owner_uids)) {
+                unset($owner_arrays[$i]);
             }
         }
-        return $owner_entries;
+        return $owner_arrays;
     }
 
     /** Returns an assosiative array where keys are UIDs and values are arrays of PI GIDs */
@@ -348,9 +348,9 @@ public function getAllUID2PIGIDs()
         $uids = $this->getAllUsersUIDs();
         $uid2pigids = array_combine($uids, array_fill(0, count($uids), []));
         // for each PI group, append that GID to the member list for each of its member UIDs
-        foreach ($this->getAllPIGroupsEntries() as $entry) {
-            $gid = $entry["cn"][0];
-            foreach ($entry["memberuid"] as $uid) {
+        foreach ($this->getAllPIGroupsArrays() as $array) {
+            $gid = $array["cn"][0];
+            foreach ($array["memberuid"] as $uid) {
                 array_push($uid2pigids[$uid], $gid);
             }
         }
@@ -388,7 +388,7 @@ public function getAllOrgGroups($UnitySQL, $UnityMailer, $UnityRedis, $UnityWebh
         return $out;
     }
 
-    public function getAllOrgGroupsEntries()
+    public function getAllOrgGroupsArrays()
     {
         return $this->org_groupOU->getChildrenArray([], true);
     }

webroot/admin/pi-mgmt.php

@@ -108,13 +108,13 @@
     </tr>
 
     <?php
-    $owner_entries = $LDAP->getAllPIGroupOwnerEntries(["uid", "gecos", "mail"]);
-    usort($owner_entries, fn($a, $b) => strcmp($a["uid"][0], $b["uid"][0]));
-    foreach ($owner_entries as $entry) {
+    $owner_arrays = $LDAP->getAllPIGroupOwnerArrays(["uid", "gecos", "mail"]);
+    usort($owner_arrays, fn($a, $b) => strcmp($a["uid"][0], $b["uid"][0]));
+    foreach ($owner_arrays as $array) {
         echo "<tr class='expandable'>";
-        echo "<td><button class='btnExpand'>&#9654;</button>" . $entry["gecos"][0] . "</td>";
-        echo "<td>" . UnityGroup::getPIUIDfromUID($entry["uid"][0]) . "</td>";
-        echo "<td><a href='mailto:" . $entry["mail"][0] . "'>" . $entry["mail"][0] . "</a></td>";
+        echo "<td><button class='btnExpand'>&#9654;</button>" . $array["gecos"][0] . "</td>";
+        echo "<td>" . UnityGroup::getPIUIDfromUID($array["uid"][0]) . "</td>";
+        echo "<td><a href='mailto:" . $array["mail"][0] . "'>" . $array["mail"][0] . "</a></td>";
         echo "</tr>";
     }
     ?>

webroot/admin/user-mgmt.php

@@ -38,19 +38,19 @@
 
     <?php
     $UID2PIGIDs = $LDAP->getAllUID2PIGIDs();
-    $user_entries = $LDAP->getAllUsersEntries(["uid", "gecos", "o", "mail"]);
-    usort($user_entries, fn ($a, $b) => strcmp($a["uid"][0], $b["uid"][0]));
-    foreach ($user_entries as $entry) {
-        $uid = $entry["uid"][0];
+    $user_arrays = $LDAP->getAllUsersArrays(["uid", "gecos", "o", "mail"]);
+    usort($user_arrays, fn ($a, $b) => strcmp($a["uid"][0], $b["uid"][0]));
+    foreach ($user_arrays as $array) {
+        $uid = $array["uid"][0];
         if ($SQL->accDeletionRequestExists($uid)) {
             echo "<tr style='color:grey; font-style: italic'>";
         } else {
             echo "<tr>";
         }
-        echo "<td>" . $entry["gecos"][0] . "</td>";
+        echo "<td>" . $array["gecos"][0] . "</td>";
         echo "<td>" . $uid . "</td>";
-        echo "<td>" . $entry["o"][0] . "</td>";
-        echo "<td><a href='mailto:" . $entry["mail"][0] . "'>" . $entry["mail"][0] . "</a></td>";
+        echo "<td>" . $array["o"][0] . "</td>";
+        echo "<td><a href='mailto:" . $array["mail"][0] . "'>" . $array["mail"][0] . "</a></td>";
         echo "<td>";
         foreach ($UID2PIGIDs[$uid] as $gid) {
             echo "<p>$gid</p>";