Opened 11 years ago

Closed 10 years ago

Last modified 10 years ago

#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 11 years ago by Dustin Machi

Milestone: 1.31.4

comment:2 Changed 10 years ago by dante

Milestone: 1.41.5

comment:3 Changed 10 years ago by Dustin Machi

Resolution: fixed
Status: newclosed

(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

comment:4 Changed 10 years ago by bill

Milestone: 1.51.4
Note: See TracTickets for help on using tickets.