You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mm...@apache.org on 2007/08/08 08:39:55 UTC
svn commit: r563770 -
/myfaces/core/trunk/api/src/main/java/javax/faces/webapp/FacesServlet.java
Author: mmarinschek
Date: Tue Aug 7 23:39:54 2007
New Revision: 563770
URL: http://svn.apache.org/viewvc?view=rev&rev=563770
Log:
refactored generation of faces-context out into a separate private method (often people copy/paste the FacesServlet to do some special stuff, even though this is not the best form of reuse, having a marker method for something which changes often might help them with determining what needs to be copied over the next time)
Modified:
myfaces/core/trunk/api/src/main/java/javax/faces/webapp/FacesServlet.java
Modified: myfaces/core/trunk/api/src/main/java/javax/faces/webapp/FacesServlet.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/api/src/main/java/javax/faces/webapp/FacesServlet.java?view=diff&rev=563770&r1=563769&r2=563770
==============================================================================
--- myfaces/core/trunk/api/src/main/java/javax/faces/webapp/FacesServlet.java (original)
+++ myfaces/core/trunk/api/src/main/java/javax/faces/webapp/FacesServlet.java Tue Aug 7 23:39:54 2007
@@ -135,11 +135,7 @@
}
if(log.isTraceEnabled()) log.trace("service begin");
- FacesContext facesContext
- = _facesContextFactory.getFacesContext(_servletConfig.getServletContext(),
- request,
- response,
- _lifecycle);
+ FacesContext facesContext = prepareFacesContext(request, response);
try {
_lifecycle.execute(facesContext);
_lifecycle.render(facesContext);
@@ -198,4 +194,13 @@
return (initParameter.equalsIgnoreCase("on") || initParameter.equals("1") || initParameter.equalsIgnoreCase("true"));
}
+
+ private FacesContext prepareFacesContext(ServletRequest request, ServletResponse response) {
+ FacesContext facesContext
+ = _facesContextFactory.getFacesContext(_servletConfig.getServletContext(),
+ request,
+ response,
+ _lifecycle);
+ return facesContext;
+ }
}