Opened 11 years ago
Closed 11 years ago
#12189 closed defect (fixed)
Minor dojo.store issues
Reported by: | vladar | Owned by: | Kris Zyp |
---|---|---|---|
Priority: | high | Milestone: | 1.6 |
Component: | Data | Version: | 1.6.0b1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
While reviewing dojo-1.6.0b1 I've noticed several minor issues:
- "id" variable is global in Memory store, method "put" (line 47)
- there is a trailing comma in the object definition, which messes up IE in JsonRest? store, method "put" (line 58)
Also, looks like there is no dedicated component in Trac yet for dojo store API, so assigned to dojo.data
Attachments (1)
Change History (3)
comment:1 Changed 11 years ago by
Owner: | changed from Jared Jurkiewicz to Kris Zyp |
---|
Changed 11 years ago by
Attachment: | store-lint-doc-fixes.patch added |
---|
comment:2 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
Updated to also remove EOL whitespace as pointed out by dante. Also, note: will have to view raw, trac doesn't seem to like big patches.