Added Smarty flag for paquettg#184

Author: paquettg

CHANGELOG.md

@@ -1,5 +1,15 @@
 ### Development
 
-## 1.2.0
+All notable changes to this project will be documented in this file.
 
+The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
+and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+
+## [Unreleased]
+
+### Added
+- New `removeSmartyScripts` configuration setting. Defaults to true.
+
+### Changed
+- Started using a changelog.
 - Fixed bug that caused an infinite loop when no content found in tags.

README.md

@@ -170,7 +170,12 @@ Preserves Line Breaks if set to `true`. If set to `false` line breaks are cleane
 
 **removeDoubleSpace**
 
-Set this to `false` if you want to preserver whitespace inside of text nodes. It is set to `true` by default.
+Set this to `false` if you want to preserve whitespace inside of text nodes. It is set to `true` by default.
+
+**removeSmartyScripts**
+
+Set this to `false` if you want to preserve smarty sccript found in the html content. It is set to `true` by default.
+
 Static Facade
 -------------
 

src/PHPHtmlParser/Dom.php

@@ -502,24 +502,26 @@ protected function clean(string $str): string
         $str = mb_eregi_replace("<!\[CDATA\[(.*?)\]\]>", '', $str);
 
         // strip out <script> tags
-        if ($this->options->get('removeScripts') == true) {
+        if ($this->options->get('removeScripts')) {
             $str = mb_eregi_replace("<\s*script[^>]*[^/]>(.*?)<\s*/\s*script\s*>", '', $str);
             $str = mb_eregi_replace("<\s*script\s*>(.*?)<\s*/\s*script\s*>", '', $str);
         }
 
         // strip out <style> tags
-        if ($this->options->get('removeStyles') == true) {
+        if ($this->options->get('removeStyles')) {
             $str = mb_eregi_replace("<\s*style[^>]*[^/]>(.*?)<\s*/\s*style\s*>", '', $str);
             $str = mb_eregi_replace("<\s*style\s*>(.*?)<\s*/\s*style\s*>", '', $str);
         }
 
         // strip out server side scripts
-        if ($this->options->get('serverSideScriptis') == true){
+        if ($this->options->get('serverSideScripts')) {
             $str = mb_eregi_replace("(<\?)(.*?)(\?>)", '', $str);
         }
 
         // strip smarty scripts
-        $str = mb_eregi_replace("(\{\w)(.*?)(\})", '', $str);
+        if ($this->options->get('removeSmartyScripts')) {
+            $str = mb_eregi_replace("(\{\w)(.*?)(\})", '', $str);
+        }
 
         return $str;
     }

src/PHPHtmlParser/Options.php

@@ -13,6 +13,7 @@
  * @property bool removeStyles
  * @property bool preserveLineBreaks
  * @property bool removeDoubleSpace
+ * @property bool removeSmartyScripts
  */
 class Options
 {
@@ -23,14 +24,15 @@ class Options
      * @param array
      */
     protected $defaults = [
-        'whitespaceTextNode' => true,
-        'strict'             => false,
-        'enforceEncoding'    => null,
-        'cleanupInput'       => true,
-        'removeScripts'      => true,
-        'removeStyles'       => true,
-        'preserveLineBreaks' => false,
-        'removeDoubleSpace'  => true,
+        'whitespaceTextNode'   => true,
+        'strict'               => false,
+        'enforceEncoding'      => null,
+        'cleanupInput'         => true,
+        'removeScripts'        => true,
+        'removeStyles'         => true,
+        'preserveLineBreaks'   => false,
+        'removeDoubleSpace'    => true,
+        'removeSmartyScripts'  => true,
     ];
 
     /**

tests/Options/CleanupTest.php

@@ -72,4 +72,24 @@ public function testRemoveScriptsFalse()
             $dom->find('script')->getAttribute('type'));
     }
 
+    public function testSmartyScripts()
+    {
+        $dom = new Dom;
+        $dom->load('
+        aa={123}
+        ');
+        $this->assertEquals(' aa= ', $dom->innerHtml);
+    }
+
+    public function testSmartyScriptsDisabled()
+    {
+        $dom = new Dom;
+        $dom->setOptions([
+            'removeSmartyScripts' => false
+        ]);
+        $dom->load('
+        aa={123}
+        ');
+        $this->assertEquals(' aa={123} ', $dom->innerHtml);
+    }
 }