Changeset 2658 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/properties

Timestamp:

2009-12-19T14:06:51+01:00 (14 years ago)

Author:

jttt

Message:

Do not update properties and selection dialogs when hidden

Location:

trunk/src/org/openstreetmap/josm/gui/dialogs/properties

Files:

• ListOfUsedTags.java (modified) (1 diff)
• PropertiesDialog.java (modified) (6 diffs)

trunk/src/org/openstreetmap/josm/gui/dialogs/properties/ListOfUsedTags.java

@@ -47 +47 @@
         else
             return values;
+    }
+
+    public void rebuildNecessary() {
+        dirty = true;
     }
 

trunk/src/org/openstreetmap/josm/gui/dialogs/properties/PropertiesDialog.java

@@ -71 +71 @@
 import org.openstreetmap.josm.gui.dialogs.ToggleDialog;
 import org.openstreetmap.josm.gui.dialogs.relation.RelationEditor;
-import org.openstreetmap.josm.gui.layer.Layer;
 import org.openstreetmap.josm.gui.layer.OsmDataLayer;
 import org.openstreetmap.josm.gui.preferences.TaggingPresetPreference;
@@ -97 +96 @@
  * @author imi
  */
-public class PropertiesDialog extends ToggleDialog implements SelectionChangedListener, MapView.LayerChangeListener {
+public class PropertiesDialog extends ToggleDialog implements SelectionChangedListener, MapView.EditLayerChangeListener {
     /**
      * Watches for double clicks and from editing or new property, depending on the
@@ -145 +144 @@
     public void showNotify() {
         DatasetEventManager.getInstance().addDatasetListener(listOfUsedTags, false);
+        listOfUsedTags.rebuildNecessary();
+        DataSet.selListeners.add(this);
+        updateSelection();
     }
 
@@ -150 +152 @@
     public void hideNotify() {
         DatasetEventManager.getInstance().removeDatasetListener(listOfUsedTags);
+        DataSet.selListeners.remove(this);
     }
 
@@ -568 +571 @@
         add(buttonPanel, BorderLayout.SOUTH);
 
-        DataSet.selListeners.add(this);
-        MapView.addLayerChangeListener(this);
+        MapView.addEditLayerChangeListener(this);
     }
 
     @Override
     public void tearDown() {
-        MapView.removeLayerChangeListener(this);
+        MapView.removeEditLayerChangeListener(this);
     }
 
@@ -776 +778 @@
     }
 
+    private void updateSelection() {
+        if (Main.main.getCurrentDataSet() == null) {
+            selectionChanged(Collections.<OsmPrimitive>emptyList());
+        } else {
+            selectionChanged(Main.main.getCurrentDataSet().getSelected());
+        }
+    }
+
     /* ---------------------------------------------------------------------------------- */
-    /* LayerChangeListener                                                                */
+    /* EditLayerChangeListener                                                                */
     /* ---------------------------------------------------------------------------------- */
-    public void activeLayerChange(Layer oldLayer, Layer newLayer) {
-        if (newLayer instanceof OsmDataLayer) {
-            OsmDataLayer dataLayer = (OsmDataLayer)newLayer;
-            selectionChanged(dataLayer.data.getSelected());
-        } else {
-            List<OsmPrimitive> selection = Collections.emptyList();
-            selectionChanged(selection);
-        }
-    }
-
-    public void layerAdded(Layer newLayer) {
-        // do nothing
-    }
-
-    public void layerRemoved(Layer oldLayer) {
-        // do nothing
+    public void editLayerChanged(OsmDataLayer oldLayer, OsmDataLayer newLayer) {
+        updateSelection();
     }