Merge pull request #468 from stephanebachelier/svg-fragment

Add support for svg fragment

lib/fileprocessor.js

@@ -15,7 +15,7 @@ var _defaultPatterns = {
       'Update the HTML with the new css filenames'
     ],
     [
-      /<img[^\>]*[^\>\S]+src=['"]([^"']+)["']/gm,
+      /<img[^\>]*[^\>\S]+src=['"]([^'"\)#]+)(#.+)?["']/gm,
       'Update the HTML with the new img filenames'
     ],
     [
@@ -65,16 +65,20 @@ var _defaultPatterns = {
       'Update the HTML with the new object filenames'
     ],
     [
-      /<image[^\>]*[^\>\S]+xlink:href=['"]([^"']+)["']/gm,
+      /<image[^\>]*[^\>\S]+xlink:href=['"]([^"'#]+)(#.+)?["']/gm,
       'Update the HTML with the new image filenames for svg xlink:href links'
     ],
     [
-      /<image[^\>]*[^\>\S]+src=['"]([^"']+)["']/gm,
+      /<image[^\>]*[^\>\S]+src=['"]([^'"\)#]+)(#.+)?["']/gm,
       'Update the HTML with the new image filenames for src links'
     ],
     [
       /<(?:img|source)[^\>]*[^\>\S]+srcset=['"]([^"'\s]+)(?:\s\d[mx])["']/gm,
       'Update the HTML with the new image filenames for srcset links'
+    ],
+    [
+      /<(?:use|image)[^\>]*[^\>\S]+xlink:href=['"]([^'"\)#]+)(#.+)?["']/gm,
+      'Update the HTML within the <use> tag when referencing an external url with svg sprites as in svg4everybody'
     ]
   ],
   css: [

test/test-fileprocessor.js

@@ -437,6 +437,30 @@ describe('FileProcessor', function () {
       replaced = fp.replaceWithRevved(content, ['app']);
       assert.equal(replaced, '<source srcset="' + filemapping['app/[email protected]'] + ' 2x" />');
     });
+
+    it('should replace svg src reference with revved version for img tag', function () {
+      var content = '<img src="image.svg#symbol">';
+      var replaced = fp.replaceWithRevved(content, ['app']);
+      assert.equal(replaced, '<img src="' + filemapping['app/image.svg'] + '#symbol">');
+    });
+
+    it('should replace svg src reference with revved version for image tag', function () {
+      var content = '<image src="image.svg#symbol">';
+      var replaced = fp.replaceWithRevved(content, ['app']);
+      assert.equal(replaced, '<image src="' + filemapping['app/image.svg'] + '#symbol">');
+    });
+
+    it('should replace use xlink:href reference with revved version for svg <use> tag', function () {
+      var content = '<svg><use xlink:href="image.svg#symbol"></svg>';
+      var replaced = fp.replaceWithRevved(content, ['app']);
+      assert.equal(replaced, '<svg><use xlink:href="' + filemapping['app/image.svg'] + '#symbol"></svg>');
+    });
+
+    it('should replace image xlink:href reference with revved version for svg <image> tag', function () {
+      var content = '<svg><image xlink:href="image.svg#symbol"></svg>';
+      var replaced = fp.replaceWithRevved(content, ['app']);
+      assert.equal(replaced, '<svg><image xlink:href="' + filemapping['app/image.svg'] + '#symbol"></svg>');
+    });
   });
 
   describe('css type', function () {