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;