Fix: stripTags strips things that aren't tags

Regex borrowed and modified from: https://github.com/kvz/phpjs/blob/master/functions/strings/strip_tags.js
esamattis · Jan 13, 2016 · f620c5c · f620c5c
1 parent c1f7b1f
commit f620c5c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/stripTags.js b/stripTags.js
@@ -1,5 +1,6 @@
 var makeString = require('./helper/makeString');
+var tagsAndComments = /<\/?([a-z][a-z0-9]*)\b[^>]*>|<!--[\s\S]*?-->/gi;
 
 module.exports = function stripTags(str) {
-  return makeString(str).replace(/<\/?[^>]+>/g, '');
+  return makeString(str).replace(tagsAndComments, '');
 };
diff --git a/tests/stripTags.js b/tests/stripTags.js
@@ -6,6 +6,7 @@ test('#stripTags', function() {
   equal(stripTags('a <a href="#">link</a>'), 'a link');
   equal(stripTags('a <a href="#">link</a><script>alert("hello world!")</scr'+'ipt>'), 'a linkalert("hello world!")');
   equal(stripTags('<html><body>hello world</body></html>'), 'hello world');
+  equal(stripTags('I have < I want, but that is > nothing'), 'I have < I want, but that is > nothing');
   equal(stripTags(123), '123');
   equal(stripTags(''), '');
   equal(stripTags(null), '');