Replace iterating over Object.keys(NodeList) with NodeList.length (…

…#865)

* Replace iterating over Object.keys(NodeList) with NodeList.length

This seemingly fixes issues with xmldom versions greater than 0.8.8 & up to 0.8.10

* Unpin @xmldom/xmldom

* Update shape.js

---------

Co-authored-by: XhmikosR <xhmikosr@gmail.com>

lib/svg-sprite/shape.js

@@ -247,7 +247,7 @@ SVGShape.prototype._stripInlineNamespaceDeclarations = function(element, nsMap)
     }
   }
 
-  for (const index of Object.keys(element.childNodes)) {
+  for (let index = 0; index < element.childNodes.length; index++) {
     const child = element.childNodes.item(index);
     if (child.nodeType === 1) {
       this._stripInlineNamespaceDeclarations(child, parentNsMap);
@@ -406,7 +406,7 @@ SVGShape.prototype._initSVG = function() {
   const children = this.dom.documentElement.childNodes;
   const meta = { title: 'title', description: 'desc' };
 
-  for (const child of Object.keys(children)) {
+  for (let child = 0; child < children.length; child++) {
     for (const [m, value] of Object.entries(meta)) {
       if (value === children.item(child).localName) {
         this[m] = children.item(child);

package.json

@@ -35,7 +35,7 @@
   },
   "dependencies": {
     "@resvg/resvg-js": "^2.6.0",
-    "@xmldom/xmldom": "0.8.8",
+    "@xmldom/xmldom": "^0.8.10",
     "async": "^3.2.5",
     "css-selector-parser": "^1.4.1",
     "csso": "^5.0.5",