New fix for the sonar spotted bug, $i was conflicting as well, using $j now, should be fine.

bf2e403d · Côme Chilliet · Benoit Mortier · eae6630d · bf2e403d
Commit bf2e403d authored 8 years ago by Côme Chilliet Committed by Benoit Mortier 8 years ago
Hide whitespace changes
Inline Side-by-side

Showing

with 2 additions and 2 deletions
+2 -2
--- a/setup/class_setupStep_Migrate.inc
+++ b/setup/class_setupStep_Migrate.inc
@@ -814,8 +814,8 @@ class Step_Migrate extends setupStep

              if (isset($ret['gosaAclTemplate'])) {
                $cnt = $ret['gosaAclTemplate']['count'];
-                for ($i = 0; $i < $cnt; $i++) {
-                  $a_str = $ret['gosaAclTemplate'][$i];
+                for ($j = 0; $j < $cnt; $j++) {
+                  $a_str = $ret['gosaAclTemplate'][$j];
                  if (preg_match("/^[0-9]*:psub:/", $a_str) && preg_match("/:all;cmdrw$/", $a_str)) {

                    $members = explode(",", $tmp[3]);