Since Struts1.1 does not require to use the servlet class org.apache.struts.tiles.ActionComponentServlet for using Tiles, the method Common.verifyForwardPath() cannot verify Tiles action mapping if the servlet instance is the one of the org.apache.struts.action.ActionServlet. It is more flexible if we can add a overloaded method verifyForwardPath() to allow passing a boolean arguement "isTitlesMapping" and overload the method verifyForward() with this argument to allow the caller to specify if the forward mapping is for Tiles or not.
Regards
Qingxian
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Since Struts1.1 does not require to use the servlet class org.apache.struts.tiles.ActionComponentServlet for using Tiles, the method Common.verifyForwardPath() cannot verify Tiles action mapping if the servlet instance is the one of the org.apache.struts.action.ActionServlet. It is more flexible if we can add a overloaded method verifyForwardPath() to allow passing a boolean arguement "isTitlesMapping" and overload the method verifyForward() with this argument to allow the caller to specify if the forward mapping is for Tiles or not.
Regards
Qingxian
I will be doing something just like that in the next release. The ActionComponentServlet check was, to say the least, a hack.
Thanks for the input!
-d.