Handle nodetype XML_COMMENT_NODE

Templates may contain XML-style comment nodes.
These should be handled like text nodes, i.e. just pass them on into the
output. Since they do not contain attributes or any other node
properties that we can handle, they must be treated specially.

Since the code now contains several different branches that depend on
the node type, a switch-case seems a more aesthetic approach compared to
multiple if-else.

Author: Oliver Baumann

src/Compiler.php

@@ -57,17 +57,21 @@ public function convert(): string
 
     public function convertNode(DOMNode $node): DOMNode
     {
-        if ($node->nodeType === XML_TEXT_NODE) {
-            return $node;
-        }
-
-        if ($node->nodeType === XML_ELEMENT_NODE) {
-            //echo "\nElement node found";
-            /** @var DOMElement $node */
-            $this->replaceShowWithIf($node);
-            $this->handleIf($node);
-        } elseif ($node->nodeType === XML_HTML_DOCUMENT_NODE) {
-            $this->logger->warning("Document node found.");
+        switch($node->nodeType) {
+            case XML_TEXT_NODE:
+                $this->logger->debug('Text node found', ['name' => $node->nodeName]);
+                // fall through to next case, because we don't need to handle either of these node-types
+            case XML_COMMENT_NODE:
+                $this->logger->debug('Comment node found', ['name' => $node->nodeName]);
+                return $node;
+            case XML_ELEMENT_NODE:
+                /** @var DOMElement $node */
+                $this->replaceShowWithIf($node);
+                $this->handleIf($node);
+                break;
+            case XML_HTML_DOCUMENT_NODE:
+                $this->logger->warning("Document node found.");
+                break;
         }
 
         $this->stripEventHandlers($node);
@@ -315,4 +319,4 @@ protected function replacePlaceholders(string $string)
 
         return $string;
     }
-}
+}