Fix mutable nodeList reference during traversal

Author: veewee

src/Xml/Dom/Traverser/Traverser.php

@@ -6,6 +6,7 @@
 
 use DOMNode;
 use function VeeWee\Xml\Dom\Locator\Attribute\attributes_list;
+use function VeeWee\Xml\Dom\Locator\Node\children;
 
 final class Traverser
 {
@@ -30,7 +31,7 @@ public function traverse(DOMNode $node): DOMNode
             $this->traverse($attribute);
         }
 
-        foreach ($node->childNodes as $child) {
+        foreach (children($node) as $child) {
             $this->traverse($child);
         }
 

tests/Xml/Dom/Traverser/TraverserTest.php

@@ -11,10 +11,12 @@
 use VeeWee\Xml\Dom\Traverser\Traverser;
 use VeeWee\Xml\Dom\Traverser\Visitor\AbstractVisitor;
 use function VeeWee\Xml\Dom\Builder\attribute;
+use function VeeWee\Xml\Dom\Configurator\comparable;
 use function VeeWee\Xml\Dom\Locator\document_element;
 use function VeeWee\Xml\Dom\Mapper\xml_string;
 use function VeeWee\Xml\Dom\Predicate\is_attribute;
 use function VeeWee\Xml\Dom\Predicate\is_element;
+use function VeeWee\Xml\Dom\Predicate\is_empty_node;
 use function VeeWee\Xml\Dom\Predicate\is_non_empty_text;
 use function VeeWee\Xml\Dom\Predicate\is_whitespace;
 
@@ -130,4 +132,50 @@ public function onNodeLeave(DOMNode $node): Action
 
         static::assertXmlStringEqualsXmlString($doc->toXmlString(), '<hello enter="yes" leave="yes" />');
     }
+
+    /** @test */
+    public function it_can_recursively_remove_empty_nodes(): void
+    {
+        $doc = Document::fromXmlString(<<<EOXML
+            <movies>
+                <movie>
+                    <name>Terminator</name>
+                    <genre>
+                        <action />
+                        <romance />
+                    </genre>
+                    <prices>
+                        <oscar />
+                    </prices>
+                </movie>
+            </movies>
+        EOXML
+        );
+
+        $transformedNode = $doc->traverse(new class extends AbstractVisitor {
+            public function onNodeLeave(\DOMNode $node): Action
+            {
+                if (!is_element($node)) {
+                    return new Action\Noop();
+                }
+
+                if (trim($node->textContent) !== '') {
+                    return new Action\Noop();
+                }
+
+                return new Action\RemoveNode();
+            }
+        });
+
+        $actual = Document::fromXmlNode($transformedNode, comparable());
+        $expected = Document::fromXmlString(<<<EOXML
+            <movies>
+                <movie>
+                    <name>Terminator</name>
+                </movie>
+            </movies>
+        EOXML, comparable());
+
+        self::assertSame($actual->toXmlString(), $expected->toXmlString());
+    }
 }