[ 
https://issues.apache.org/struts/browse/STR-2314?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Paul Benedict closed STR-2314.
------------------------------

    Resolution: Won't Fix
      Assignee:     (was: Struts Developers)

This is simply not possible because it assumes only one form exists at a time.

> Make ActionForm accessible via request attribute for *each* request
> -------------------------------------------------------------------
>
>                 Key: STR-2314
>                 URL: https://issues.apache.org/struts/browse/STR-2314
>             Project: Struts 1
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 1.2.6 Beta
>         Environment: Operating System: All
> Platform: PC
>            Reporter: Matthew Sgarlata
>            Priority: Minor
>
> What would the Struts committers think of advertising the ActionForm at a set
> request attribute key for *each* request?  This would allow other frameworks 
> and
> toolkits easier access to the Struts ActionForm.  (I have a specific use case 
> in
> mind, if you'd like more info).  It would also dramatically simplify the code 
> in
> FormTag.initFormBean, which ends up having to work backwards to determine 
> where
> the ActionForm is after-the-fact, and eventually stores the ActionForm in a 
> set
> request attribute anyway (Constants.BEAN_KEY).
> Changes I propose (and I will submit patches if someone will commit them!):
> - Add code to the RequestProcessor.processActionForm method that causes it to
> store a reference to the form in request scope at Constants.BEAN_KEY.  
> (Existing
> code which stores the ActionForm at the request or session attribute 
> configured
> through the ActionMapping will remain unchanged).
> - Simplify FormTag.initFormBean to simply read the form bean from the request,
> rather than working backwards to figure it out.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to