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=38343>. 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=38343 ------- Additional Comments From [EMAIL PROTECTED] 2006-02-10 03:04 ------- (In reply to comment #4) > This is interesting. I did not write this class with the intention of > creating a > rendering phase (parameter present) and a view phase (parameter absent), but I > suppose it could be leverage for such a task. You're probably more suited to > come up with a living example of this -- and I would like to see it I have one ;-) I was thinking about putting it in new version of Struts Dialogs (which is 1.25), but now I think I will start a new major version number, because I can throw away at least half of my code. This will break compatibility, so I will have to support two major versions, but I cannot help it, I like your solution so much :) > -- but I am > not sure if I want to add a "getView" method on this. Perhaps that's an > implementation that could come from a subclass, and that's what I'd actually > prefer as I think about your suggestion. I agree. After all, not everybody does things like me ;) > As you know the default is to return null if it can't find any match. Because > this is a 2nd generation subclass of DispatchAction, I think the "unspecified" > method is called but it's not directly by this. Right, that is what I meant: to have default unspecified() that would call default and empty getView(). If a user of this class overrides unspecified(), then he will not be bothered about getView() stuff. If the user overrides neither unspecified() nor getView(), then default getView() would throw exception like unspecifed() throws now (it does, does it?) But you may be right, this clutters things up, adding nothing more but semantic icing. I will be quite happy having just unspecified() for my no-param render phase. -- 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]
