Closed Bug 12729 Opened 27 years ago Closed 27 years ago

hidden params are not submitted correctly (HTTP Get)

Categories

(Core :: DOM: Core & HTML, defect, P3)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: tiemann, Assigned: kmcclusk)

References

()

Details

Attachments

(1 file)

Hidden params in a form are sometimes not submitted correctly. I tracked it down and found that a "<p>" inbetween the parameters causes the hidden parameter before not to be submitted. The form uses HTTP-Get method. A test page to reproduce the bug is appended below. I tested on Win NT only. Mozilla M8 shows correct behaviour. M9 fails. Bjoern ----------------------------------------------------------------------------- <html><body> <hr> Form with two hidden parameters, a "&lt;p&gt;" inbetween: <form method=get action="http://www.mozilla.org/cgi-bin/script"> <input type=hidden name=param1 value=val1> <p> <input type=hidden name=param2 value=val2> <input type=submit value=submit> </form> <p>Loads URL (wrong!): <pre>http://www.mozilla.org/cgi-bin/script?param2=val2</pre> <hr> Form with two hidden parameters: <form method=get action="http://www.mozilla.org/cgi-bin/script"> <input type=hidden name=param1 value=val1> <input type=hidden name=param2 value=val2> <input type=submit value=submit> </form> <p>Loads URL (correct!): <pre>http://www.mozilla.org/cgi-bin/script?param1=val1&param2=val2</pre> <hr> </body></html>
Assignee: karnaze → kmcclusk
Reassigning to Kevin.
The problem is the mForm field for the hidden element under the paragraph is set to nsnull. The content sink sets the mForm field correctly, but later the sink does a CloseForm which demmotes the paragraph container causing the hidden element to be removed. The removal of the hidden element causes it's mForm field to be set to nsnull. The mForm fields needs to be reset to the appropriate container after it is removed in SinkContext::DemoteContainer;
Status: NEW → ASSIGNED
CC'ing vidur
*** Bug 12630 has been marked as a duplicate of this bug. ***
Target Milestone: M10
*** Bug 12853 has been marked as a duplicate of this bug. ***
*** Bug 12771 has been marked as a duplicate of this bug. ***
*** Bug 12727 has been marked as a duplicate of this bug. ***
Have fix waiting for tree to open
Status: ASSIGNED → RESOLVED
Closed: 27 years ago
Resolution: --- → FIXED
Fixed in Sept 1, 1999 3:28PM build.
Status: RESOLVED → VERIFIED
Verified using the 1999090908 build, NT. The attached test case does the right thing now.
Component: HTML: Form Submission → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: