Changeset 15728 in josm for trunk/src

Timestamp:

2020-01-19T10:54:23+01:00 (5 years ago)

Author:

simon04

Message:

fix #18477 - Split Way: smart way selection

If only one highway/railway/waterway is applicable, use that one.

File:

• trunk/src/org/openstreetmap/josm/actions/SplitWayAction.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/actions/SplitWayAction.java

@@ -16 +16 @@
 import java.util.List;
 import java.util.concurrent.atomic.AtomicInteger;
+import java.util.stream.Collectors;
 
 import javax.swing.DefaultListCellRenderer;
@@ -103 +104 @@
         if (applicableWays.size() > 1) {
              applicableWays.removeIf(w -> selectedNodes.stream().noneMatch(w::isInnerNode));
+        }
+
+        // Smart way selection: if only one highway/railway/waterway is applicable, use that one
+        if (applicableWays.size() > 1) {
+            final List<Way> mainWays = applicableWays.stream()
+                    .filter(w -> w.hasKey("highway", "railway", "waterway"))
+                    .collect(Collectors.toList());
+            if (mainWays.size() == 1) {
+                applicableWays = mainWays;
+            }
         }