Add strip_tags option

cocur · Mar 21, 2018 · a68d238 · a68d238
1 parent 91184c0
commit a68d238
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 0 deletions.
diff --git a/src/Bridge/Symfony/Configuration.php b/src/Bridge/Symfony/Configuration.php
@@ -28,6 +28,7 @@ public function getConfigTreeBuilder()
             ->children()
                 ->booleanNode('lowercase')->end()
                 ->booleanNode('trim')->end()
+                ->booleanNode('strip_tags')->end()
                 ->scalarNode('separator')->end()
                 ->scalarNode('regexp')->end()
                 ->arrayNode('rulesets')->prototype('scalar')->end()

diff --git a/src/Slugify.php b/src/Slugify.php
@@ -46,6 +46,7 @@ class Slugify implements SlugifyInterface
         'separator' => '-',
         'lowercase' => true,
         'trim' => true,
+        'strip_tags' => false,
         'rulesets'  => [
             'default',
             // Languages are preferred if they appear later, list is ordered by number of
@@ -111,6 +112,10 @@ public function slugify($string, $options = null)
             $rules = $this->rules;
         }
 
+        $string = ($options['strip_tags'])
+            ? strip_tags($string)
+            : $string;
+
         $string = strtr($string, $rules);
         unset($rules);
 

diff --git a/tests/Bridge/Symfony/ConfigurationTest.php b/tests/Bridge/Symfony/ConfigurationTest.php
@@ -21,6 +21,7 @@ public function testAll()
         $configs = [
             [
                 'lowercase' => true,
+                'strip_tags' => false,
                 'separator' => '_',
                 'regexp' => 'abcd',
                 'rulesets' => ['burmese', 'hindi']
@@ -39,6 +40,15 @@ public function testLowercaseOnlyAcceptsBoolean()
         $this->process($configs);
     }
 
+    /**
+     * @expectedException \Symfony\Component\Config\Definition\Exception\InvalidTypeException
+     */
+    public function testStripTagsOnlyAcceptsBoolean()
+    {
+        $configs = [['strip_tags' => 'abc']];
+        $this->process($configs);
+    }
+
     /**
      * Processes an array of configurations and returns a compiled version.
      *

diff --git a/tests/SlugifyTest.php b/tests/SlugifyTest.php
@@ -211,6 +211,10 @@ public function slugifyOptionsArray()
 
         $this->assertEquals('file-name', $this->slugify->slugify('file name '));
         $this->assertEquals('file-name-', $this->slugify->slugify('file name ', ['trim' => false]));
+
+        $this->assertEquals('file-name', $this->slugify->slugify('<file name'));
+        $this->assertEquals('p-file-p-foo-a-href-bar-name-a', $this->slugify->slugify('<p>file</p><!-- foo --> <a href="#bar">name</a>'));
+        $this->assertEquals('file-name', $this->slugify->slugify('<p>file</p><!-- foo --> <a href="#bar">name</a>', ['strip_tags' => true]));
     }
 
     /**