Skip to main content

Bug Tracker

Side navigation

#7275 closed bug (duplicate)

Opened October 21, 2010 10:08PM UTC

Closed October 26, 2010 11:35PM UTC

Last modified March 14, 2012 01:22AM UTC

jQuery.fn.find() returns no results in Firefox for forms that contain input element with name "id"

Reported by: hallettj@gmail.com Owned by:
Priority: undecided Milestone: 1.5
Component: traversing Version: 1.4.3
Keywords: fn find id regression Cc:
Blocked by: Blocking:
Description

jQuery: 1.4.3

browser: Firefox 3.6.11

OS: Ubuntu 9.10 and Ubuntu 10.04

Steps to reproduce in Firefox 3.6:

1. Create a form element with some child elements.

2. Inside the form, place an input element with a name attribute set to "id".

3. Using jQuery reference the form and try to find() any set of elements inside of it. For example: $('#myForm').find('input').

Step 3 will yield an empty result set no matter what selector is given to find().

This issue does not appear to affect jQuery 1.4.2.

The problem is that jQuery tries to change the id attribute of the form and to use that new id value as a point of reference when invoking querySelectorAll(). Because the form has an input element named "id" accessing the "id" attribute of the form access that input element instead of the form's own element id.

Test case:

http://jsfiddle.net/hallettj/pWNFe/

Running the above test case in Firefox 3.6 outputs "false" in the console. In Chrome 7 the same test case outputs "true".

Attachments (0)
Change History (7)

Changed October 21, 2010 10:31PM UTC by addyosmani comment:1

keywords: → fn find id regression needsreview
status: newopen

Flagging as a regression we need to review. This is related to the fact that DOM 0 makes frames automatically expand onto the Doc object and input elements more or less automatically expand onto the form element. It's an issue we've seen in at least one other ticket but this is not a dupe.

Changed October 22, 2010 06:41AM UTC by addyosmani comment:2

#7280 is a duplicate of this ticket.

Changed October 22, 2010 06:02PM UTC by rwaldron comment:3

resolution: → duplicate
status: openclosed

Changed October 26, 2010 11:34PM UTC by rwaldron comment:4

resolution: duplicate
status: closedreopened

Changed October 26, 2010 11:35PM UTC by rwaldron comment:5

component: unfiledtraversing
resolution: → duplicate
status: reopenedclosed

Changed October 26, 2010 11:35PM UTC by rwaldron comment:6

Duplicate of #7212.

Changed November 02, 2010 02:16AM UTC by dmethvin comment:7

keywords: fn find id regression needsreviewfn find id regression