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 38343] - [extras] ParameterListActionDispatcher
Date Wed, 08 Mar 2006 06:50:28 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=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 niallp@apache.org  2006-03-08 06:50 -------
(In reply to comment #27)
> 2) The JavaDoc reads in regards to not finding a match: "null is returned
> which means the unspecified method will be invoked." I believe you have
> changed the functionality so null is no longer returned, which
> means "unspecified" is no longer invoked. Can you confirm this?

I've removed the "incorrect" check from getMethodName (it no longer throws an 
exception if the method is null) and implemented the "unspecified" method to 
log/throw an exception which doesn't expose the mapping's parameter. Thanks for 
spotting this.

Also when I looked at it, the getParameter() method was unecessary - I modified 
your version to throw an exception if the parameter was missing - but the 
inherited versions of this method were already doing that.

http://svn.apache.org/viewcvs?rev=384133&view=rev
http://svn.apache.org/viewcvs?rev=384134&view=rev

Thanks Paul.

-- 
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