Changeset 5415 in josm for trunk

Timestamp:

2012-08-10T00:53:55+02:00 (12 years ago)

Author:

Don-vip

Message:

fix #7936 - Validator dialog is not updated when the active edit layer is deleted

Location:

trunk/src/org/openstreetmap/josm

Files:

• data/validation/OsmValidator.java (modified) (1 diff)
• gui/dialogs/ValidatorDialog.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/data/validation/OsmValidator.java

@@ -296 +296 @@
     @Override
     public void layerRemoved(Layer oldLayer) {
-        if (oldLayer instanceof OsmDataLayer && Main.map.mapView.getActiveLayer() == oldLayer) {
-            Main.map.validatorDialog.tree.setErrorList(new ArrayList<TestError>());
-        }
         if (oldLayer == errorLayer) {
             errorLayer = null;

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

@@ -85 +85 @@
     private DefaultMutableTreeNode lastSelectedNode = null;
 
+    private OsmDataLayer linkedLayer;
+
     /**
      * Constructor
@@ -367 +369 @@
             }
         }
-        Main.main.getCurrentDataSet().setSelected(sel);
+        DataSet ds = Main.main.getCurrentDataSet();
+        if (ds != null) {
+            ds.setSelected(sel);
+        }
     }
 
@@ -427 +432 @@
     public void activeLayerChange(Layer oldLayer, Layer newLayer) {
         if (newLayer instanceof OsmDataLayer) {
-            tree.setErrorList(((OsmDataLayer) newLayer).validationErrors);
+            linkedLayer = (OsmDataLayer)newLayer;
+            tree.setErrorList(linkedLayer.validationErrors);
         }
     }
@@ -435 +441 @@
 
     @Override
-    public void layerRemoved(Layer oldLayer) {}
+    public void layerRemoved(Layer oldLayer) {
+        if (oldLayer == linkedLayer) {
+            tree.setErrorList(new ArrayList<TestError>());
+        }
+    }
 
     /**