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=22357>. 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=22357 ------- Additional Comments From [EMAIL PROTECTED] 2006-03-10 04:59 ------- Paul's point is a good one - the changes made to how "cancel" is handled for bug 38374 need to be considered when deciding on what action to take here. Having said that I don't object to Michaels suggestion - the "cancelleable" feature only applies when struts handles validation - which requires an ActionForm and validate set to true - otherwise its ignored. Perhaps the way to ensure everyone is kept happy is to implement this for the ComposableRequestProcessor in Struts 1.3 by factoring it out into a separate Command. That way anyone not liking the default behaviour can easily change it. Anyone wanting this in the original RequestProcessor could easily implement it either by having a custom RequestProcessor - or have a "base" Action that actually checks the request parameters in isCancelled(). i.e... protected boolean isCancelled(HttpServletRequest request) { if ((request.getParameter(Constants.CANCEL_PROPERTY) != null) || (request.getParameter(Constants.CANCEL_PROPERTY_X) != null)) { return true; } else { return false; } } -- 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]
