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/02/11 21:54:14 UTC

DO NOT REPLY [Bug 38620] New: - ActionForm should not be populated automatically [Struts Classic 1.x]

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

           Summary: ActionForm should not be populated automatically [Struts
                    Classic 1.x]
           Product: Struts
           Version: Unknown
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Action
        AssignedTo: dev@struts.apache.org
        ReportedBy: jmikus@gmail.com


Whenever a request is submitted to an Action, a corresponding ActionForm is
always populated, despite the type of request or despite of whether request
contains Cancel event or not. This brings a lot of problems related to
refreshing the same page, or forwarding to another action, or accepting invalid
input when Cancel was activated.

For compatibility reasons, autopopulation of an ActionForm should be kept by
default, but it should be possible to turn it off either on application level
using <controller> element or on action level using <action> element, or both.

Struts allows to perform validation explicitly, the same should be allowed for
form population. A developer knows better when to accept the input and when not to.

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

DO NOT REPLY [Bug 38620] - ActionForm should not be populated automatically [Struts Classic 1.x]

Posted by bu...@apache.org.
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=38620>.
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=38620





------- Additional Comments From martinc@apache.org  2006-02-12 06:24 -------
You can already avoid automatic population of a form bean by simply not
associating a form bean with the action mapping. I'm not sure I see a need for
another way to do this.

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

DO NOT REPLY [Bug 38620] - ActionForm should not be populated automatically [Struts Classic 1.x]

Posted by bu...@apache.org.
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=38620>.
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=38620


jmikus@gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |INVALID




------- Additional Comments From jmikus@gmail.com  2006-02-14 20:15 -------
(In reply to comment #1)
> You can already avoid automatic population of a form bean by simply not
> associating a form bean with the action mapping. I'm not sure I see a need for
> another way to do this.

Right. I can borrow the processActionForm() and processPopulate() code from
RequestProcessor. For output, I use full name/property specifiers anyway.
Closing this bug as INVALID. Should have asked in the usergroup first.

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