From 94cb9faaa27e273d61079f6c0cabb9b6978458d8 Mon Sep 17 00:00:00 2001 From: Quincy Morgan Date: Mon, 18 Feb 2019 10:16:22 -0500 Subject: [PATCH] Don't check for self-intersection in the crossing ways validation --- modules/validations/crossing_ways.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/modules/validations/crossing_ways.js b/modules/validations/crossing_ways.js index 072d2cdd4..a0dcc2bd0 100644 --- a/modules/validations/crossing_ways.js +++ b/modules/validations/crossing_ways.js @@ -247,10 +247,13 @@ export function validationCrossingWays() { var intersected = tree.intersects(extent, graph); for (var j = 0; j < intersected.length; j++) { - if (intersected[j].type !== 'way') continue; + var way = intersected[j]; + if (way.type !== 'way') continue; + + // don't check for self-intersection in this validation + if (way.id === primaryWay.id) continue; // only check crossing highway, waterway, building, and railway - var way = intersected[j]; var wayFeatureType = getFeatureTypeForCrossingCheck(way, graph); if (wayFeatureType === null || isLegitCrossing(primaryWay, primaryFeatureType, way, wayFeatureType, graph) ||