Changeset 2930 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/ConflictResolutionDialog.java

Timestamp:

03.02.2010 09:07:36 (2 years ago)

Author:

jttt

Message:

Make sure ConflictResolutionDialog is disposed when closed

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/ConflictResolutionDialog.java (modified) (10 diffs)

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

@@ -21 +21 @@
 import javax.swing.JOptionPane;
 import javax.swing.JPanel;
+import javax.swing.WindowConstants;
 
 import org.openstreetmap.josm.Main;
@@ -43 +44 @@
     private ConflictResolver resolver;
 
+    private ApplyResolutionAction applyResolutionAction;
+
     /**
      * restore position and size on screen from preference settings
@@ -95 +98 @@
         Main.pref.put("conflictresolutiondialog.width", Integer.toString(d.width));
         Main.pref.put("conflictresolutiondialog.height", Integer.toString(d.height));
+    }
+
+    @Override
+    public void removeNotify() {
+        super.removeNotify();
+        unregisterListeners();
     }
 
@@ -104 +113 @@
         } else {
             rememberPositionAndDimension();
+            unregisterListeners();
         }
         super.setVisible(isVisible);
+    }
+
+    private void closeDialog() {
+        setVisible(false);
+        dispose();
     }
 
@@ -117 +132 @@
         pnl.setLayout(new FlowLayout(FlowLayout.CENTER));
 
-        ApplyResolutionAction applyResolutionAction = new ApplyResolutionAction();
-        resolver.addPropertyChangeListener(applyResolutionAction);
+        applyResolutionAction = new ApplyResolutionAction();
         JButton btn = new JButton(applyResolutionAction);
         btn.setName("button.apply");
@@ -135 +149 @@
     }
 
+    private void registerListeners() {
+        resolver.addPropertyChangeListener(applyResolutionAction);
+    }
+
+    private void unregisterListeners() {
+        resolver.removePropertyChangeListener(applyResolutionAction);
+    }
+
     /**
      * builds the GUI
      */
     protected void build() {
+        setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
         updateTitle();
         getContentPane().setLayout(new BorderLayout());
@@ -149 +172 @@
         resolver.addPropertyChangeListener(this);
         HelpUtil.setHelpContext(this.getRootPane(), "Dialog/ConflictDialog");
+
+        registerListeners();
     }
 
@@ -172 +197 @@
 
         public void actionPerformed(ActionEvent arg0) {
-            setVisible(false);
+            closeDialog();
         }
     }
@@ -231 +256 @@
                 switch(ret) {
                 case JOptionPane.YES_OPTION:
-                    setVisible(false);
+                    closeDialog();
                     break;
                 default:
@@ -240 +265 @@
                 Command cmd = resolver.buildResolveCommand();
                 Main.main.undoRedo.add(cmd);
-                setVisible(false);
+                closeDialog();
             } catch(OperationCancelledException e) {
                 // do nothing. Exception already reported