Changeset 2523 in josm for trunk/src/org/openstreetmap

Timestamp:

2009-11-27T22:30:40+01:00 (14 years ago)

Author:

stoecker

Message:

minor cleanup

File:

• trunk/src/org/openstreetmap/josm/gui/ConditionalOptionPaneUtil.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/gui/ConditionalOptionPaneUtil.java

@@ -108 +108 @@
         if (!getDialogShowingEnabled(preferenceKey))
             return DIALOG_DISABLED_OPTION;
-        MessagePanel pnl = new MessagePanel(preferenceKey, message);
+        MessagePanel pnl = new MessagePanel(false, message);
         int ret = JOptionPane.showOptionDialog(parent, pnl, title, optionType, messageType, null,options,defaultOption);
 
@@ -147 +147 @@
      */
     static public boolean showConfirmationDialog(String preferenceKey, Component parent, Object message, String title, int optionType, int messageType, int trueOption) throws HeadlessException {
-        if (!getDialogShowingEnabled(preferenceKey) && (getDialogReturnValue(preferenceKey) >= 0))
+        boolean donotshow = getDialogShowingEnabled(preferenceKey);
+        if (donotshow && (getDialogReturnValue(preferenceKey) >= 0))
             return getDialogReturnValue(preferenceKey) == trueOption;
-        MessagePanel pnl = new MessagePanel(preferenceKey, message);
+        MessagePanel pnl = new MessagePanel(donotshow, message);
         int ret = JOptionPane.showConfirmDialog(parent, pnl, title, optionType, messageType);
         if ((ret >= 0) && !pnl.getDialogShowingEnabled()) {
@@ -179 +180 @@
         if (!getDialogShowingEnabled(preferenceKey))
             return;
-        MessagePanel pnl = new MessagePanel(preferenceKey, message);
+        MessagePanel pnl = new MessagePanel(false, message);
         JOptionPane.showMessageDialog(parent, pnl, title, messageType);
         if(!pnl.getDialogShowingEnabled())
@@ -194 +195 @@
     private static class MessagePanel extends JPanel {
         JCheckBox cbShowDialog;
-        String preferenceKey;
-
-        public MessagePanel(String preferenceKey, Object message) {
-            this.preferenceKey = preferenceKey;
+
+        public MessagePanel(boolean donotshow, Object message) {
             cbShowDialog = new JCheckBox(tr("Do not show again"));
-            cbShowDialog.setSelected(!ConditionalOptionPaneUtil.getDialogShowingEnabled(preferenceKey));
+            cbShowDialog.setSelected(donotshow);
             setLayout(new GridBagLayout());