struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 35703] - [tiles] TilesUtilImpl doInclude() should call TilesRequestProcessor doInclude()... but it doesn't
Date Fri, 27 Jan 2006 17:04:01 GMT
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 niallp@apache.org  2006-01-27 18:04 -------
(In reply to comment #7)
> Agree with Niall's comments on updating the patch.
> I finally add some time to work on it (sorry for the delay).

My comment #5 was this should be closed as WONTFIX or INVALID!

(In reply to comment #13)
I agree with what Greg has said in comment #11 and comment #13.

As Greg said, perhaps we could modify the RequestProcessor to call TilesUtil 
(haven't looked at the code recently). The question though is "is it worth 
it"? From what I see all it would do is save you having to duplicate your 
custom code in two places - that doesn't seem too much of a hardship to me. 
With the plan to develop "standalone tiles" which should replace this its 
probably not worth the effort IMO and we should just close this as WONTFIX.

-- 
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: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


Mime
View raw message