Updated contributing and changelog

Author: paquettg

Diff for: CHANGELOG.md

@@ -16,6 +16,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 - Fixed issue with \ causing an infite loop.
+- CDATA should not be altered when cleanupInput is false.
+
 
 ### Removed
 - Removed curl interface and curl implementation.

Diff for: CONTRIBUTING.md

@@ -4,4 +4,11 @@ This page contains guidelines for contributing to the PHPHtmlParser package. Ple
 
 ## Coding Guidelines
 
-We follow the [PSR-4](https://www.php-fig.org/psr/psr-4/) autoloading standard and follow the [PSR-12](https://www.php-fig.org/psr/psr-12/) coding style guide. Please ensure you comply to these standards when creating a PR.
+We follow the [PSR-4](https://www.php-fig.org/psr/psr-4/) autoloading standard and follow the [PSR-12](https://www.php-fig.org/psr/psr-12/) coding style guide. To make it easy to comply with the coding standard we use php-cs-fixer to manage the style of the code base. Before pushing your code please ensure you run the following on your changes.
+
+```bash
+./vendor/bin/php-cs-fixer fix
+```
+
+Please ensure you comply to these standards when creating a PR to make it easy to review and merge. 
+Thank you.

Diff for: src/PHPHtmlParser/Dom.php

@@ -520,11 +520,6 @@ protected function clean(string $str): string
             }
         }
 
-        //sometime need predicate an encode is from encoding 
-        if ($this->options->get('useFromEncoding') != NULL) {
-            $str = mb_convert_encoding( $str, "UTF-8", $this->options->get('useFromEncoding'));
-        }
-
         // remove white space before closing tags
         $str = \mb_eregi_replace("'\s+>", "'>", $str);
         if ($str === false) {
@@ -619,7 +614,7 @@ protected function parse(): void
         $this->root = new HtmlNode('root');
         $this->root->setHtmlSpecialCharsDecode($this->options->htmlSpecialCharsDecode);
         $activeNode = $this->root;
-        while ($activeNode!== null) {
+        while ($activeNode !== null) {
             if ($activeNode && $activeNode->tag->name() === 'script'
                 && $this->options->get('cleanupInput') != true
             ) {

Diff for: src/PHPHtmlParser/Dom/AbstractNode.php

@@ -244,6 +244,7 @@ public function getAncestor(int $id)
             if ($this->parent->id() == $id) {
                 return $this->parent;
             }
+
             return $this->parent->getAncestor($id);
         }
     }

Diff for: tests/DomTest.php

@@ -13,13 +13,13 @@ public function tearDown()
     }
 
     /**
-     * <![CDATA[ should not be modified when cleanupInput is set to false
+     * <![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->setOptions(['cleanupInput' => false]);
         $dom->load($html);
         $this->assertSame($html, $dom->root->outerHtml());
     }