You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by em...@apache.org on 2017/10/25 14:22:24 UTC
svn commit: r1813299 - in
/myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces:
ee/MyFacesContainerInitializer.java webapp/AbstractFacesInitializer.java
Author: embreijo
Date: Wed Oct 25 14:22:24 2017
New Revision: 1813299
URL: http://svn.apache.org/viewvc?rev=1813299&view=rev
Log:
MYFACES-4166 Make FACES_SERVLET_ADDED_ATTRIBUTE key public
Modified:
myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/ee/MyFacesContainerInitializer.java
myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/webapp/AbstractFacesInitializer.java
Modified: myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/ee/MyFacesContainerInitializer.java
URL: http://svn.apache.org/viewvc/myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/ee/MyFacesContainerInitializer.java?rev=1813299&r1=1813298&r2=1813299&view=diff
==============================================================================
--- myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/ee/MyFacesContainerInitializer.java (original)
+++ myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/ee/MyFacesContainerInitializer.java Wed Oct 25 14:22:24 2017
@@ -113,9 +113,8 @@ public class MyFacesContainerInitializer
/**
* If the servlet mapping for the FacesServlet is added dynamically, Boolean.TRUE
* is stored under this key in the ServletContext.
- * ATTENTION: this constant is duplicate in AbstractFacesInitializer.
*/
- private static final String FACES_SERVLET_ADDED_ATTRIBUTE = "org.apache.myfaces.DYNAMICALLY_ADDED_FACES_SERVLET";
+ public static final String FACES_SERVLET_ADDED_ATTRIBUTE = "org.apache.myfaces.DYNAMICALLY_ADDED_FACES_SERVLET";
private static final String INITIALIZE_ALWAYS_STANDALONE = "org.apache.myfaces.INITIALIZE_ALWAYS_STANDALONE";
Modified: myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/webapp/AbstractFacesInitializer.java
URL: http://svn.apache.org/viewvc/myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/webapp/AbstractFacesInitializer.java?rev=1813299&r1=1813298&r2=1813299&view=diff
==============================================================================
--- myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/webapp/AbstractFacesInitializer.java (original)
+++ myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/webapp/AbstractFacesInitializer.java Wed Oct 25 14:22:24 2017
@@ -27,6 +27,7 @@ import org.apache.myfaces.config.element
import org.apache.myfaces.context.ReleaseableExternalContext;
import org.apache.myfaces.context.servlet.StartupFacesContextImpl;
import org.apache.myfaces.context.servlet.StartupServletExternalContextImpl;
+import org.apache.myfaces.ee.MyFacesContainerInitializer;
import org.apache.myfaces.shared.application.FacesServletMappingUtils;
import org.apache.myfaces.shared.context.ExceptionHandlerImpl;
import org.apache.myfaces.shared.util.StateUtils;
@@ -81,13 +82,6 @@ import org.apache.myfaces.view.facelets.
public abstract class AbstractFacesInitializer implements FacesInitializer
{
private static final Logger log = Logger.getLogger(AbstractFacesInitializer.class.getName());
-
- /**
- * If the servlet mapping for the FacesServlet is added dynamically, Boolean.TRUE
- * is stored under this key in the ServletContext.
- * ATTENTION: this constant is duplicate in MyFacesContainerInitializer.
- */
- private static final String FACES_SERVLET_ADDED_ATTRIBUTE = "org.apache.myfaces.DYNAMICALLY_ADDED_FACES_SERVLET";
/**
* This parameter specifies the ExpressionFactory implementation to use.
@@ -169,7 +163,8 @@ public abstract class AbstractFacesIniti
{
// check if the FacesServlet has been added dynamically
// in a Servlet 3.0 environment by MyFacesContainerInitializer
- Boolean mappingAdded = (Boolean) servletContext.getAttribute(FACES_SERVLET_ADDED_ATTRIBUTE);
+ Boolean mappingAdded = (Boolean) servletContext.getAttribute(
+ MyFacesContainerInitializer.FACES_SERVLET_ADDED_ATTRIBUTE);
if (mappingAdded == null || !mappingAdded)
{
if (log.isLoggable(Level.WARNING))
@@ -376,7 +371,8 @@ public abstract class AbstractFacesIniti
{
// check if the FacesServlet has been added dynamically
// in a Servlet 3.0 environment by MyFacesContainerInitializer
- Boolean mappingAdded = (Boolean) servletContext.getAttribute(FACES_SERVLET_ADDED_ATTRIBUTE);
+ Boolean mappingAdded = (Boolean) servletContext.getAttribute(
+ MyFacesContainerInitializer.FACES_SERVLET_ADDED_ATTRIBUTE);
if (mappingAdded == null || !mappingAdded)
{
if (log.isLoggable(Level.WARNING))