Changeset 3555 in josm for trunk

Timestamp:

2010-09-22T23:03:40+02:00 (14 years ago)

Author:

bastiK

Message:

applied #5480 (patch by Christian Müller) - some cleanup in selectaction and fix a minor cosmetic bug

File:

• trunk/src/org/openstreetmap/josm/actions/mapmode/SelectAction.java (modified) (8 diffs)

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

@@ -275 +275 @@
 
         MapView c = Main.map.mapView;
-
         Collection<OsmPrimitive> sel = getCurrentDataSet().getSelected();
+
+        // take nearest node
         OsmPrimitive osm = c.getNearestNode(p, OsmPrimitive.isSelectablePredicate);
 
@@ -282 +283 @@
             for (Node n : c.getNearestNodes(p, OsmPrimitive.isSelectablePredicate)) {
                 if (sel.contains(n)) {
+                    // take nearest selected node
                     osm = n;
+                    break;
                 }
             }
@@ -290 +293 @@
 
             Collection<WaySegment> virtualWaysInSel = new ArrayList<WaySegment>();
-            osm = c.getNearestWay(p, OsmPrimitive.isSelectablePredicate);
-
-            for(WaySegment nearestWS : c.getNearestWaySegments(p, OsmPrimitive.isSelectablePredicate)) {
+            Collection<WaySegment> wss = c.getNearestWaySegments(p, OsmPrimitive.isSelectablePredicate);
+            for(WaySegment nearestWS : wss) {
                 if (nearestWS == null) {
                     continue;
                 }
-                if (sel.contains(w = nearestWS.way)) {
+
+                w = nearestWS.way;
+                if (osm == null && sel.contains(w)) {
+                    // take nearest selected way
                     osm = w;
                 }
@@ -330 +335 @@
 
             if (virtualNode != null) {
+                // insert virtualNode into all segments if nothing was selected,
+                // else only into the (previously) selected segments
                 virtualWays = virtualWaysInSel.isEmpty() ? virtualWays : virtualWaysInSel;
-                osm = virtualWays.iterator().next().way;
+            }
+
+            if (osm == null && !wss.isEmpty()) {
+                // take nearest way
+                osm = wss.iterator().next().way;
             }
         }
@@ -435 +446 @@
             boolean shift = (e.getModifiers() & ActionEvent.SHIFT_MASK) != 0;
             boolean alt = (e.getModifiers() & (ActionEvent.ALT_MASK|InputEvent.ALT_GRAPH_MASK)) != 0
-                            || Main.pref.getBoolean("selectaction.rotates", false);
+                            || Main.pref.getBoolean("selectaction.cycles.multiple.matches", false);
 
             virtualWays.clear();
@@ -444 +455 @@
                 if (!c.isEmpty() && alt) {
                     if (c.iterator().next() instanceof Node) {
-                        // there is at least one node under the cursor:
-                        //   - make sure (first element of new list) equals (result of getNearestCollection)
-                        //   - do not consider ways at all, but all nearest nodes
+                        // consider all nearest nodes
                         c = new ArrayList<OsmPrimitive>(Main.map.mapView.getNearestNodes(e.getPoint(), OsmPrimitive.isSelectablePredicate));
                     } else {
-                        // consider all ways..
+                        // consider all nearest primitives (should be only ways at this point..)
                         c = Main.map.mapView.getAllNearest(e.getPoint(), OsmPrimitive.isSelectablePredicate);
                     }
@@ -536 +545 @@
             boolean ctrl, boolean released, boolean area) {
         DataSet ds = getCurrentDataSet();
+
+        // decides on mousePressed whether
+        //      to cycle on mouseReleased (selList already selected)
+        //      or not                    (selList is a new selection)
+        selMorePrims = (released || area) ? selMorePrims : ds.getSelected().containsAll(selectionList);
+
+        // not allowed together: do not change dataset selection, return early
         if ((shift && ctrl) || (ctrl && !released) || (!virtualWays.isEmpty()))
-            return; // not allowed together
+            return;
 
         // toggle through possible objects on mouse release
@@ -589 +605 @@
         }
 
-        // hard-wiring to false due to performance reasons, should do w/out
-        selMorePrims = (released || area) ? false : ds.getSelected().containsAll(selectionList);
-
         if (ctrl) {
             // Ctrl on an item toggles its selection status,