CalvinBackup/iD
1
0
Fork 0
mirror of https://github.com/FoggedLens/iD.git synced 2026-05-24 17:14:04 +02:00
Code Issues Packages Projects Releases Wiki Activity

Don't consider untagged multipolygons as old multipolygons

Browse Source 
Also add a lot of old-style multipolygon tests
(closes #4009)
This commit is contained in:
Bryan Housel
2017-05-04 12:16:04 -04:00
parent 6f66cf84bc
commit 89013627fb
3 changed files with 116 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
modules/osm/multipolygon.js
+9 -2
View File
@@ -6,7 +6,7 @@ import { osmIsInterestingTag } from './tags';
// For fixing up rendering of multipolygons with tags on the outer member.
// https://github.com/openstreetmap/iD/issues/613
export function osmIsSimpleMultipolygonOuterMember(entity, graph) {
if (entity.type !== 'way')
if (entity.type !== 'way' || Object.keys(entity.tags).filter(osmIsInterestingTag).length === 0)
return false;
var parents = graph.parentRelations(entity);
@@ -52,7 +52,14 @@ export function osmSimpleMultipolygonOuterMember(entity, graph) {
}
}
return outerMember && graph.hasEntity(outerMember.id);
if (!outerMember)
return false;
var outerEntity = graph.hasEntity(outerMember.id);
if (!outerEntity || !Object.keys(outerEntity.tags).filter(osmIsInterestingTag).length)
return false;
return outerEntity;
}