Don’t choke on valueless event attributes

diff --git a/src/htmlminifier.js b/src/htmlminifier.js
index abae81c..cb24628 100644 (file)
--- a/src/htmlminifier.js
+++ b/src/htmlminifier.js
@@ -186,7 +186,7 @@
   }
 
   function cleanAttributeValue(tag, attrName, attrValue, options, attrs) {
-    if (isEventAttribute(attrName)) {
+    if (attrValue && isEventAttribute(attrName)) {
       attrValue = trimWhitespace(attrValue).replace(/^javascript:\s*/i, '').replace(/\s*;$/, '');
       if (options.minifyJS) {
         var wrappedCode = '(function(){' + attrValue + '})()';

diff --git a/tests/minifier.js b/tests/minifier.js
index c64b556..efb61ef 100644 (file)
--- a/tests/minifier.js
+++ b/tests/minifier.js
@@ -54,6 +54,8 @@
 
     // https://github.com/kangax/html-minifier/issues/169
     equal(minify('<a href>ok</a>'), '<a href>ok</a>');
+
+    equal(minify('<a onclick></a>'), '<a onclick></a>');
   });
 
   test('`minifiy` exists', function() {