You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by bu...@apache.org on 2006/03/07 18:17:23 UTC

DO NOT REPLY [Bug 38343] - [extras] ParameterListActionDispatcher

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-07 17:17 -------
The one thing I don't understand about this is why it is configured through the 
ActionMapping's parameter?

The method names in the Action can't be configured so the only value I could 
see is if the Action is reused and you wanted to "obfuscate" in a different way 
for different mappings. But I don't really see much need for that.

How about adding the parameter list to the constructor, so you would do 
something like...

    private ActionDispatcher dispatcher
                    = new ParameterListActionDispatcher(this,
                      "save,back,recalc=recalculate,default=save");

or maybe add methods...

  public class FooAction extends Action {

    private ParameterListActionDispatcher dispatcher
                    = new ParameterListActionDispatcher(this);

    public FooAction() {
        dispatcher.addMethodParam("save");
        dispatcher.addMethodParam("back");
        dispatcher.addMethodParam("recalc", "recalculate");
        dispatcher.addDefaultParam("save");
    }

  }

...or perhaps both.

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