Changeset 13736 in josm for trunk/src/org/openstreetmap/josm/actions

Timestamp:

2018-05-13T01:32:56+02:00 (6 years ago)

Author:

Don-vip

Message:

see #16260 - fix build

File:

• trunk/src/org/openstreetmap/josm/actions/JoinAreasAction.java (modified) (6 diffs)

trunk/src/org/openstreetmap/josm/actions/JoinAreasAction.java

@@ -63 +63 @@
     private DataSet ds;
     private final transient List<Relation> addedRelations = new LinkedList<>();
+    private final boolean addUndoRedo;
 
     /**
@@ -457 +458 @@
     /**
      * Constructs a new {@code JoinAreasAction} with optional shortcut and adapters.
-     * @param addShortcutToolbarAdapters controls whether the shortcut should be registered or not, as for toolbar registration and adapters
+     * @param addShortcutToolbarAdapters controls whether the shortcut should be registered or not,
+     * as for toolbar registration, adapters creation and undo/redo integration
      * @since 11611
      */
@@ -464 +466 @@
         Shortcut.registerShortcut("tools:joinareas", tr("Tool: {0}", tr("Join overlapping Areas")), KeyEvent.VK_J, Shortcut.SHIFT)
         : null, addShortcutToolbarAdapters, null, addShortcutToolbarAdapters);
+        addUndoRedo = addShortcutToolbarAdapters;
     }
 
@@ -566 +569 @@
             //FIXME: this is dirty hack
             makeCommitsOneAction(tr("Reverting changes"));
-            MainApplication.undoRedo.undo();
-            MainApplication.undoRedo.redoCommands.clear();
+            if (addUndoRedo) {
+                MainApplication.undoRedo.undo();
+                MainApplication.undoRedo.redoCommands.clear();
+            }
         }
     }
@@ -828 +833 @@
     }
 
-    private static void commitCommand(Command c) {
-        if (Main.main != null) {
+    private void commitCommand(Command c) {
+        if (Main.main != null && addUndoRedo) {
             MainApplication.undoRedo.add(c);
         } else {
@@ -1596 +1601 @@
     private void makeCommitsOneAction(String message) {
         cmds.clear();
-        if (Main.main != null) {
+        if (Main.main != null && addUndoRedo) {
             UndoRedoHandler ur = MainApplication.undoRedo;
             int i = Math.max(ur.commands.size() - cmdsCount, 0);