DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=35703>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=35703 ------- Additional Comments From [EMAIL PROTECTED] 2006-01-29 16:11 ------- There's still a huge misunderstanding here. This bug report identifies an issue with current Tiles integration with Struts up to 1.2.x. Anyway, I decided to provide a simple web app example in the next few days to help you guys really understand that. I believe it's always worth it when there's a malfunction somewhere to insist on tackling the issue. Anyway we should stop discussing Tiles standalone components here, which is irrelevant, but keep our minds focussed on the components of Tiles responsible for integration with Struts i.e. Tiles plugin. AFAIK in current Tiles plugin architecture there are 3 components which actually cause the issue described in this bug report: - TilesRequestProcessor (extends RequestProcessor) - TilesUtilStrutsImpl (extends TilesUtilImpl) - TilesUtilStrutsModuleImpl (extends TilesUtilStrutsImpl) The reason for that being that all calls to doForward() and doInclude() methods end up in the above 3 components + TilesUtilImpl (used for Tiles standalone). TilesUtilStrutsImpl and TilesUtilStrutsModuleImpl are the substitutes for TilesUtilImpl when it comes to Tiles + Struts integration. And that's where the contract of the RequestProcessor gets broken hence this bug report. It then seems as the natural places to issue the actual calls to TilesRequestProcessor, exactly where the person who designed TilesUtilStrutsModuleImpl and TilesUtilStrutsModuleImpl classes in the first place already planned them to be (again see comments in TilesUtilStrutsModuleImpl header). Now if you want to re-design Tiles + Struts integration components that's a completely different matter - and I've got no objection to that. But that's got nothing to do with current issue which affects Tiles integration with existing releases of Struts up to 1.2.x. And this patch merely aims at fixing the issue in that context, no more. -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
