You are viewing a plain text version of this content. The canonical link for it is here.

Posted to scm@geronimo.apache.org by xu...@apache.org on 2011/09/19 15:43:18 UTC

svn commit: r1172626 - /geronimo/server/trunk/plugins/myfaces/geronimo-myfaces/src/main/java/org/apache/geronimo/myfaces/MyFacesSystemInitializer.java

Author: xuhaihong
Date: Mon Sep 19 13:43:18 2011
New Revision: 1172626

URL: http://svn.apache.org/viewvc?rev=1172626&view=rev
Log:
Comment out the clean up codes for FactoryFinderProviderFactory, as MyFaces will think it is an initialization error

Modified:
    geronimo/server/trunk/plugins/myfaces/geronimo-myfaces/src/main/java/org/apache/geronimo/myfaces/MyFacesSystemInitializer.java

Modified: geronimo/server/trunk/plugins/myfaces/geronimo-myfaces/src/main/java/org/apache/geronimo/myfaces/MyFacesSystemInitializer.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/myfaces/geronimo-myfaces/src/main/java/org/apache/geronimo/myfaces/MyFacesSystemInitializer.java?rev=1172626&r1=1172625&r2=1172626&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/myfaces/geronimo-myfaces/src/main/java/org/apache/geronimo/myfaces/MyFacesSystemInitializer.java (original)
+++ geronimo/server/trunk/plugins/myfaces/geronimo-myfaces/src/main/java/org/apache/geronimo/myfaces/MyFacesSystemInitializer.java Mon Sep 19 13:43:18 2011
@@ -43,7 +43,8 @@ public class MyFacesSystemInitializer im
 
     @Override
     public void doStop() throws Exception {
-        FactoryFinderProviderFactory.setInstance(null);
+        //Need to think about how to clean this in OSGi environment
+        //FactoryFinderProviderFactory.setInstance(null);
     }
 
 }