Merge pull request #15 from dmnsgn/master

Scrum · web-flow · commit 9c735fed039c · 2020-05-29T13:14:52.000+03:00
fix: handle classIgnore for img with no class attribute
diff --git a/lib/index.js b/lib/index.js
@@ -20,7 +20,7 @@ module.exports = function (options) {
   return function posthtmlWebp (tree) {
     tree.match([{ tag: 'img' }, { tag: 'amp-img' }], function (imgNode) {
       if (imgNode.skip) return imgNode
-      var classes = imgNode.attrs && imgNode.attrs.class && (imgNode.attrs.class.split(' ') || [])
+      var classes = (imgNode.attrs && imgNode.attrs.class && imgNode.attrs.class.split(' ')) || []
       var extension = imgNode.attrs.src.split('.').pop()
       var isIgnoredByClass = options.classIgnore.filter(className => classes.includes(className)).length > 0
       var isIgnoredByExtension = options.extensionIgnore.filter(fileExtension => fileExtension === extension).length > 0
diff --git a/test/fixtures/ignore.expected.html b/test/fixtures/ignore.expected.html
@@ -2,5 +2,6 @@
 <html>
     <body>
         <img class="ignore-class" srcset="photo-lg.jpg 1000w, photo-md.jpg 500w, photo-sm.jpg 250w" src="photo.png">
+        <picture><source type="image/webp" srcset="photo.png.webp"><img src="photo.png"></picture>
     </body>
 </html>
diff --git a/test/fixtures/ignore.html b/test/fixtures/ignore.html
@@ -2,5 +2,6 @@
 <html>
     <body>
         <img class="ignore-class" srcset="photo-lg.jpg 1000w, photo-md.jpg 500w, photo-sm.jpg 250w" src="photo.png">
+        <img src="photo.png">
     </body>
 </html>
