Opened 8 years ago
Closed 8 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: |
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.
Change History (2)
comment:1 Changed 8 years ago by flesler
- need changed from Review to Commit
- Owner set to flesler
- Status changed from new to assigned
comment:2 Changed 8 years ago by flesler
- Resolution set to fixed
- Status changed from assigned to closed
Applied at [5826]. Passes all the tests.
Note: See
TracTickets for help on using
tickets.
Seems good to me. It's shorter, faster and what you need.