Changeset 312 in josm for src/org/openstreetmap/josm/command

Timestamp:

2007-08-23T15:59:32+02:00 (18 years ago)

Author:

imi

Message:

• fixed bug to allow move of one node
• removed "new shiny beta" warning
• fixed uploading data marks as modified when saved before

Location:

src/org/openstreetmap/josm/command

Files:

• MoveCommand.java (modified) (4 diffs)
• RotateCommand.java (modified) (6 diffs)

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

@@ -5 +5 @@
 import static org.openstreetmap.josm.tools.I18n.trn;
 
-import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
@@ -47 +46 @@
          * Small helper for holding the interesting part of the old data state of the
          * objects. 
-         * @author imi
          */
-        class OldState
-        {
-                double x,y,lat,lon;
+        public static class OldState {
+                LatLon latlon;
+                EastNorth eastNorth;
                 boolean modified;
         }
+        
         /**
          * List of all old states of the objects.
@@ -72 +71 @@
                 for (Node n : this.objects) {
                         OldState os = new OldState();
-                        os.x = n.eastNorth.east();
-                        os.y = n.eastNorth.north();
-                        os.lat = n.coor.lat();
-                        os.lon = n.coor.lon();
+                        os.eastNorth = n.eastNorth;
+                        os.latlon = n.coor;
                         os.modified = n.modified;
                         oldState.add(os);
@@ -110 +107 @@
                 for (Node n : objects) {
                         OldState os = it.next();
-                        n.eastNorth = new EastNorth(os.x, os.y);
-                        n.coor = new LatLon(os.lat, os.lon);
+                        n.eastNorth = os.eastNorth;
+                        n.coor = os.latlon;
                         n.modified = os.modified;
                 }

src/org/openstreetmap/josm/command/RotateCommand.java

@@ -4 +4 @@
 import static org.openstreetmap.josm.tools.I18n.trn;
 
-import java.util.Arrays;
 import java.util.Collection;
 import java.util.HashMap;
-import java.util.Iterator;
 import java.util.LinkedList;
-import java.util.List;
 import java.util.Map;
 
@@ -17 +14 @@
 
 import org.openstreetmap.josm.Main;
-import org.openstreetmap.josm.command.MoveCommand.OldState;
 import org.openstreetmap.josm.data.coor.EastNorth;
 import org.openstreetmap.josm.data.coor.LatLon;
@@ -53 +49 @@
         
         /**
-         * Small helper for holding the interesting part of the old data state of the
-         * objects. 
-         */
-        class OldState
-        {
-                double x,y,lat,lon;
-                boolean modified;
-                Node originalNode;
-        }
-        
-        /**
          * List of all old states of the objects.
          */
-        private Map<Node, OldState> oldState = new HashMap<Node, OldState>();
+        private Map<Node, MoveCommand.OldState> oldState = new HashMap<Node, MoveCommand.OldState>();
         
         /**
@@ -80 +65 @@
                         
                 for (Node n : this.objects) {
-                        OldState os = new OldState();
-                        os.x = n.eastNorth.east();
-                        os.y = n.eastNorth.north();
-                        os.lat = n.coor.lat();
-                        os.lon = n.coor.lon();
+                        MoveCommand.OldState os = new MoveCommand.OldState();
+                        os.eastNorth = n.eastNorth;
+                        os.latlon = n.coor;
                         os.modified = n.modified;
                         oldState.put(n, os);
-                        pivot.eastNorth = new EastNorth(pivot.eastNorth.east()+os.x, pivot.eastNorth.north()+os.y);
+                        pivot.eastNorth = new EastNorth(pivot.eastNorth.east()+os.eastNorth.east(), pivot.eastNorth.north()+os.eastNorth.north());
                         pivot.coor = Main.proj.eastNorth2latlon(pivot.eastNorth);
                 }
@@ -118 +101 @@
                         double cosPhi = Math.cos(rotationAngle);
                         double sinPhi = Math.sin(rotationAngle);
-                        double x = oldState.get(n).x - pivot.eastNorth.east();
-                        double y = oldState.get(n).y - pivot.eastNorth.north();
+                        EastNorth oldEastNorth = oldState.get(n).eastNorth;
+                        double x = oldEastNorth.east() - pivot.eastNorth.east();
+                        double y = oldEastNorth.north() - pivot.eastNorth.north();
                         double nx =  sinPhi * x + cosPhi * y + pivot.eastNorth.east();
                         double ny = -cosPhi * x + sinPhi * y + pivot.eastNorth.north();
                         n.eastNorth = new EastNorth(nx, ny);
                         n.coor = Main.proj.eastNorth2latlon(n.eastNorth);
-                        if (setModified) n.modified = true;
+                        if (setModified)
+                                n.modified = true;      
                 }
         }
@@ -134 +119 @@
         @Override public void undoCommand() {
                 for (Node n : objects) {
-                        OldState os = oldState.get(n);
-                        n.eastNorth = new EastNorth(os.x, os.y);
-                        n.coor = new LatLon(os.lat, os.lon);
+                        MoveCommand.OldState os = oldState.get(n);
+                        n.eastNorth = os.eastNorth;
+                        n.coor = os.latlon;
                         n.modified = os.modified;
                 }