You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by da...@apache.org on 2005/12/07 23:28:47 UTC

svn commit: r354878 - /geronimo/trunk/modules/system/src/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java

Author: dain
Date: Wed Dec  7 14:28:44 2005
New Revision: 354878

URL: http://svn.apache.org/viewcvs?rev=354878&view=rev
Log:
Fixed GERONIMO-1295
attributeChanged needs to be called from addConfiguration, removeConfiguration and setReference

Modified:
    geronimo/trunk/modules/system/src/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java

Modified: geronimo/trunk/modules/system/src/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java
URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/system/src/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java?rev=354878&r1=354877&r2=354878&view=diff
==============================================================================
--- geronimo/trunk/modules/system/src/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java (original)
+++ geronimo/trunk/modules/system/src/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java Wed Dec  7 14:28:44 2005
@@ -266,6 +266,7 @@
             }
         }
         gbean.setReferencePatterns(reference.getName(), patterns);
+        attributeChanged();
     }
 
     public synchronized void setShouldLoad(String configurationName, ObjectName gbeanName, boolean load) {
@@ -396,6 +397,7 @@
     public synchronized void addConfiguration(String configurationName) {
         ConfigurationOverride configuration = serverOverride.getConfiguration(configurationName, true);
         configuration.setLoad(true);
+        attributeChanged();
     }
 
     public synchronized void removeConfiguration(String configName) {
@@ -408,6 +410,7 @@
             } else {
                 configuration.setLoad(false);
             }
+            attributeChanged();
         }
     }