You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ba...@apache.org on 2005/07/18 20:44:44 UTC

svn commit: r219550 - /myfaces/impl/trunk/src/java/org/apache/myfaces/context/servlet/ServletExternalContextImpl.java

Author: baranda
Date: Mon Jul 18 11:43:43 2005
New Revision: 219550

URL: http://svn.apache.org/viewcvs?rev=219550&view=rev
Log:
Added Adam Winer's suggestion about the ExternalContext (MYFACES-325)

Modified:
    myfaces/impl/trunk/src/java/org/apache/myfaces/context/servlet/ServletExternalContextImpl.java

Modified: myfaces/impl/trunk/src/java/org/apache/myfaces/context/servlet/ServletExternalContextImpl.java
URL: http://svn.apache.org/viewcvs/myfaces/impl/trunk/src/java/org/apache/myfaces/context/servlet/ServletExternalContextImpl.java?rev=219550&r1=219549&r2=219550&view=diff
==============================================================================
--- myfaces/impl/trunk/src/java/org/apache/myfaces/context/servlet/ServletExternalContextImpl.java (original)
+++ myfaces/impl/trunk/src/java/org/apache/myfaces/context/servlet/ServletExternalContextImpl.java Mon Jul 18 11:43:43 2005
@@ -400,6 +400,16 @@
     {
         RequestDispatcher requestDispatcher
             = _servletRequest.getRequestDispatcher(requestURI);
+        
+        // If there is no dispatcher, send NOT_FOUND
+        if (requestDispatcher == null)
+        {
+           ((HttpServletResponse)_servletResponse).sendError(
+                  HttpServletResponse.SC_NOT_FOUND);
+
+           return;
+       } 
+        
         try
         {
             requestDispatcher.forward(_servletRequest, _servletResponse);