You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dw...@apache.org on 2008/03/05 20:09:54 UTC

svn commit: r633980 - /geronimo/server/branches/2.0/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java

Author: dwoods
Date: Wed Mar  5 11:09:52 2008
New Revision: 633980

URL: http://svn.apache.org/viewvc?rev=633980&view=rev
Log:
use readOnly setting if provided

Modified:
    geronimo/server/branches/2.0/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java

Modified: geronimo/server/branches/2.0/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.0/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java?rev=633980&r1=633979&r2=633980&view=diff
==============================================================================
--- geronimo/server/branches/2.0/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java (original)
+++ geronimo/server/branches/2.0/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java Wed Mar  5 11:09:52 2008
@@ -468,6 +468,9 @@
     }
 
     public void startConfiguration(Artifact configurationName) {
+        if (readOnly) {
+            return;
+        }
         ConfigurationOverride configuration = serverOverride.getConfiguration(configurationName, false);
         if (configuration == null) {
             return;
@@ -477,6 +480,9 @@
     }
 
     public synchronized void addConfiguration(Artifact configurationName) {
+        if (readOnly) {
+            return;
+        }
         // Check whether we have it already
         ConfigurationOverride configuration = serverOverride.getConfiguration(configurationName, false);
         // If not, initialize it
@@ -488,6 +494,9 @@
     }
 
     public synchronized void removeConfiguration(Artifact configName) {
+        if (readOnly) {
+            return;
+        }
         ConfigurationOverride configuration = serverOverride.getConfiguration(configName);
         if (configuration == null) {
             return;
@@ -501,6 +510,9 @@
     }
 
     public void stopConfiguration(Artifact configName) {
+        if (readOnly) {
+            return;
+        }
         ConfigurationOverride configuration = serverOverride.getConfiguration(configName);
         if (configuration == null) {
             return;
@@ -510,6 +522,9 @@
     }
 
     public void migrateConfiguration(Artifact oldName, Artifact newName, Configuration configuration) {
+        if (readOnly) {
+            return;
+        }
         ConfigurationOverride configInfo = serverOverride.getConfiguration(oldName);
         if (configInfo == null) {
             throw new IllegalArgumentException("Trying to migrate unknown configuration: " + oldName);