Modify ↓
Ticket #5686 (closed enhancement: wontfix)
$.map is not consistent with $#each, $#map and $.each
| Reported by: | robertsosinski | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.4 |
| Component: | core | Version: | |
| Keywords: | Cc: | ||
| Blocking: | Blocked by: |
Description
I notice that with $.map, both the function parameter order and "this" reassignment is inconsistent with other enumerable methods, such as $#each, $#map and $.each. Attached is a script showing how this can make for easy mistakes.
I think it would make more sense to have the function passed to $.map take (index, element) instead of (element, index), and also reassign "this" to the current element worked with. Unless, there are specific technical reasons for the API to work in its current fashion.
Thanks,
Robert
Attachments
Change History
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Note: See
TracTickets for help on using
tickets.

