Changeset 745 in josm for trunk/src/org/openstreetmap/josm/gui

Timestamp:

2008-08-04T16:51:56+02:00 (16 years ago)

Author:

stoecker

Message:

added security check for layer deletion. Fixes #262

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/LayerListDialog.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/dialogs/LayerListDialog.java

@@ -22 +22 @@
 import javax.swing.JLabel;
 import javax.swing.JList;
+import javax.swing.JOptionPane;
 import javax.swing.JPanel;
 import javax.swing.JScrollPane;
@@ -69 +70 @@
                         int sel = instance.getSelectedIndex();
                         Layer l = layer != null ? layer : (Layer)instance.getSelectedValue();
-                        if (l instanceof OsmDataLayer && !DontShowAgainInfo.show("delete_layer", tr("Do you really want to delete the whole layer?")))
-                                return;
+                        if (l instanceof OsmDataLayer)
+                        {
+                                if (((OsmDataLayer)l).isModified())
+                                {
+                                        if(JOptionPane.showConfirmDialog(instance, tr("There are unsaved changes. Delete the layer anwyay?"),
+                                        tr("Unsaved Changes"), JOptionPane.YES_NO_OPTION) != JOptionPane.YES_OPTION)
+                                                return;
+                                }
+                                else if(!DontShowAgainInfo.show("delete_layer", tr("Do you really want to delete the whole layer?"), false))
+                                        return;
+                        }
                         Main.main.removeLayer(l);
                         if (sel >= instance.getModel().getSize())