Bug 11078 – document.all.item(someString) doesn't work

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Jared Roussel]

There are a few sites that I've noticed so far that won't post the form when the submit button is pressed using Safari. I have noticed three instances of this so far. One is on eBay when you want to add another category to your sell listing, one is during a session of an account I login to, but most importantly, a verifiable and specific one is at http://dnb.com/globalsignin/forgetpassword.asp. The submit button will just not post the form on Safari. I've noticed that all of the sites in these examples are ASP pages.

The form in the URL fails to submit when the Submit button is clicked due to a JavaScript error. The root cause is that it uses document.all.item("name"). Firefox and Opera allow access by name using item() whereas WebKit allows it only using namedItem().

The form can be sumbitted by pressing Enter when the text field has focus. This obviously bypasses the Submitt button's onclick handler, and results in the same form loading again without any message (the behavior is the same in Firefox and WinIE as well).

Created attachment 10834 [details]
Test item() by name

Created attachment 12313 [details]
Make item() fall back to namedItem()

This patches implements Firefox's behavior. IE apparently does not try to convert strings to numbers, making it fail the first case in the attached layout test.

[Darin Adler]

Comment on attachment 12313 [details]
Make item() fall back to namedItem()

r=me

[Alexey Proskuryakov]

Do we need a similar quirk in other item() methods (e.g., HTMLSelectElement::item()?)

+            return toJS(exec,coll.item(index));

Should have a space after a comma.

[Alexey Proskuryakov]

Comment on attachment 12313 [details]
Make item() fall back to namedItem()

Committed revision 18715.

Mitz has suggested leaving this bug open as a reminder to research the behavior of other item() methods. Clearing the review flag from a landed patch.

[Alexey Proskuryakov]

See also: http://ln.hixie.ch/?start=1161042744&count=1

[Eric Seidel (no email)]

Hum... unfortunately leaving a bug open after landing the patch means that it appears in the "forgotten bugs" queue.  Perhaps there could be a new bug to cover any remaining issues in item() and this could be closed?

[Alexey Proskuryakov]

Obviously, this bug didn't serve as a good reminder, but it's still wrong to have the quirk in only one version of item().

[Alexey Proskuryakov]

We now have a specific bug tracking the incorrectness of fallback, bug 74468.

Marking FIXED since keeping this bug open proved confusing, and didn't result in any action. Renaming to be clear about what was fixed.