You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by fm...@apache.org on 2007/04/19 12:13:28 UTC

svn commit: r530358 - /incubator/felix/trunk/configadmin/src/main/java/org/apache/felix/cm/impl/Factory.java

Author: fmeschbe
Date: Thu Apr 19 03:13:21 2007
New Revision: 530358

URL: http://svn.apache.org/viewvc?view=rev&rev=530358
Log:
FELIX-271 Factory Configuration may get corrupted

Modified:
    incubator/felix/trunk/configadmin/src/main/java/org/apache/felix/cm/impl/Factory.java

Modified: incubator/felix/trunk/configadmin/src/main/java/org/apache/felix/cm/impl/Factory.java
URL: http://svn.apache.org/viewvc/incubator/felix/trunk/configadmin/src/main/java/org/apache/felix/cm/impl/Factory.java?view=diff&rev=530358&r1=530357&r2=530358
==============================================================================
--- incubator/felix/trunk/configadmin/src/main/java/org/apache/felix/cm/impl/Factory.java (original)
+++ incubator/felix/trunk/configadmin/src/main/java/org/apache/felix/cm/impl/Factory.java Thu Apr 19 03:13:21 2007
@@ -86,7 +86,7 @@
         this( persistenceManager, factoryPid );
 
         // set bundle location
-        setBundleLocation( ( String ) props.get( ConfigurationAdmin.SERVICE_BUNDLELOCATION ) );
+        bundleLocation = ( String ) props.get( ConfigurationAdmin.SERVICE_BUNDLELOCATION );
 
         // set pids
         String[] pidList = ( String[] ) props.get( FACTORY_PID_LIST );
@@ -94,7 +94,7 @@
         {
             for ( int i = 0; i < pidList.length; i++ )
             {
-                addPID( pidList[i] );
+                pids.add( pidList[i] );
             }
         }
     }