Fix: guard type of attributes' value

posthtml · May 15, 2022 · 30c543d · 30c543d
1 parent 1f693d2
commit 30c543d
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 3 deletions.
diff --git a/lib/modules/collapseAttributeWhitespace.es6 b/lib/modules/collapseAttributeWhitespace.es6
@@ -73,6 +73,8 @@ export function onAttrs() {
         const newAttrs = attrs;
 
         Object.entries(attrs).forEach(([attrName, attrValue]) => {
+            if (typeof attrValue !== 'string') return;
+
             if (attributesWithLists.has(attrName)) {
                 const newAttrValue = attrValue.replace(/\s+/g, ' ').trim();
                 newAttrs[attrName] = newAttrValue;

diff --git a/lib/modules/deduplicateAttributeValues.es6 b/lib/modules/deduplicateAttributeValues.es6
@@ -9,6 +9,10 @@ export function onAttrs() {
                 return;
             }
 
+            if (typeof attrs[attrName] !== 'string') {
+                return;
+            }
+
             const attrValues = attrs[attrName].split(/\s/);
             const uniqeAttrValues = new Set();
             const deduplicatedAttrValues = [];

diff --git a/lib/modules/normalizeAttributeValues.es6 b/lib/modules/normalizeAttributeValues.es6
@@ -123,7 +123,7 @@ export function onAttrs() {
                     || caseInsensitiveAttributes[attrName].includes(node.tag)
                 )
             ) {
-                newAttrValue = attrValue.toLowerCase ? attrValue.toLowerCase() : attrValue;
+                newAttrValue = typeof attrValue.toLowerCase === 'function' ? attrValue.toLowerCase() : attrValue;
             }
 
             if (

diff --git a/lib/modules/removeEmptyAttributes.es6 b/lib/modules/removeEmptyAttributes.es6
@@ -108,8 +108,10 @@ export function onAttrs() {
                     )
                 )
             ) {
-                if (attrValue === '' || (attrValue || '').match(/^\s+$/)) {
-                    delete newAttrs[attrName];
+                if (typeof attrValue === 'string') {
+                    if (attrValue === '' || attrValue.trim() === '') {
+                        delete newAttrs[attrName];
+                    }
                 }
             }
         });