Fix: re-index array

NodeIterator breaks if array is not re-indexed.
Trainmaster · Sep 21, 2014 · f6eb155 · f6eb155
1 parent 4fd86bc
commit f6eb155
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/src/Vision/DataStructures/Tree/Node.php b/src/Vision/DataStructures/Tree/Node.php
@@ -71,6 +71,8 @@ public function removeChild(NodeInterface $child)
         if ($key !== false) {
             unset($this->children[$key]);
         }
+
+        $this->children = array_values($this->children);
     }
 
     /**

diff --git a/src/Vision/Form/AbstractType.php b/src/Vision/Form/AbstractType.php
@@ -138,9 +138,12 @@ public function hasChildren()
     public function removeChild(NodeInterface $node)
     {
         $key = array_search($node, $this->elements, true);
+
         if ($key !== false) {
             unset($this->elements[$key]);
         }
+
+        $this->elements = array_values($this->elements);
     }
 
     /**
-Original file line number
+Diff line change
@@ Expand Up / @@ -71,6 +71,8 @@ public function removeChild(NodeInterface $child) @@
             if ($key !== false) {
                 unset($this->children[$key]);
             }
+            $this->children = array_values($this->children);
         }
         /**
@@ Expand Down @@