Skip to main content

Bug Tracker

Side navigation

#3249 closed enhancement (fixed)

Opened August 12, 2008 10:02PM UTC

Closed August 13, 2008 01:45AM UTC

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:
Description

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 (0.4 KB) - added by nathanhammond August 12, 2008 10:03PM UTC.

    Patch

Change History (2)

Changed August 13, 2008 01:20AM UTC by flesler comment:1

need: ReviewCommit
owner: → flesler
status: newassigned

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

Changed August 13, 2008 01:45AM UTC by flesler comment:2

resolution: → fixed
status: assignedclosed

Applied at [5826]. Passes all the tests.