skip non-way members, skip incompleted ways, prevent global modification of Document with way

Author: deevroman

src/building.js

@@ -198,7 +198,11 @@ class Building {
       for (let i = 0; i < parts.length; i++) {
         if (parts[i].querySelector('[k="building:part"]')) {
           const id = parts[i].getAttribute('id');
-          this.parts.push(new MultiBuildingPart(id, this.fullXmlData, this.nodelist, this.outerElement.options));
+          try {
+            this.parts.push(new MultiBuildingPart(id, this.fullXmlData, this.nodelist, this.outerElement.options));
+          } catch (e) {
+            window.printError(e);
+          }
         }
       }
     }

src/multibuildingpart.js

@@ -14,8 +14,8 @@ class MultiBuildingPart extends BuildingPart {
    */
   buildShape() {
     this.type = 'multipolygon';
-    const innerMembers = this.way.querySelectorAll('member[role="inner"]');
-    const outerMembers = this.way.querySelectorAll('member[role="outer"]');
+    const innerMembers = this.way.querySelectorAll('member[role="inner"][type="way"]');
+    const outerMembers = this.way.querySelectorAll('member[role="outer"][type="way"]');
     const innerShapes = [];
     var shapes = [];
     for (let i = 0; i < innerMembers.length; i++) {
@@ -25,7 +25,10 @@ class MultiBuildingPart extends BuildingPart {
     const ways = [];
     for (let j = 0; j < outerMembers.length; j++) {
       const way = this.fullXmlData.getElementById(outerMembers[j].getAttribute('ref'));
-      ways.push(way);
+      if (way === null) {
+        throw `Incompleted way ${outerMembers[j].getAttribute('ref')}`;
+      }
+      ways.push(way.cloneNode(true));
     }
     const closedWays = BuildingShapeUtils.combineWays(ways);
     for (let k = 0; k < closedWays.length; k++) {

test/multipolygon.test.js

@@ -16,7 +16,7 @@ const data = `
   <node id="6" lat="4.001" lon="4.001"/>
   <node id="7" lat="4.001" lon="4"/>
   <relation id="4">
-    <member ref="1" role="outer"/>
+    <member ref="1" role="outer" type="way"/>
     <tag k="type" v="multipolygon"/>
     <tag k="building" v="yes"/>
     <tag k="roof:shape" v="skillion"/>

test/split_way_multipolygon.test.js

@@ -19,9 +19,9 @@ const data = `
   <node id="9" lat="4.00025" lon="4.00075"/>
   <node id="10" lat="4.00075" lon="4.0005"/>
   <relation id="5">
-    <member ref="1" role="outer"/>
-    <member ref="3" role="outer"/>
-    <member ref="2" role="inner"/>
+    <member ref="1" role="outer" type="way"/>
+    <member ref="3" role="outer" type="way"/>
+    <member ref="2" role="inner" type="way"/>
     <tag k="type" v="multipolygon"/>
     <tag k="building" v="yes"/>
     <tag k="roof:shape" v="skillion"/>

test/split_way_multipolygon_reverse.test.js

@@ -19,9 +19,9 @@ const data = `
   <node id="9" lat="4.00025" lon="4.00075"/>
   <node id="10" lat="4.00075" lon="4.0005"/>
   <relation id="5">
-    <member ref="1" role="outer"/>
-    <member ref="3" role="outer"/>
-    <member ref="2" role="inner"/>
+    <member ref="1" role="outer" type="way"/>
+    <member ref="3" role="outer" type="way"/>
+    <member ref="2" role="inner" type="way"/>
     <tag k="type" v="multipolygon"/>
     <tag k="building" v="yes"/>
     <tag k="roof:shape" v="skillion"/>