Fixed parsing, CDATA should not be altered when cleanupInput is set to false

Author: onlinesid

src/PHPHtmlParser/Dom.php

@@ -623,7 +623,13 @@ protected function parse(): void
         $this->root->setHtmlSpecialCharsDecode($this->options->htmlSpecialCharsDecode);
         $activeNode = $this->root;
         while ( ! is_null($activeNode)) {
-            $str = $this->content->copyUntil('<');
+            if ($activeNode && $activeNode->tag->name() === 'script'
+                && $this->options->get('cleanupInput') != true
+            ) {
+                $str = $this->content->copyUntil('</');
+            } else {
+                $str = $this->content->copyUntil('<');
+            }
             if ($str == '') {
                 $info = $this->parseTag();
                 if ( ! $info['status']) {

tests/DomTest.php

@@ -12,6 +12,18 @@ public function tearDown()
         Mockery::close();
     }
 
+    /**
+     * <![CDATA[ should not be modified when cleanupInput is set to false
+     */
+    public function testParsingCData()
+    {
+        $html = "<script type=\"text/javascript\">/* <![CDATA[ */var et_core_api_spam_recaptcha = '';/* ]]> */</script>";
+        $dom = new Dom();
+        $dom->setOptions(['cleanupInput' => false,]);
+        $dom->load($html);
+        $this->assertSame($html, $dom->root->outerHtml());
+    }
+
     public function testLoad()
     {
         $dom = new Dom;