#8175 closed defect (fixed)
JsonPathStore Scoping issue
Reported by: | dylan | Owned by: | Dustin Machi |
---|---|---|---|
Priority: | high | Milestone: | 1.4 |
Component: | DojoX Data | Version: | 1.2.1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
In lines 954-955 of JsonPathStore?.js:
this.index[id] = null; delete this.index[id]
id is out of scope.
Per kzyp, perhaps you want:
delete this.index[item[this.idAttribute]];
Change History (4)
comment:1 Changed 12 years ago by
Milestone: | 1.3 → 1.4 |
---|
comment:2 Changed 11 years ago by
Milestone: | 1.4 → 1.5 |
---|
comment:3 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:4 Changed 11 years ago by
Milestone: | 1.5 → 1.4 |
---|
Note: See
TracTickets for help on using
tickets.
(In [20831]) jsonPathStore is out of sync due to the more advanced dojox.json capabilities. To avoid confusion with these, I'm removing this widget as I dont' plan to support it going forward. There is an equivalent widget which uses dojox.json ref and query in dojoc/dmachi/data called JsonStore? for those who still need this capability. While they are not drop in replacements of each other, they are pretty close and it should not take a significant amount of time to convert, and the benefits gained in performance and querying flexibility are worth it. closes #7473 closes #8175 closes #7331 closes #7403 closes #8312 closes #8334 closes #9976 closes #10290