Hi All,

I have a checkin request for Page Flow. The following patch file
(output from "svn diff") contains a minor fix.

Fixes:
- Modified code to support our RequestProcessor override method
PageFlowRequestProcessor.processMapping(). Now if there is no
Global.app for shared flow and a page has an unresolved action
we will produce the correct action not found error message.

CR: Rich
DRT/BVT: NetUI (WinXP)
BB: self (WinXP)

Thanks,
Carlin
Index: 
netui/src/pageflow/org/apache/beehive/netui/pageflow/PageFlowRequestProcessor.java
===================================================================
--- 
netui/src/pageflow/org/apache/beehive/netui/pageflow/PageFlowRequestProcessor.java
  (revision 47174)
+++ 
netui/src/pageflow/org/apache/beehive/netui/pageflow/PageFlowRequestProcessor.java
  (working copy)
@@ -967,13 +967,17 @@
         }
         else
         {
-            trySharedFlowAction( path, request, response );
+            if ( !trySharedFlowAction( path, request, response ) )
+            {
+                return processUnresolvedAction( path, InternalUtils.decodeURI( 
request ),
+                                                request, response, 
returningForm );
+            }
         }
         
         return null;
     }
     
-    protected void trySharedFlowAction( String actionPath, HttpServletRequest 
request, HttpServletResponse response )
+    protected boolean trySharedFlowAction( String actionPath, 
HttpServletRequest request, HttpServletResponse response )
         throws IOException
     {
         SharedFlowController sf = PageFlowUtils.getSharedFlow( request, 
getServletContext() );
@@ -1011,7 +1015,10 @@
             {
                 _log.error( "Could not forward to shared flow URI " + uri, e );
             }
+            return true;
         }
+
+        return false;
     }
     
     protected ActionMapping processUnresolvedAction( String actionPath, String 
originalRequestURI,

Reply via email to