Changeset 4539 in josm for trunk/src/org/openstreetmap/josm/actions

Timestamp:

2011-10-23T19:41:25+02:00 (13 years ago)

Author:

bastiK

Message:

applied #6853 - In draw mode, select the way to be extended immediately (patch by olejorgenb)

File:

• trunk/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java

@@ -53 +53 @@
 import org.openstreetmap.josm.tools.Pair;
 import org.openstreetmap.josm.tools.Shortcut;
+import org.openstreetmap.josm.tools.Utils;
 
 /**
- *
+ * Mapmode to add nodes, create and extend ways.
  */
 public class DrawAction extends MapMode implements MapViewPaintable, SelectionChangedListener, AWTEventListener {
@@ -98 +99 @@
         updateStatusLine();
         if ((!drawHelperLine || wayIsFinished) && !drawTargetHighlight) return;
+        // update selection to reflect which way being modified
+        if (currentBaseNode != null && getCurrentDataSet().getSelected().isEmpty() == false) {
+            Way continueFrom = getWayForNode(currentBaseNode);
+            if (alt && continueFrom != null) {
+                getCurrentDataSet().beginUpdate(); // to prevent the selection listener to screw around with the state
+                getCurrentDataSet().addSelected(currentBaseNode);
+                getCurrentDataSet().clearSelection(continueFrom);
+                getCurrentDataSet().endUpdate();
+            } else if (!alt && continueFrom != null) {
+                getCurrentDataSet().addSelected(continueFrom);
+            }
+        }
         Main.map.mapView.repaint();
     }
@@ -294 +307 @@
                 // (this is just a convenience option so that people don't
                 // have to switch modes)
-                newSelection.clear();
-                newSelection.add(n);
+
                 getCurrentDataSet().setSelected(n);
+                // If we extend/continue an existing way, select it already now to make it obvious
+                Way continueFrom = getWayForNode(n);
+                if (continueFrom != null) {
+                    getCurrentDataSet().addSelected(continueFrom);
+                }
+
                 // The user explicitly selected a node, so let him continue drawing
                 wayIsFinished = false;
@@ -718 +736 @@
     public Way getWayForNode(Node n) {
         Way way = null;
-        for (Way w : OsmPrimitive.getFilteredList(n.getReferrers(), Way.class)) {
+        for (Way w : Utils.filteredCollection(n.getReferrers(), Way.class)) {
             if (!w.isUsable() || w.getNodesCount() < 1) {
                 continue;