Add multipassCount to info object passed to the plugins.

Use info object to skip subsequent passes in prefixIds plugin to prevent unintended, multiple prefixing.
svg · Oct 29, 2019 · 71c7fe7 · 71c7fe7
1 parent 67be534
commit 71c7fe7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/lib/svgo.js b/lib/svgo.js
@@ -38,6 +38,7 @@ SVGO.prototype.optimize = function(svgstr, info) {
                     return;
                 }
 
+                info.multipassCount = counter;
                 if (++counter < maxPassCount && svgjs.data.length < prevResultSize) {
                     prevResultSize = svgjs.data.length;
                     this._optimizeOnce(svgjs.data, info, optimizeOnceCallback);

diff --git a/plugins/prefixIds.js b/plugins/prefixIds.js
@@ -125,6 +125,11 @@ var addPrefixToUrlAttr = function(attr) {
  */
 exports.fn = function(node, opts, extra) {
 
+    // skip subsequent passes when multipass is used
+    if(extra.multipassCount && extra.multipassCount > 0) {
+        return node;
+    }
+
     // prefix, from file name or option
     var prefix = 'prefix';
     if (opts.prefix) {