Opened 12 years ago

Closed 12 years ago

#3372 closed defect (fixed)

0.9: Remove _itemMap in JsonItemStore, not needed with keys as numerics.

Reported by: Jared Jurkiewicz Owned by: Jared Jurkiewicz
Priority: high Milestone: 0.9beta
Component: Data Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description

0.9: Remove _itemMap in JsonItemStore?, not needed with keys as numerics.

The original idea behind having the map was so that non-numrerics could be used as the ids as well in the O(1) isItem, but that wasn't how it ended up being implemented. So, realistically, the single arrayOfAllItems can be used in its place. Removing the map and slightly altering isItem. More efficient, too.

Attachments (1)

dojo.data_JsonItemStore_20070612.patch (1.2 KB) - added by Jared Jurkiewicz 12 years ago.

Download all attachments as: .zip

Change History (2)

Changed 12 years ago by Jared Jurkiewicz

comment:1 Changed 12 years ago by Jared Jurkiewicz

Resolution: fixed
Status: newclosed

(In [9047]) Trivial cleanup. fixes #3372

Note: See TracTickets for help on using tickets.