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=30605>.
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=30605

change Action.execute() method parameter type

           Summary: change Action.execute() method parameter type
           Product: Struts
           Version: 1.1 Final
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Enhancement
          Priority: Other
         Component: Controller
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]


I think making parameters of Action.execute() method to be of class (for 
example ActionParameter) wrapping current parameters would be good idea.

It lets people who want to enhance the framework to extend the ActionParameter 
to pass additional data to Action.execute() methods in case of need without 
making the method signature change.

I mean something like:

public class ActionParameters {
  private ServletRequest servletRequest;
  private ServletResponse servletResponse;
  private ActionMapping actionMapping;
  private ActionForm actionForm;

  // the class should have setters and getters for these properties.
}

and change the signater of Action.execute() to

public ActionForward execute(ActionParameter actionParameter) throws Exception;

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to