summaryrefslogtreecommitdiffstats
path: root/a8777ae113732dd18f9cf04d825322f596fedac7.patch
blob: 4f2f02898e6930748c52b391d4e2008fa7a03b9c (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
From a8777ae113732dd18f9cf04d825322f596fedac7 Mon Sep 17 00:00:00 2001
From: Jan Schneider <jan@horde.org>
Date: Mon, 13 Feb 2017 14:57:35 +0100
Subject: [PATCH] Don't use each().

For modern PHP versions there is no improved performance or memory usage compared to foreach() anymore. Beside that it's deprecated in PHP 7.2.
---
 lib/Horde/String.php | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/lib/Horde/String.php b/lib/Horde/String.php
index 26dfcbc..aa8f990 100644
--- a/lib/Horde/String.php
+++ b/lib/Horde/String.php
@@ -66,8 +66,7 @@ public static function convertCharset($input, $from, $to, $force = false)
 
         if (is_array($input)) {
             $tmp = array();
-            reset($input);
-            while (list($key, $val) = each($input)) {
+            foreach ($input as $key => $val) {
                 $tmp[self::_convertCharset($key, $from, $to)] = self::convertCharset($val, $from, $to, $force);
             }
             return $tmp;
@@ -84,7 +83,7 @@ public static function convertCharset($input, $from, $to, $force = false)
 
             $input = clone $input;
             $vars = get_object_vars($input);
-            while (list($key, $val) = each($vars)) {
+            foreach ($vars as $key => $val) {
                 $input->$key = self::convertCharset($val, $from, $to, $force);
             }
             return $input;