Opened 12 years ago
Closed 9 years ago
#11652 closed defect (wontfix)
null return value for JsonRestStore.getValue(..) cause error
| Reported by: | riceyeh | Owned by: | Kris Zyp |
|---|---|---|---|
| Priority: | high | Milestone: | tbd |
| Component: | DojoX Data | Version: | 1.5 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
Just find that it is impossible to get null value from a JsonRestStore?.getValue(..). Tracing the code, I find there are 2 places which does not check null value.
- line 415 in JsonRest?:
if(result.nodeType && result.cloneNode)
It should be changed to if(result && result.nodeType && result.cloneNode)
- line 223 in ServiceStore?:
delete item._loadObject;
It should be changed to item && delete item._loadObject;
Rice
Change History (3)
comment:1 Changed 12 years ago by
| Owner: | changed from Jared Jurkiewicz to Kris Zyp |
|---|
comment:2 Changed 11 years ago by
comment:3 Changed 9 years ago by
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
dojox/data is abandoned. Some dojox/data stores have been upgraded to use the Dojo Store API and can be found at https://github.com/kfranqueiro/dojo-smore.
Note: See
TracTickets for help on using
tickets.
line 390 method _processResults() in JsonRestStore?.js should check null value also: var count = results? results.length : 0;