clean-html - npm Package Compare versions

Comparing version 1.4.3 to 1.5.0

index.js

@@ -175,9 +175,17 @@ var htmlparser = require('htmlparser2'),
 
-    if (options['remove-empty-tags'].indexOf(node.name) != -1) {
+    if (isListedInOptions('remove-empty-tags', node.name)) {
         return isEmpty(node);
     }
 
-    return options['remove-tags'].indexOf(node.name) != -1;
+    return isListedInOptions('remove-tags', node.name);
 }
 
+function isListedInOptions(optionsArrayName, name) {
+    var matches = options[optionsArrayName].filter(function(option) {
+        return option instanceof RegExp && option.test(name) || option === name;
+    });
+
+    return !!matches.length;
+}
+
 function renderText(node) {
@@ -235,3 +243,3 @@ if (shouldRemove(node)) {
     for (var attrib in node.attribs) {
-        if (options['remove-attributes'].indexOf(attrib) == -1) {
+        if (!isListedInOptions('remove-attributes', attrib)) {
             openTag += ' ' + attrib + '="' + removeExtraSpace(node.attribs[attrib]) + '"';
@@ -238,0 +246,0 @@ }

package.json

 {
   "name": "clean-html",
-  "version": "1.4.3",
+  "version": "1.5.0",
   "description": "HTML cleaner and beautifier",
@@ -5,0 +5,0 @@ "main": "index.js",

readme.md

@@ -120,3 +120,3 @@ ## HTML cleaner and beautifier
 
-Type: Array  
+Type: Mixed Array (strings or RegExp pattern)  
 Default: `['align', 'bgcolor', 'border', 'cellpadding', 'cellspacing', 'color', 'height', 'target', 'valign', 'width']`
@@ -135,3 +135,3 @@
 
-Type: Array  
+Type: Mixed Array (strings or RegExp pattern)  
 Default: `[]`
@@ -143,3 +143,3 @@
 
-Type: Array  
+Type: Mixed Array (strings or RegExp pattern)  
 Default: `['center', 'font']`
@@ -146,0 +146,0 @@

release-notes.md

@@ -0,1 +1,7 @@
+1.5.0
+-----
+
+Regular expressions are now supported in the remove-attributes,
+remove-empty-tags and remove-tags options.
+
 1.4.3
@@ -2,0 +8,0 @@ -----

test.js

@@ -61,2 +61,6 @@ var assert = require('assert'),
 });
+// test that attribute is removed when it matches at least one pattern included in remove-attributes
+cleaner.clean('<span _test-color="red">foo</span>', {'remove-attributes': [/_test-[a-z0-9-]+/i]}, function (html) {
+    assert.equal(html, '<span>foo</span>');
+});
 
@@ -80,2 +84,6 @@ // test that comment is not removed when remove-comments is false
 });
+// test that empty tag is removed when it matches at least one pattern included in remove-empty-tags
+cleaner.clean('<app-pam-pam-pam></app-pam-pam-pam>', {'remove-empty-tags': [/^app-.*/i]}, function (html) {
+    assert.equal(html, '');
+});
 
@@ -90,2 +98,6 @@ // test that tag is not removed when not included in remove-tags
 });
+// test that tag is removed and child is preserved when it matches at least one pattern included in remove-tags
+cleaner.clean('<app-test>foo</app-test>', {'remove-tags': [/app-.+/]}, function (html) {
+    assert.equal(html, 'foo');
+});
 
@@ -92,0 +104,0 @@ // test that unsupported tags are removed

No alert changes

Improved metrics

Total package byte prevSize

28271

increased by4.46%

Lines of code

605

increased by3.07%

No dependency changes