diff --git a/modules/core/validation/models.js b/modules/core/validation/models.js index 4599a5abc..18be90de2 100644 --- a/modules/core/validation/models.js +++ b/modules/core/validation/models.js @@ -29,19 +29,13 @@ export function validationIssue(attrs) { parts.push(this.subtype); } - // include entities this issue is for + // include the entities this issue is for // (sort them so the id is deterministic) if (this.entityIds) { var entityKeys = this.entityIds.slice().sort(); parts.push.apply(parts, entityKeys); } - // include loc since two separate issues can have an - // idential type and entities, e.g. in crossing_ways - if (this.loc) { - parts.push.apply(parts, this.loc); - } - return parts.join(':'); } diff --git a/modules/validations/crossing_ways.js b/modules/validations/crossing_ways.js index f36edd9bd..edf2f52ba 100644 --- a/modules/validations/crossing_ways.js +++ b/modules/validations/crossing_ways.js @@ -494,6 +494,8 @@ export function validationCrossingWays() { edges: crossing.edges, connectionTags: connectionTags }, + // differentiate based on the loc since two ways can cross multiple times + hash: JSON.stringify(crossing.crossPoint), loc: crossing.crossPoint, fixes: fixes });