From 62efa28796a08a8a72f3ecc7c61061fb66f4143b Mon Sep 17 00:00:00 2001
From: Bruce Bannon <github.ful9l@simplelogin.com>
Date: Thu, 28 Nov 2024 02:01:26 -0700
Subject: [PATCH] convert for() to foreach()

---
 access.php | 35 +++++++++++++++++++----------------
 1 file changed, 19 insertions(+), 16 deletions(-)

diff --git a/access.php b/access.php
index 24e0107ed..2e3c64363 100644
--- a/access.php
+++ b/access.php
@@ -183,18 +183,20 @@
         <option value="__default__"'
    . ( $guser == '__default__' ? $selected : '' )
    . '>' . $defConfigStr . '</option>';
-  for( $i = 0, $cnt = count( $userlist ); $i < $cnt; $i++ ) {
+
+  foreach ( $userlist as $i ) {
     echo '
-        <option value="' . $userlist[$i]['cal_login'] . '"'
-     . ( $guser == $userlist[$i]['cal_login'] ? $selected : '' )
-     . '>' . $userlist[$i]['cal_fullname'] . '</option>';
+        <option value="' . $i['cal_login'] .
+      ( $guser === $i['cal_login'] ? '" selected>' : '">' ) .
+      $i['cal_fullname'] . '</option>';
   }
-  for( $i = 0, $cnt = count( $nonuserlist ); $i < $cnt; $i++ ) {
+
+  foreach ( $nonuserlist as $i ) {
     echo '
-        <option value="' . $nonuserlist[$i]['cal_login'] . '"'
-     . ( $guser == $nonuserlist[$i]['cal_login'] ? $selected : '' )
-     . '>' . $nonuserlist[$i]['cal_fullname'] . ' '
-     . ( $nonuserlist[$i]['cal_is_public'] == 'Y' ? '*' : '' ) . '</option>';
+        <option value="' . $i['cal_login'] .
+      ( $guser === $i['cal_login'] ? '" selected>' : '">' ) .
+      $i['cal_fullname'] . ' ' .
+      ( $i['cal_is_public'] === 'Y' ? '*' : '' ) . '</option>';
   }
 
   echo $goStr;
@@ -294,14 +296,15 @@
      . ( $otheruser == '__default__' ? $selected : '' )
      . '>' . $defConfigStr . '</option>';
 
-    for( $i = 0, $cnt = count( $userlist ); $i < $cnt; $i++ ) {
-      if( $userlist[$i]['cal_login'] != $guser )
+    foreach ( $userlist as $i ) {
+      if ( $i['cal_login'] !== $guser )
         echo '
-        <option value="' . $userlist[$i]['cal_login'] . '"'
-         . ( ! empty( $otheruser ) && $otheruser == $userlist[$i]['cal_login']
-          ? $selected : '' )
-         . '>' . $userlist[$i]['cal_fullname'] . '</option>';
+        <option value="' . $i['cal_login'] .
+          ( ! empty ( $otheruser ) && $otheruser === $i['cal_login']
+            ? '" selected>' : '">' ) .
+          $i['cal_fullname'] . '</option>';
     }
+
     echo $goStr;
   }
 }
@@ -475,4 +478,4 @@ function get_list_of_users( $user ) {
   return $u;
 }
 
-?>
+?>
\ No newline at end of file