Changeset 16558 in josm for trunk/src/org

Timestamp:

2020-06-07T15:56:16+02:00 (4 years ago)

Author:

GerdP

Message:

fix #19138: Warn when you try to delete a node which is part of a way which is hidden by a filter

• add check and dialog similar to that which is done when a node is moved

File:

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

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

@@ -23 +23 @@
 import org.openstreetmap.josm.data.osm.Relation;
 import org.openstreetmap.josm.data.osm.WaySegment;
+import org.openstreetmap.josm.gui.ExtendedDialog;
 import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MapFrame;
@@ -161 +162 @@
         if (c != null) {
             UndoRedoHandler.getInstance().add(c);
+            if (changesHiddenWay(c)) {
+                final ConfirmDeleteDialog ed = new ConfirmDeleteDialog();
+                ed.setContent(tr("Are you sure that you want to delete elements with attached ways that are hidden by filters?"));
+                ed.toggleEnable("deletedHiddenElements");
+                if (ed.showDialog().getValue() != 1) {
+                    UndoRedoHandler.getInstance().undo(1);
+                    return;
+                }
+            }
+
             //FIXME: This should not be required, DeleteCommand should update the selection, otherwise undo/redo won't work.
             lm.getEditDataSet().setSelected();
@@ -426 +437 @@
         giveUserFeedback(oldEvent, modifiers);
     }
+
+    private boolean changesHiddenWay(Command c) {
+        return c.getParticipatingPrimitives().stream().anyMatch(OsmPrimitive::isDisabledAndHidden);
+    }
+
+    static class ConfirmDeleteDialog extends ExtendedDialog {
+        ConfirmDeleteDialog() {
+            super(MainApplication.getMainFrame(),
+                    tr("Delete elements"),
+                    tr("Delete them"), tr("Undo delete"));
+            setButtonIcons("dialogs/delete", "cancel");
+            setCancelButton(2);
+        }
+    }
+
 }