Index: /trunk/data_nodist/multipolygon.osm =================================================================== --- /trunk/data_nodist/multipolygon.osm (revision 14436) +++ /trunk/data_nodist/multipolygon.osm (revision 14437) @@ -1,3749 +1,3749 @@ - + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + - + - - - - - + + + + + - - - - - - - - - - - - - + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - + - + - - - + + + - - - - - - - - - - - + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + - - - - - - - - - - - - - - + + + + + + + + + + + + + + - + - - - - - - - - - + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - + + + + + + + + + + + + + + - - - - - - - + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - + + + + + + + + + - - - - - - - - - - - - - - + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - + - - - - - - - - - + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - + - + - + - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - + + + + + + + + + + + + - - - - - - + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + - - - - - + + + + + - - - - - - + + + + + + - - - - - - + + + + + + - - - - - - + + + + + + - - - - - - - + + + + + + + - - - - + + + + - - - - - - - + + + + + + + - - - - - - + + + + + + - - - - - - - + + + + + + + - - - - + + + + - - - + + + - - - - - - + + + + + + - - - - - - + + + + + + - - - - - - - + + + + + + + - - - - - - - + + + + + + + - - - - - - - + + + + + + + - - - - + + + + - - - - - - - + + + + + + + - - - + + + - - - - - - + + + + + + - - - + + + - - - - - - + + + + + + - - - + + + - - - - + + + + - - - - - - - + + + + + + + - - - - + + + + - - - + + + - - - - - - + + + + + + - - - + + + - - - - - - - + + + + + + + - - - - - - - + + + + + + + - - - - - - - + + + + + + + - - - - + + + + - - - - - - + + + + + + - - - - + + + + - - - + + + - - - + + + - - - - - - + + + + + + - - - + + + - - - - - - + + + + + + - - - + + + - - - - - - + + + + + + - - - - + + + + - - - + + + - - - - - - + + + + + + - - - - - - - + + + + + + + - - - + + + - - - + + + - - - - - - - + + + + + + + - - - + + + - - - - - - + + + + + + - - - - - - + + + + + + - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + - - - - - - - + + + + + + + - - - - + + + + - - - - + + + + - - - - - - - + + + + + + + - - - - - - - + + + + + + + - - - + + + - - - + + + - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + - - - - - - + + + + + + - - - - - - - + + + + + + + - - - - - - - - - - - - + + + + + + + + + + + + - - - - - - - + + + + + + + - - - + + + - - - + + + - - - - - + + + + + - - - - - + + + + + - - - - - - + + + + + + - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + - - - - - - - + + + + + + + - - - + + + - - - - - - + + + + + + - - - + + + - - - - - - + + + + + + - - - + + + - - - - + + + + - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + - - - + + + - - - - + + + + - - - - + + + + - - - - - + + + + + - - - - - - - + + + + + + + - - - - - - + + + + + + - - - - - - - + + + + + + + - - - - - - - - - - - - - - + + + + + + + + + + + + + + - - - - - - + + + + + + - - - - - - - + + + + + + + - - - - - - - - - - - - - - + + + + + + + + + + + + + + - - - - - - + + + + + + - - - - - - + + + + + + - - - - - - - - - - - + + + + + + + + + + + - - - - - - - + + + + + + + - - - + + + - - - + + + - - - - - - - + + + + + + + - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + - - - - + + + + - - - + + + - - - - - - + + + + + + - - - - - - - - - - - - - + + + + + + + + + + + + + - - - - - - + + + + + + - - - - - - - + + + + + + + - - - - - + + + + + - - - - - - - + + + + + + + - - - - - - + + + + + + - - - - - - - - - - - + + + + + + + + + + + - - - - + + + + - - - - - - - + + + + + + + - - - - - - - + + + + + + + - - - - - + + + + + - - - - + + + + - - - + + + - - - + + + - - - - - - + + + + + + - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + - - - - - - - + + + + + + + - - - - - - - + + + + + + + - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + - - - - - - - - - - - + + + + + + + + + + + - - - - - - + + + + + + - - - + + + - - - - + + + + - - - + + + - - - - - + + + + + - - - - + + + + - - - - - + + + + + - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + - - - - + + + + - - - - - - + + + + + + - - - - - - - - - - - - + + + + + + + + + + + + - - - - - - + + + + + + - - - - - - + + + + + + - - - - - - - - - - - - - - + + + + + + + + + + + + + + - - - - + + + + - - - - - - - + + + + + + + - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + - - - - - - + + + + + + - - - + + + - - - + + + - - - - - - - - - - - - - + + + + + + + + + + + + + - - - - - - - - + + + + + + + + - - - - - - - + + + + + + + - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + - - - + + + - - - - - + + + + + - - - - - - + + + + + + - - - - + + + + - - - - - - + + + + + + - - - - - - + + + + + + - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + - - - + + + - - - - + + + + - - - - - - + + + + + + - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + - - - - + + + + - - - + + + - - - + + + - - - - - - - - - - - + + + + + + + + + + + - - - - - - - - + + + + + + + + - - - + + + - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - + + + + + + - - - - + + + + - - - + + + - - - - - - - + + + + + + + - - - - - + + + + + - - - - - - + + + + + + - - - - - + + + + + - - - - - + + + + + - - - - - - - + + + + + + + - - - - - - + + + + + + - - - - - - - + + + + + + + - - - - + + + + - - - - - - - + + + + + + + - - - + + + - - - - - - + + + + + + - - - - - - - + + + + + + + - - - - + + + + - - - + + + - - - + + + - - - - - - - + + + + + + + - - - + + + - - - - - - + + + + + + - - - + + + - - - + + + - - - - - - + + + + + + - - - - - - + + + + + + - - - - - - + + + + + + - - - + + + - - - - - + + + + + - - - - - - + + + + + + - - - - - + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -3751,14 +3751,14 @@ - - - + + + - - - - + + + + @@ -3766,29 +3766,29 @@ - - - - + + + + - - - + + + - - - + + + - - - - + + + + @@ -3796,128 +3796,128 @@ - - - - + + + + - - - - - - + + + + + + - - - + + + - - - - - - - - - - - - + + + + + + + + + + + + - - - - + + + + - - - - + + + + - - - - - - + + + + + + - - - - + + + + - - - + + + - - - + + + - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + - - - - - - - - + + + + + + + + - - - + + + @@ -3925,32 +3925,32 @@ - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + - - - + + + @@ -3958,31 +3958,31 @@ - - - - - - + + + + + + - - - - + + + + - - - + + + - - - + + + @@ -3990,47 +3990,47 @@ - - - + + + - - - - - - + + + + + + - - - - - + + + + + - - - - - - - - + + + + + + + + - - - + + + - + @@ -4038,108 +4038,108 @@ - - - - - - - - - - - - + + + + + + + + + + + + - - - - + + + + - - - + + + - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + - - - - + + + + - - - + + + - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + - - - - + + + + - - - - - - + + + + + + @@ -4147,23 +4147,23 @@ - - - - - - + + + + + + - - - + + + - - - + + + @@ -4171,7 +4171,7 @@ - - - + + + @@ -4179,83 +4179,83 @@ - - - + + + - - - + + + - - - - + + + + - - - - - + + + + + - - - - + + + + - - - - - + + + + + - - - - + + + + - - - + + + - - - - - + + + + + - - - + + + - - - - + + + + @@ -4263,7 +4263,7 @@ - - - + + + @@ -4271,15 +4271,15 @@ - - - - + + + + - - - + + + @@ -4287,8 +4287,8 @@ - - - - + + + + @@ -4296,7 +4296,7 @@ - - - + + + @@ -4304,7 +4304,7 @@ - - - + + + @@ -4312,11 +4312,11 @@ - - - - - - - + + + + + + + @@ -4324,7 +4324,7 @@ - - - + + + @@ -4332,7 +4332,7 @@ - - - + + + @@ -4340,7 +4340,7 @@ - - - + + + @@ -4348,8 +4348,8 @@ - - - - + + + + @@ -4357,8 +4357,8 @@ - - - - + + + + @@ -4366,8 +4366,8 @@ - - - - + + + + @@ -4375,7 +4375,7 @@ - - - + + + @@ -4383,7 +4383,7 @@ - - - + + + @@ -4391,7 +4391,7 @@ - - - + + + @@ -4399,7 +4399,7 @@ - - - + + + @@ -4407,7 +4407,7 @@ - - - + + + @@ -4415,8 +4415,8 @@ - - - - + + + + @@ -4424,10 +4424,10 @@ - - - - - - + + + + + + @@ -4435,9 +4435,9 @@ - - - - - + + + + + @@ -4445,31 +4445,31 @@ - - - - - - + + + + + + - - - - + + + + - - - - - - - - - + + + + + + + + + @@ -4477,8 +4477,8 @@ - - - - + + + + @@ -4486,16 +4486,16 @@ - - - - + + + + - - - - + + + + @@ -4503,7 +4503,7 @@ - - - + + + @@ -4511,5 +4511,5 @@ - + Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/MultipolygonTest.java =================================================================== --- /trunk/src/org/openstreetmap/josm/data/validation/tests/MultipolygonTest.java (revision 14436) +++ /trunk/src/org/openstreetmap/josm/data/validation/tests/MultipolygonTest.java (revision 14437) @@ -54,6 +54,4 @@ /** Multipolygon is not closed */ public static final int NON_CLOSED_WAY = 1603; - /** No outer way for multipolygon */ - public static final int MISSING_OUTER_WAY = 1604; /** Multipolygon inner way is outside */ public static final int INNER_WAY_OUTSIDE = 1605; @@ -133,32 +131,17 @@ public void visit(Relation r) { if (r.isMultipolygon() && r.getMembersCount() > 0) { - checkMembersAndRoles(r); - checkOuterWay(r); + List tmpErrors = new ArrayList<>(30); + boolean hasUnexpectedWayRoles = checkMembersAndRoles(r, tmpErrors); boolean hasRepeatedMembers = checkRepeatedWayMembers(r); // Rest of checks is only for complete multipolygon - if (!hasRepeatedMembers && !r.hasIncompleteMembers()) { + if (!hasUnexpectedWayRoles && !hasRepeatedMembers && !r.hasIncompleteMembers()) { Multipolygon polygon = new Multipolygon(r); checkStyleConsistency(r, polygon); checkGeometryAndRoles(r, polygon); - } - } - } - - /** - * Checks that multipolygon has at least an outer way:
    - *
  • {@link #MISSING_OUTER_WAY}: No outer way for multipolygon
    • - *
- * @param r relation - */ - private void checkOuterWay(Relation r) { - for (RelationMember m : r.getMembers()) { - if (m.isWay() && "outer".equals(m.getRole())) { - return; - } - } - errors.add(TestError.builder(this, Severity.WARNING, MISSING_OUTER_WAY) - .message(r.isBoundary() ? tr("No outer way for boundary") : tr("No outer way for multipolygon")) - .primitives(r) - .build()); + // see #17010: don't report problems twice + tmpErrors.removeIf(e -> e.getCode() == WRONG_MEMBER_ROLE); + } + errors.addAll(tmpErrors); + } } @@ -528,5 +511,5 @@ for (long wayId : pol.outerWay.getWayIds()) { RelationMember member = wayMap.get(wayId); - if (!member.getRole().equals(calculatedRole)) { + if (!calculatedRole.equals(member.getRole())) { errors.add(TestError.builder(this, Severity.ERROR, WRONG_MEMBER_ROLE) .message(RelationChecker.ROLE_VERIF_PROBLEM_MSG, @@ -705,11 +688,16 @@ * * @param r relation - */ - private void checkMembersAndRoles(Relation r) { + * @param tmpErrors list that will contain found errors + * @return true if ways with roles other than inner, outer or empty where found + */ + private boolean checkMembersAndRoles(Relation r, List tmpErrors) { + boolean hasUnexpectedWayRole = false; for (RelationMember rm : r.getMembers()) { if (rm.isWay()) { - if (!(rm.hasRole("inner", "outer") || !rm.hasRole())) { - errors.add(TestError.builder(this, Severity.WARNING, WRONG_MEMBER_ROLE) - .message(tr("No useful role for multipolygon member")) + if (rm.hasRole() && !(rm.hasRole("inner", "outer"))) + hasUnexpectedWayRole = true; + if (!(rm.hasRole("inner", "outer")) || !rm.hasRole()) { + tmpErrors.add(TestError.builder(this, Severity.ERROR, WRONG_MEMBER_ROLE) + .message(tr("Role for multipolygon way member should be inner or outer")) .primitives(Arrays.asList(r, rm.getMember())) .build()); @@ -717,5 +705,5 @@ } else { if (!r.isBoundary() || !rm.hasRole("admin_centre", "label", "subarea", "land_area")) { - errors.add(TestError.builder(this, Severity.WARNING, WRONG_MEMBER_TYPE) + tmpErrors.add(TestError.builder(this, Severity.WARNING, WRONG_MEMBER_TYPE) .message(r.isBoundary() ? tr("Non-Way in boundary") : tr("Non-Way in multipolygon")) .primitives(Arrays.asList(r, rm.getMember())) @@ -724,4 +712,5 @@ } } + return hasUnexpectedWayRole; } @@ -749,4 +738,6 @@ Map> seenMemberPrimitives = new HashMap<>(); for (RelationMember rm : r.getMembers()) { + if (!rm.isWay()) + continue; List list = seenMemberPrimitives.get(rm.getMember()); if (list == null) { Index: /trunk/src/org/openstreetmap/josm/data/validation/tests/RelationChecker.java =================================================================== --- /trunk/src/org/openstreetmap/josm/data/validation/tests/RelationChecker.java (revision 14436) +++ /trunk/src/org/openstreetmap/josm/data/validation/tests/RelationChecker.java (revision 14437) @@ -19,7 +19,9 @@ import org.openstreetmap.josm.data.osm.Relation; import org.openstreetmap.josm.data.osm.RelationMember; +import org.openstreetmap.josm.data.validation.OsmValidator; import org.openstreetmap.josm.data.validation.Severity; import org.openstreetmap.josm.data.validation.Test; import org.openstreetmap.josm.data.validation.TestError; +import org.openstreetmap.josm.gui.progress.ProgressMonitor; import org.openstreetmap.josm.gui.tagging.presets.TaggingPreset; import org.openstreetmap.josm.gui.tagging.presets.TaggingPresetItem; @@ -61,4 +63,5 @@ */ public static final String ROLE_VERIF_PROBLEM_MSG = tr("Role verification problem"); + private boolean ignoreMultiPolygons; /** @@ -100,5 +103,28 @@ @Override + public void startTest(ProgressMonitor progressMonitor) { + super.startTest(progressMonitor); + + for (Test t : OsmValidator.getEnabledTests(false)) { + if (t instanceof MultipolygonTest) { + ignoreMultiPolygons = true; + break; + } + } + } + + @Override public void visit(Relation n) { + Map map = buildRoleInfoMap(n); + if (map.isEmpty()) { + errors.add(TestError.builder(this, Severity.ERROR, RELATION_EMPTY) + .message(tr("Relation is empty")) + .primitives(n) + .build()); + } + if (ignoreMultiPolygons && n.isMultipolygon()) { + // see #17010: don't report same problem twice + return; + } Map allroles = buildAllRoles(n); if (allroles.isEmpty() && n.hasTag("type", "route") @@ -115,11 +141,5 @@ } - Map map = buildRoleInfoMap(n); - if (map.isEmpty()) { - errors.add(TestError.builder(this, Severity.ERROR, RELATION_EMPTY) - .message(tr("Relation is empty")) - .primitives(n) - .build()); - } else if (!allroles.isEmpty()) { + if (!map.isEmpty() && !allroles.isEmpty()) { checkRoles(n, allroles, map); } Index: /trunk/test/unit/org/openstreetmap/josm/data/validation/tests/RelationCheckerTest.java =================================================================== --- /trunk/test/unit/org/openstreetmap/josm/data/validation/tests/RelationCheckerTest.java (revision 14436) +++ /trunk/test/unit/org/openstreetmap/josm/data/validation/tests/RelationCheckerTest.java (revision 14437) @@ -47,5 +47,8 @@ @Test public void testUnknownType() { - List errors = testRelation(createRelation("type=foobar")); + Relation r = createRelation("type=foobar"); + r.addMember(new RelationMember("", new Way())); + List errors = testRelation(r); + assertTrue(errors.size() >= 1); assertEquals("Relation type is unknown", errors.get(0).getMessage());