Changeset 5653 in josm

Timestamp:

2012-12-31T14:41:24+01:00 (12 years ago)

Author:

Don-vip

Message:

Fix #8128 - NPE while dragging object

File:

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

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

@@ -674 +674 @@
         Command c = getLastCommand();
         if (mode == Mode.move) {
+            if (startEN == null) return false; // fix #8128
             getCurrentDataSet().beginUpdate();
             if (c instanceof MoveCommand && affectedNodes.equals(((MoveCommand) c).getParticipatingPrimitives())) {
@@ -686 +687 @@
                     // Revert move
                     ((MoveCommand) c).resetToCheckpoint();
+                    getCurrentDataSet().endUpdate();
                     JOptionPane.showMessageDialog(
                             Main.parent,
@@ -695 +697 @@
                 }
             }
-            getCurrentDataSet().endUpdate();
-            return true;
-        } 
-
-        startEN = currentEN; // drag can continue after scaling/rotation
-
-        if (mode == Mode.rotate) {
-            getCurrentDataSet().beginUpdate();
-            if (c instanceof RotateCommand && affectedNodes.equals(((RotateCommand) c).getTransformedNodes())) {
-                ((RotateCommand) c).handleEvent(currentEN);
+        } else {
+            startEN = currentEN; // drag can continue after scaling/rotation
+    
+            if (mode == Mode.rotate) {
+                getCurrentDataSet().beginUpdate();
+                if (c instanceof RotateCommand && affectedNodes.equals(((RotateCommand) c).getTransformedNodes())) {
+                    ((RotateCommand) c).handleEvent(currentEN);
+                } else {
+                    Main.main.undoRedo.add(new RotateCommand(selection, currentEN));
+                }
+            } else if (mode == Mode.scale) {
+                getCurrentDataSet().beginUpdate();
+                if (c instanceof ScaleCommand && affectedNodes.equals(((ScaleCommand) c).getTransformedNodes())) {
+                    ((ScaleCommand) c).handleEvent(currentEN);
+                } else {
+                    Main.main.undoRedo.add(new ScaleCommand(selection, currentEN));
+                }
             } else {
-                Main.main.undoRedo.add(new RotateCommand(selection, currentEN));
-            }
-            getCurrentDataSet().endUpdate();
-        } else if (mode == Mode.scale) {
-            getCurrentDataSet().beginUpdate();
-            if (c instanceof ScaleCommand && affectedNodes.equals(((ScaleCommand) c).getTransformedNodes())) {
-                ((ScaleCommand) c).handleEvent(currentEN);
-            } else {
-                Main.main.undoRedo.add(new ScaleCommand(selection, currentEN));
-            }
-            getCurrentDataSet().endUpdate();
-        }
+                return false;
+            }
+        }
+        getCurrentDataSet().endUpdate();
         return true;
     }