Changeset 35409 in osm for applications/editors/josm/plugins/reverter/src/reverter/RevertChangesetTask.java

Timestamp:

2020-04-04T07:40:00+02:00 (4 years ago)

Author:

gerdp

Message:

see #19028: Plugin reverter uses a more recent API that minimal requested

• revert temp. changes done in o35407 and increase main version

File:

• applications/editors/josm/plugins/reverter/src/reverter/RevertChangesetTask.java (modified) (4 diffs)

applications/editors/josm/plugins/reverter/src/reverter/RevertChangesetTask.java

@@ -15 +15 @@
 
 import org.openstreetmap.josm.command.Command;
-import org.openstreetmap.josm.command.SequenceCommand;
 import org.openstreetmap.josm.command.conflict.ConflictAddCommand;
 import org.openstreetmap.josm.data.UndoRedoHandler;
@@ -99 +98 @@
                 newLayer = false; // reuse layer for subsequent reverts
             } catch (OsmTransferException e) {
-                Logging.error(e);
+                                if (!allcmds.isEmpty()) {
+                                        GuiHelper.runInEDT(() -> UndoRedoHandler.getInstance().undo(allcmds.size()));
+                                }
+                                Logging.error(e);
                 throw e;
             } catch (UserCancelException e) {
+                                if (!allcmds.isEmpty()) {
+                                        GuiHelper.runInEDT(() -> UndoRedoHandler.getInstance().undo(allcmds.size()));
+                                }
                 Logging.warn("Revert canceled");
                 Logging.trace(e);
@@ -108 +113 @@
         }
         if (!allcmds.isEmpty()) {
-            Command cmd = allcmds.size() == 1 ? allcmds.get(0) : new SequenceCommand(tr("Revert changesets"), allcmds);
+            Command cmd = allcmds.size() == 1 ? allcmds.get(0) : new RevertChangesetCommand(tr("Revert changesets"), allcmds);
             GuiHelper.runInEDT(() -> {
-                UndoRedoHandler.getInstance().add(cmd);
+                UndoRedoHandler.getInstance().add(cmd, false);
                 if (numberOfConflicts > 0) {
                     MainApplication.getMap().conflictDialog.warnNumNewConflicts(numberOfConflicts);
@@ -169 +174 @@
                 numberOfConflicts++;
             }
+            c.executeCommand();
         }
         final String desc;