Changeset 19001 in josm for trunk

Timestamp:

2024-02-29T07:51:49+01:00 (10 months ago)

Author:

GerdP

Message:

fix #23517: "Tools->Create multipolygon" sorts inner before outer

• reorder members so that all outer rings come first and for rings with same role those with more members come first (before calling RelationSorter.sortMembersByConnectivity())

File:

• trunk/src/org/openstreetmap/josm/data/validation/tests/MultipolygonTest.java (modified) (7 diffs)

trunk/src/org/openstreetmap/josm/data/validation/tests/MultipolygonTest.java

@@ -46 +46 @@
  */
 public class MultipolygonTest extends Test {
+
+    private static final String OUTER = "outer";
+    private static final String INNER = "inner";
 
     /** Non-Way in multipolygon */
@@ -445 +448 @@
         }
         if (r == createdRelation) {
+            // see #23517: sort rings so that outer come first
+            list.sort((r1, r2) -> {
+                // outer first
+                int d = Integer.compare(r1.level % 2, r2.level % 2);
+                if (d != 0)
+                    return d;
+                // ring with more members first
+                return Integer.compare(r2.outerWay.getWayIds().size(), r1.outerWay.getWayIds().size());
+            });
             List<RelationMember> modMembers = new ArrayList<>();
             for (PolygonLevel pol : list) {
-                final String calculatedRole = (pol.level % 2 == 0) ? "outer" : "inner";
+                final String calculatedRole = (pol.level % 2 == 0) ? OUTER : INNER;
                 for (long wayId : pol.outerWay.getWayIds()) {
                     RelationMember member = wayMap.get(wayId);
@@ -457 +469 @@
         }
         for (PolygonLevel pol : list) {
-            final String calculatedRole = (pol.level % 2 == 0) ? "outer" : "inner";
+            final String calculatedRole = (pol.level % 2 == 0) ? OUTER : INNER;
             for (long wayId : pol.outerWay.getWayIds()) {
                 RelationMember member = wayMap.get(wayId);
@@ -469 +481 @@
                             .highlight(member.getMember())
                             .build());
-                    if (pol.level == 0 && "inner".equals(member.getRole())) {
+                    if (pol.level == 0 && INNER.equals(member.getRole())) {
                         // maybe only add this error if we found an outer ring with correct role(s) ?
                         errors.add(TestError.builder(this, Severity.ERROR, INNER_WAY_OUTSIDE)
@@ -570 +582 @@
     private void findIntersectingWaysIncomplete(Relation r) {
         Set<OsmPrimitive> outerWays = r.getMembers().stream()
-                .filter(m -> m.getRole().isEmpty() || "outer".equals(m.getRole()))
+                .filter(m -> m.getRole().isEmpty() || OUTER.equals(m.getRole()))
                 .map(RelationMember::getMember)
                 .collect(Collectors.toSet());
@@ -644 +656 @@
         for (RelationMember rm : r.getMembers()) {
             if (rm.isWay()) {
-                if (rm.hasRole() && !rm.hasRole("inner", "outer"))
+                if (rm.hasRole() && !rm.hasRole(INNER, OUTER))
                     hasUnexpectedWayRole = true;
-                if (!rm.hasRole("inner", "outer") || !rm.hasRole()) {
+                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"))
@@ -906 +918 @@
             createdRelation = null; // makes sure that repeatCheck is only set once
         } while (repeatCheck);
-        errors.removeIf(e->e.getSeverity() == Severity.OTHER);
+        errors.removeIf(e -> e.getSeverity() == Severity.OTHER);
         return r;
     }