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)
Change History (2)
Changed 12 years ago by
| Attachment: | dojo.data_JsonItemStore_20070612.patch added |
|---|
comment:1 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
(In [9047]) Trivial cleanup. fixes #3372