Remove removable attrs with unspecified values

diff --git a/src/htmlminifier.js b/src/htmlminifier.js
index 0ec98b5..f98b455 100644 (file)
--- a/src/htmlminifier.js
+++ b/src/htmlminifier.js
@@ -209,7 +209,10 @@
       return trimWhitespace(attrValue);
     }
     else if (attrName === 'style') {
-      attrValue = trimWhitespace(attrValue).replace(/\s*;\s*$/, '');
+      attrValue = trimWhitespace(attrValue);
+      if (attrValue) {
+        attrValue = attrValue.replace(/\s*;\s*$/, '');
+      }
       if (options.minifyCSS) {
         return minifyCSS(attrValue, options.minifyCSS);
       }
@@ -278,7 +281,7 @@
       '?:down|up|over|move|out)|key(?:press|down|up)))$');
 
   function canDeleteEmptyAttribute(tag, attrName, attrValue) {
-    var isValueEmpty = /^(["'])?\s*\1$/.test(attrValue);
+    var isValueEmpty = !attrValue || /^(["'])?\s*\1$/.test(attrValue);
     if (isValueEmpty) {
       return (
         (tag === 'input' && attrName === 'value') ||

diff --git a/tests/minifier.js b/tests/minifier.js
index 93fa1f6..4079e95 100644 (file)
--- a/tests/minifier.js
+++ b/tests/minifier.js
@@ -260,6 +260,11 @@
 
     input = '<img src="" alt="">';
     equal(minify(input, { removeEmptyAttributes: true }), '<img src="" alt="">');
+
+    // preserve unrecognized attribute
+    // remove recognized attrs with unspecified values
+    input = '<div data-foo class id style title lang dir onfocus onblur onchange onclick ondblclick onmousedown onmouseup onmouseover onmousemove onmouseout onkeypress onkeydown onkeyup></div>';
+    equal(minify(input, { removeEmptyAttributes: true }), '<div data-foo></div>');
   });
 
   test('cleaning class/style attributes', function() {