Bug Tracker

Opened 14 years ago

Closed 14 years ago

#3249 closed enhancement (fixed)

Slight change to jQuery.fn.serializeArray()

Reported by: nathanhammond Owned by: flesler
Priority: minor Milestone: 1.3
Component: ajax Version: 1.2.6
Keywords: Cc:
Blocked by: Blocking:


I'm writing a plugin that would emulate the Form.elements array for any arbitrary element that is a descendant of a form. This new array would contain only those form elements that descend from it, in semantic order.

The benefits of this are right now being negated by one line in jQuery.fn.serializeArray() where it queries the nodeName to see if it is a form rather than querying the existence of the property.

I've attached a patch that would make that change.

Attachments (1)

serialize.diff (400 bytes) - added by nathanhammond 14 years ago.

Download all attachments as: .zip

Change History (3)

Changed 14 years ago by nathanhammond

Attachment: serialize.diff added


comment:1 Changed 14 years ago by flesler

need: ReviewCommit
Owner: set to flesler
Status: newassigned

Seems good to me. It's shorter, faster and what you need.

comment:2 Changed 14 years ago by flesler

Resolution: fixed
Status: assignedclosed

Applied at [5826]. Passes all the tests.

Note: See TracTickets for help on using tickets.