Changeset 25018 for trunk/admin/user_list.php

Timestamp:

Oct 19, 2013, 7:43:04 PM (11 years ago)

Author:

mistic100

Message:

remove all array_push (50% slower than []) + some changes missing for feature:2978

File:

• trunk/admin/user_list.php (modified) (15 diffs)

trunk/admin/user_list.php

@@ -120 +120 @@
     $user['groups'] = array();
 
-    array_push($users, $user);
+    $users[] = $user;
   }
 
@@ -141 +141 @@
     while ($row = pwg_db_fetch_assoc($result))
     {
-      array_push(
-        $users[$user_nums[$row['user_id']]]['groups'],
-        $row['group_id']
-        );
+      $users[ $user_nums[ $row['user_id'] ] ]['groups'][] = $row['group_id'];
     }
   }
@@ -192 +189 @@
     if(empty($_POST['password']))
     {
-      array_push($page['errors'], l10n('Password is missing. Please enter the password.'));
+      $page['errors'][] = l10n('Password is missing. Please enter the password.');
     }
     else if(empty($_POST['password_conf']))
     {
-      array_push($page['errors'], l10n('Password confirmation is missing. Please confirm the chosen password.'));
+      $page['errors'][] = l10n('Password confirmation is missing. Please confirm the chosen password.');
     }
     else if(empty($_POST['email']))
     {
-      array_push($page['errors'], l10n('Email address is missing. Please specify an email address.'));
+      $page['errors'][] = l10n('Email address is missing. Please specify an email address.');
     }
     else if ($_POST['password'] != $_POST['password_conf'])
     {
-      array_push($page['errors'], l10n('The passwords do not match'));
+      $page['errors'][] = l10n('The passwords do not match');
     }
     else
@@ -213 +210 @@
       if (count($page['errors']) == 0)
       {
-        array_push(
-          $page['infos'],
-          l10n(
-            'user "%s" added',
-            $_POST['login']
-          )
-        );
+        $page['infos'][] = l10n('user "%s" added', $_POST['login']);
       }
     }
@@ -233 +224 @@
     if (count($page['errors']) == 0)
     {
-      array_push(
-        $page['infos'],
-        l10n(
-          'user "%s" added',
-          stripslashes($_POST['login'])
-          )
-        );
+      $page['infos'][] = l10n('user "%s" added', stripslashes($_POST['login']));
     }
   }
@@ -296 +281 @@
       foreach($page['filtered_users'] as $local_user)
       {
-        array_push($collection, $local_user['id']);
+        $collection[] = $local_user['id'];
       }
       break;
@@ -312 +297 @@
   if (count($collection) == 0)
   {
-    array_push($page['errors'], l10n('Select at least one user'));
+    $page['errors'][] = l10n('Select at least one user');
   }
 }
@@ -323 +308 @@
   if (in_array($conf['guest_id'], $collection))
   {
-    array_push($page['errors'], l10n('Guest cannot be deleted'));
+    $page['errors'][] = l10n('Guest cannot be deleted');
   }
   if (($conf['guest_id'] != $conf['default_user_id']) and
       in_array($conf['default_user_id'], $collection))
   {
-    array_push($page['errors'], l10n('Default user cannot be deleted'));
+    $page['errors'][] = l10n('Default user cannot be deleted');
   }
   if (in_array($conf['webmaster_id'], $collection))
   {
-    array_push($page['errors'], l10n('Webmaster cannot be deleted'));
+    $page['errors'][] = l10n('Webmaster cannot be deleted');
   }
   if (in_array($user['id'], $collection))
   {
-    array_push($page['errors'], l10n('You cannot delete your account'));
+    $page['errors'][] = l10n('You cannot delete your account');
   }
 
@@ -347 +332 @@
         delete_user($user_id);
       }
-      array_push(
-        $page['infos'],
-        l10n_dec(
-          '%d user deleted', '%d users deleted',
-          count($collection)
-          )
+      
+      $page['infos'][] = l10n_dec(
+        '%d user deleted', '%d users deleted',
+        count($collection)
         );
+      
       foreach ($page['filtered_users'] as $filter_key => $filter_user)
       {
@@ -364 +348 @@
     else
     {
-      array_push($page['errors'], l10n('You need to confirm deletion'));
+      $page['errors'][] = l10n('You need to confirm deletion');
     }
   }
@@ -392 +376 @@
       foreach ($associable as $item)
       {
-        array_push($datas,
-                   array('group_id'=>$_POST['associate'],
-                         'user_id'=>$item));
+        $datas[] = array(
+          'group_id' => $_POST['associate'],
+          'user_id' => $item
+          );
       }
 
@@ -427 +412 @@
   if ($conf['activate_comments'])
   {
-    array_push($formfields, 'show_nb_comments');
-    array_push($true_false_fields, 'show_nb_comments');
+    $formfields[] = 'show_nb_comments';
+    $true_false_fields[] = 'show_nb_comments';
   }
 
@@ -445 +430 @@
     if ($_POST[$test] != 'leave')
     {
-      array_push($dbfields['update'], $formfield);
+      $dbfields['update'][] = $formfield;
     }
   }
@@ -481 +466 @@
           if (!in_array('level', $dbfields['update']))
           {
-            array_push($dbfields['update'], 'level');
+            $dbfields['update'][] = 'level';
           }
         }
@@ -508 +493 @@
       }
 
-      array_push($datas, $data);
+      $datas[] = $data;
     }