Changeset 1441 in josm for trunk/src/org

Timestamp:

2009-02-24T18:49:06+01:00 (16 years ago)

Author:

stoecker

Message:

fixed #1469. patch by xeen

File:

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

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

@@ -3 +3 @@
 
 import static org.openstreetmap.josm.tools.I18n.tr;
+import static org.openstreetmap.josm.tools.I18n.trn;
 
 import java.awt.Cursor;
@@ -61 +62 @@
     private boolean cancelDrawMode = false;
     Node virtualNode = null;
-    WaySegment virtualWay = null;
+    Collection<WaySegment> virtualWays = new ArrayList<WaySegment>();
     SequenceCommand virtualCmds = null;
 
@@ -147 +148 @@
         if(!Main.map.mapView.isVisibleDrawableLayer())
             return;
-        
+
         cancelDrawMode = true;
         if (mode == Mode.select) return;
@@ -181 +182 @@
             return;
 
-        if (virtualWay != null)    {
+        if (virtualWays.size() > 0) {
             Collection<Command> virtualCmds = new LinkedList<Command>();
             virtualCmds.add(new AddCommand(virtualNode));
-            Way w = virtualWay.way;
-            Way wnew = new Way(w);
-            wnew.addNode(virtualWay.lowerIndex+1, virtualNode);
-            virtualCmds.add(new ChangeCommand(w, wnew));
+            for(WaySegment virtualWay : virtualWays) {
+                Way w = virtualWay.way;
+                Way wnew = new Way(w);
+                wnew.addNode(virtualWay.lowerIndex+1, virtualNode);
+                virtualCmds.add(new ChangeCommand(w, wnew));
+            }
             virtualCmds.add(new MoveCommand(virtualNode, dx, dy));
-            Main.main.undoRedo.add(new SequenceCommand(tr("Add and move a virtual new node to way"), virtualCmds));
+            String text = trn("Add and move a virtual new node to way",
+            "Add and move a virtual new node to {0} ways", virtualWays.size(),
+            virtualWays.size());
+
+            Main.main.undoRedo.add(new SequenceCommand(text, virtualCmds));
             selectPrims(Collections.singleton((OsmPrimitive)virtualNode), false, false);
-            virtualWay = null;
+            virtualWays.clear();
             virtualNode = null;
         } else {
@@ -222 +229 @@
                         JOptionPane.showMessageDialog(Main.parent,
                             tr("Cannot move objects outside of the world."));
+                        restoreCursor();
                         return;
                     }
@@ -239 +247 @@
     }
 
-    private Collection<OsmPrimitive> getNearestCollectionVirtual(Point p) {
+    private Collection<OsmPrimitive> getNearestCollectionVirtual(Point p, boolean allSegements) {
         MapView c = Main.map.mapView;
         int snapDistance = Main.pref.getInteger("mappaint.node.virtual-snap-distance", 8);
         snapDistance *= snapDistance;
         OsmPrimitive osm = c.getNearestNode(p);
-        virtualWay = null;
+        virtualWays.clear();
         virtualNode = null;
+        Node virtualWayNode = null;
 
         if (osm == null)
         {
-            WaySegment nearestWaySeg = c.getNearestWaySegment(p);
-            if (nearestWaySeg != null)
-            {
-                osm = nearestWaySeg.way;
+            Collection<WaySegment> nearestWaySegs = allSegements
+                 ? c.getNearestWaySegments(p)
+                 : Collections.singleton(c.getNearestWaySegment(p));
+
+            for(WaySegment nearestWS : nearestWaySegs) {
+                if (nearestWS == null)
+                    continue;
+
+                osm = nearestWS.way;
                 if(Main.pref.getInteger("mappaint.node.virtual-size", 8) > 0)
                 {
                     Way w = (Way)osm;
-                    Point p1 = c.getPoint(w.nodes.get(nearestWaySeg.lowerIndex).eastNorth);
-                    Point p2 = c.getPoint(w.nodes.get(nearestWaySeg.lowerIndex+1).eastNorth);
+                    Point p1 = c.getPoint(w.nodes.get(nearestWS.lowerIndex).eastNorth);
+                    Point p2 = c.getPoint(w.nodes.get(nearestWS.lowerIndex+1).eastNorth);
                     if(SimplePaintVisitor.isLargeSegment(p1, p2, Main.pref.getInteger("mappaint.node.virtual-space", 70)))
                     {
@@ -263 +277 @@
                         if (p.distanceSq(pc) < snapDistance)
                         {
-                            virtualWay = nearestWaySeg;
-                            virtualNode = new Node(Main.map.mapView.getLatLon(pc.x, pc.y));
-                            osm = w;
+                            // Check that only segments on top of each other get added to the
+                            // virtual ways list. Otherwise ways that coincidentally have their
+                            // virtual node at the same spot will be joined which is likely unwanted
+                            if(virtualWayNode != null) {
+                                if(  !w.nodes.get(nearestWS.lowerIndex+1).equals(virtualWayNode)
+                                  && !w.nodes.get(nearestWS.lowerIndex  ).equals(virtualWayNode))
+                                    continue;
+                            } else
+                                virtualWayNode = w.nodes.get(nearestWS.lowerIndex+1);
+
+                            virtualWays.add(nearestWS);
+                            if(virtualNode == null)
+                                virtualNode = new Node(Main.map.mapView.getLatLon(pc.x, pc.y));
                         }
                     }
@@ -288 +312 @@
         if(!Main.map.mapView.isVisibleDrawableLayer())
             return;
-        
+
         cancelDrawMode = false;
         if (! (Boolean)this.getValue("active")) return;
@@ -294 +318 @@
             return;
         boolean ctrl = (e.getModifiers() & ActionEvent.CTRL_MASK) != 0;
-        // boolean alt = (e.getModifiers() & ActionEvent.ALT_MASK) != 0;
+        boolean alt = (e.getModifiers() & ActionEvent.ALT_MASK) != 0;
         boolean shift = (e.getModifiers() & ActionEvent.SHIFT_MASK) != 0;
-        
+
         // We don't want to change to draw tool if the user tries to (de)select
         // stuff but accidentally clicks in an empty area when selection is empty
@@ -306 +330 @@
         initialMoveThresholdExceeded = false;
 
-        Collection<OsmPrimitive> osmColl = getNearestCollectionVirtual(e.getPoint());
+        Collection<OsmPrimitive> osmColl = getNearestCollectionVirtual(e.getPoint(), alt);
 
         if (ctrl && shift) {
@@ -330 +354 @@
         {
             virtualNode = null;
-            virtualWay = null;
+            virtualWays.clear();
         }
 
         updateStatusLine();
-        Main.map.mapView.repaint();
+        // Mode.select redraws when selectPrims is called
+        // Mode.move   redraws when mouseDragged is called
+        // Mode.rotate redraws here
+        if(mode == Mode.rotate)
+            Main.map.mapView.repaint();
 
         mousePos = e.getPoint();
@@ -345 +373 @@
         if(!Main.map.mapView.isVisibleDrawableLayer())
             return;
-        
+
         if (mode == Mode.select) {
             selectionManager.unregister(Main.map.mapView);
@@ -398 +426 @@
         }
 
-        updateStatusLine();
+        // I don't see why we need this.
+        //updateStatusLine();
         mode = null;
         updateStatusLine();