[ http://issues.apache.org/struts/browse/STR-2167?page=all ]
Michael Jouravlev closed STR-2167.
----------------------------------
Resolution: Won't Fix
Assignee: (was: Struts Developers)
Existing Action.execute() contract cannot be changed because it would break
backward compatibility. Struts 1.3 introduced Command class that can be used in
place of Action, it accepts one org.apache.struts.chain.contexts.ActionContext
parameter.
> change Action.execute() method parameter type
> ---------------------------------------------
>
> Key: STR-2167
> URL: http://issues.apache.org/struts/browse/STR-2167
> Project: Struts 1
> Issue Type: Improvement
> Components: Core
> Affects Versions: 1.1 Final
> Environment: Operating System: All
> Platform: All
> Reporter: Amir Pashazadeh
> Priority: Minor
>
> 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;
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
