Changeset 6173 in josm for trunk/src/org/openstreetmap/josm/command/MoveCommand.java

Timestamp:

2013-08-22T00:33:32+02:00 (11 years ago)

Author:

Don-vip

Message:

fix #8987 - immutable coordinates (patch by shinigami)

File:

• trunk/src/org/openstreetmap/josm/command/MoveCommand.java (modified) (7 diffs)

trunk/src/org/openstreetmap/josm/command/MoveCommand.java

@@ -9 +9 @@
 import java.util.LinkedList;
 import java.util.List;
+
 import javax.swing.Icon;
 
@@ -48 +49 @@
 
     /**
-     * Small helper for holding the interesting part of the old data state of the
-     * objects.
-     */
-    public static class OldState {
-        LatLon latlon;
-        EastNorth en; // cached EastNorth to be used for applying exact displacenment
-        boolean modified;
-    }
-
-    /**
      * List of all old states of the objects.
      */
-    private List<OldState> oldState = new LinkedList<OldState>();
+    private List<OldNodeState> oldState = new LinkedList<OldNodeState>();
 
     public MoveCommand(OsmPrimitive osm, double x, double y) {
@@ -85 +76 @@
         this.nodes = AllNodesVisitor.getAllNodes(objects);
         for (Node n : this.nodes) {
-            OldState os = new OldState();
-            os.latlon = new LatLon(n.getCoor());
-            os.en = n.getEastNorth();
-            os.modified = n.isModified();
-            oldState.add(os);
-        }
-    }
-
-     public MoveCommand(Collection<OsmPrimitive> objects, EastNorth start, EastNorth end) {
-         this(objects, end.getX()-start.getX(), end.getY()-start.getY());
-         startEN =  start;
-     }
-
-     public MoveCommand(OsmPrimitive p, EastNorth start, EastNorth end) {
-         this(Collections.singleton(p), end.getX()-start.getX(), end.getY()-start.getY());
-         startEN =  start;
-     }
+            oldState.add(new OldNodeState(n));
+        }
+    }
+
+    public MoveCommand(Collection<OsmPrimitive> objects, EastNorth start, EastNorth end) {
+        this(objects, end.getX()-start.getX(), end.getY()-start.getY());
+        startEN =  start;
+    }
+
+    public MoveCommand(OsmPrimitive p, EastNorth start, EastNorth end) {
+        this(Collections.singleton(p), end.getX()-start.getX(), end.getY()-start.getY());
+        startEN =  start;
+    }
 
     /**
@@ -135 +122 @@
     }
 
-     /**
+    /**
      * Changes base point of movement
      * @param newDraggedStartPoint - new starting point after movement (where user clicks to start new drag)
@@ -141 +128 @@
     public void changeStartPoint(EastNorth newDraggedStartPoint) {
         startEN = new EastNorth(newDraggedStartPoint.getX()-x, newDraggedStartPoint.getY()-y);
-     }
+    }
 
     /**
@@ -161 +148 @@
 
     private void updateCoordinates() {
-        Iterator<OldState> it = oldState.iterator();
-        for (Node n : nodes) {
-            OldState os = it.next();
-            n.setEastNorth(os.en.add(x, y));
+        Iterator<OldNodeState> it = oldState.iterator();
+        for (Node n : nodes) {
+            OldNodeState os = it.next();
+            n.setEastNorth(os.eastNorth.add(x, y));
         }
     }
@@ -183 +170 @@
 
     @Override public void undoCommand() {
-        Iterator<OldState> it = oldState.iterator();
-        for (Node n : nodes) {
-            OldState os = it.next();
+        Iterator<OldNodeState> it = oldState.iterator();
+        for (Node n : nodes) {
+            OldNodeState os = it.next();
             n.setCoor(os.latlon);
             n.setModified(os.modified);