Form submit should not append vars for javascript: actions

VERIFIED FIXED in Future

Status

()

Core
HTML: Form Submission
P3
normal
VERIFIED FIXED
18 years ago
17 years ago

People

(Reporter: Eric Pollmann, Assigned: Eric Pollmann)

Tracking

Trunk
Future
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: Fix in hand, URL)

Attachments

(1 attachment)

(Assignee)

Description

18 years ago
When a form has ACTION="javascript://..." METHOD=GET, we are appending the URI
encoded variables and values in the form to the javascript expression.  This
causes javascript errors.
(Assignee)

Comment 1

18 years ago
See bug 26587
Status: NEW → ASSIGNED
(Assignee)

Comment 2

18 years ago
See related bug 25330
(Assignee)

Updated

18 years ago
Whiteboard: Fix in hand
Target Milestone: M15
(Assignee)

Comment 3

18 years ago
Created attachment 6922 [details]
Test Case
(Assignee)

Comment 4

18 years ago
Fix checked in.  Submitting this form should display javascript:23 (a page with
the text '23' on it and nothing else).  The URL bar should *not* read
javscript:23?Sub=Dimension.  Thanks!
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 5

17 years ago
Updating QA contact.
QA Contact: ckritzer → vladimire

Comment 6

17 years ago
Reopening The testcase is not working. The form isnt even submitted or at least 
nothing changes when submit button is clicked. No javascript errors are 
generated though.

Ran this test in N4.7 - JavaScript error.
Ran this test in IE5  - Works like it should.
N6 build 2000-09-25-08-M18 on Windows 98  - No errors, nothing happens.
N6 build 2000-09-19-21-M18 on Linux RedHat6.2 - No errors, nothing happens.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 7

17 years ago
Removing past milestone
Status: REOPENED → ASSIGNED
Target Milestone: M15 → ---

Comment 8

17 years ago
vladimire@netscape.com: does it still happen with the most recent build?
Target Milestone: --- → Future

Comment 9

17 years ago
This works now, marking fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago17 years ago
Resolution: --- → FIXED

Comment 10

17 years ago
Marking verified. Vladimire tested it. Its Fixed.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.