Fix cleanupIDs if defs is last elem of svg

svg · Nov 9, 2019 · 7746480 · 7746480
1 parent 1152257
commit 7746480
Show file tree

Hide file tree

Showing 2 changed files with 46 additions and 3 deletions.
diff --git a/plugins/cleanupIDs.js b/plugins/cleanupIDs.js
@@ -63,11 +63,13 @@ exports.fn = function(data, params) {
                     hasStyleOrScript = true;
                     continue;
                 }
+
                 // Don't remove IDs if the whole SVG consists only of defs.
-                if (item.isElem('defs') && item.parentNode.isElem('svg')) {
+                if (item.isElem('svg')) {
                     var hasDefsOnly = true;
-                    for (var j = i + 1; j < items.content.length; j++) {
-                        if (items.content[j].isElem()) {
+
+                    for (var j = 0; j < item.content.length; j++) {
+                        if (!item.content[j].isElem('defs')) {
                             hasDefsOnly = false;
                             break;
                         }

diff --git a/test/plugins/cleanupIDs.20.svg b/test/plugins/cleanupIDs.20.svg