Changeset 11733 in josm for trunk/src

Timestamp:

2017-03-16T17:07:39+01:00 (7 years ago)

Author:

stoecker

Message:

see #14526 - fix error handling in SequenceCommand

File:

• trunk/src/org/openstreetmap/josm/command/SequenceCommand.java (modified) (2 diffs)

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

@@ -73 +73 @@
 
     protected final void undoCommands(int start) {
-        // We probably aborted this halfway though the
-        // execution sequence because of a sub-command
-        // error.  We already undid the sub-commands.
-        if (!sequenceComplete)
-            return;
         for (int i = start; i >= 0; --i) {
             sequence[i].undoCommand();
@@ -84 +79 @@
 
     @Override public void undoCommand() {
+        // We probably aborted this halfway though the
+        // execution sequence because of a sub-command
+        // error.  We already undid the sub-commands.
+        if (!sequenceComplete)
+            return;
         undoCommands(sequence.length-1);
     }