Bug Tracker

Ticket #5686 (closed enhancement: wontfix)

Opened 5 years ago

Last modified 4 years ago

$.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

map_example.html Download (1.6 KB) - added by robertsosinski 5 years ago.

Change History

Changed 5 years ago by robertsosinski

comment:1 Changed 5 years ago by dmethvin

  • Type changed from bug to enhancement

It may not be consistent, but there is probably a lot of code that would break if this was changed.

comment:2 Changed 4 years ago by snover

  • Status changed from new to closed
  • Resolution set to wontfix

There is no way to fix this and maintain backward-compatibility.

Note: See TracTickets for help on using tickets.