Changeset 24026 in osm for applications/editors

Timestamp:

2010-11-02T19:14:17+01:00 (14 years ago)

Author:

oliverw

Message:

Do not invalidate container on single entity changes, just notify clients.

Location:

applications/editors/josm/plugins/FixAddresses/src/org/openstreetmap/josm/plugins/fixAddresses

Files:

• AddressEditContainer.java (modified) (2 diffs)
• AddressNode.java (modified) (2 diffs)

applications/editors/josm/plugins/FixAddresses/src/org/openstreetmap/josm/plugins/fixAddresses/AddressEditContainer.java

@@ -142 +142 @@
                 for (IAddressEditContainerListener listener : shadowListeners) {
                         listener.containerChanged(this);
+                }
+        }
+        
+        /**
+         * Notifies clients that an entity within the address container changed.
+         */
+        protected void fireEntityChanged(INodeEntity entity) {
+                if (entity == null) throw new RuntimeException("Entity must not be null");
+                
+                List<IAddressEditContainerListener> shadowListeners = 
+                        new ArrayList<IAddressEditContainerListener>(listeners);
+                
+                for (IAddressEditContainerListener listener : shadowListeners) {
+                        listener.entityChanged(entity);
                 }
         }
@@ -643 +657 @@
         @Override
         public void entityChanged(INodeEntity entity) {
-                invalidate();           
+                fireEntityChanged(entity);      
         }
 }

applications/editors/josm/plugins/FixAddresses/src/org/openstreetmap/josm/plugins/fixAddresses/AddressNode.java

@@ -374 +374 @@
         public void setGuessedValue(String tag, String value) {
                 guessedValues.put(tag, value);
+                fireEntityChanged(this);
         }
         
@@ -406 +407 @@
          */
         public void setDerivedValue(String tag, String value) {
-                derivedValues.put(tag, value);
+                derivedValues.put(tag, value);          
         }