You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by de...@apache.org on 2007/03/11 23:16:23 UTC

svn commit: r517031 - /myfaces/shared/branches/3_0_0/core/src/main/java/org/apache/myfaces/shared/config/MyfacesConfig.java

Author: dennisbyrne
Date: Sun Mar 11 15:16:22 2007
New Revision: 517031

URL: http://svn.apache.org/viewvc?view=rev&rev=517031
Log:
MYFACES-1557, patch for thread safety by David Jencks

Modified:
    myfaces/shared/branches/3_0_0/core/src/main/java/org/apache/myfaces/shared/config/MyfacesConfig.java

Modified: myfaces/shared/branches/3_0_0/core/src/main/java/org/apache/myfaces/shared/config/MyfacesConfig.java
URL: http://svn.apache.org/viewvc/myfaces/shared/branches/3_0_0/core/src/main/java/org/apache/myfaces/shared/config/MyfacesConfig.java?view=diff&rev=517031&r1=517030&r2=517031
==============================================================================
--- myfaces/shared/branches/3_0_0/core/src/main/java/org/apache/myfaces/shared/config/MyfacesConfig.java (original)
+++ myfaces/shared/branches/3_0_0/core/src/main/java/org/apache/myfaces/shared/config/MyfacesConfig.java Sun Mar 11 15:16:22 2007
@@ -92,10 +92,21 @@
     {
         MyfacesConfig myfacesConfig = (MyfacesConfig)extCtx
                                         .getApplicationMap().get(APPLICATION_MAP_PARAM_NAME);
-        if (myfacesConfig != null) return myfacesConfig;
+        if (myfacesConfig == null)
+        {
+        	
+        	myfacesConfig = createAndInitializeMyFacesConfig(extCtx);
+        	
+        	extCtx.getApplicationMap().put(APPLICATION_MAP_PARAM_NAME, myfacesConfig);
+        	
+        }
 
-        myfacesConfig = new MyfacesConfig();
-        extCtx.getApplicationMap().put(APPLICATION_MAP_PARAM_NAME, myfacesConfig);
+        return myfacesConfig;
+    }
+
+	private static MyfacesConfig createAndInitializeMyFacesConfig(ExternalContext extCtx) {
+		
+		MyfacesConfig myfacesConfig = new MyfacesConfig();
 
         myfacesConfig.setPrettyHtml(getBooleanInitParameter(extCtx, INIT_PARAM_PRETTY_HTML,
                                                             INIT_PARAM_PRETTY_HTML_DEFAULT));
@@ -119,9 +130,8 @@
             	log.info("Tomahawk jar not available. Autoscrolling, DetectJavascript, AddResourceClass and CheckExtensionsFilter are disabled now.");
             }
         }
-                
-        return myfacesConfig;
-    }
+		return myfacesConfig;
+	}
 
     private static boolean getBooleanInitParameter(ExternalContext externalContext,
                                                    String paramName,