You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ja...@apache.org on 2010/06/18 00:32:48 UTC
svn commit: r955770 - in
/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context:
FacesContextFactoryImpl.java servlet/FacesContextImpl.java
Author: jakobk
Date: Thu Jun 17 22:32:47 2010
New Revision: 955770
URL: http://svn.apache.org/viewvc?rev=955770&view=rev
Log:
MYFACES-2759 Clean up FacesContextImpl
Modified:
myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/FacesContextFactoryImpl.java
myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/servlet/FacesContextImpl.java
Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/FacesContextFactoryImpl.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/FacesContextFactoryImpl.java?rev=955770&r1=955769&r2=955770&view=diff
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/FacesContextFactoryImpl.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/FacesContextFactoryImpl.java Thu Jun 17 22:32:47 2010
@@ -151,7 +151,7 @@ public class FacesContextFactoryImpl ext
if (context instanceof ServletContext)
{
- FacesContext facesContext = new FacesContextImpl(externalContext, null, this);
+ FacesContext facesContext = new FacesContextImpl(externalContext, this);
facesContext.setExceptionHandler(_exceptionHandlerFactory.getExceptionHandler());
Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/servlet/FacesContextImpl.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/servlet/FacesContextImpl.java?rev=955770&r1=955769&r2=955770&view=diff
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/servlet/FacesContextImpl.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/servlet/FacesContextImpl.java Thu Jun 17 22:32:47 2010
@@ -18,7 +18,6 @@
*/
package org.apache.myfaces.context.servlet;
-import java.lang.String;
import java.util.ArrayList;
import java.util.Collections;
import java.util.HashMap;
@@ -97,20 +96,14 @@ public class FacesContextImpl extends Fa
init(new ServletExternalContextImpl(servletContext, servletRequest, servletResponse));
}
- public FacesContextImpl(final ExternalContext externalContext,
- final ReleaseableExternalContext defaultExternalContext ,
+ public FacesContextImpl(final ExternalContext externalContext,
final ReleaseableFacesContextFactory facesContextFactory)
{
_facesContextFactory = facesContextFactory;
- init(externalContext, defaultExternalContext);
+ init(externalContext);
}
- private void init(final ReleaseableExternalContext externalContext)
- {
- init((ExternalContext) externalContext, externalContext);
- }
-
- private void init(final ExternalContext externalContext, final ReleaseableExternalContext defaultExternalContext)
+ private void init(final ExternalContext externalContext)
{
_externalContext = externalContext;
FacesContext.setCurrentInstance(this); //protected method, therefore must be called from here
@@ -496,17 +489,6 @@ public class FacesContextImpl extends Fa
_exceptionHandler = exceptionHandler;
}
- // Portlet need to do this to change from ActionRequest/Response to
- // RenderRequest/Response
- /* This code comes from jsf 1.1 and is not valid anymore
- public final void setExternalContext(ReleaseableExternalContext extContext)
- {
- assertNotReleased();
-
- _externalContext = extContext;
- FacesContext.setCurrentInstance(this); // TODO: figure out if I really need to do this
- }*/
-
@Override
public final ELContext getELContext()
{