dojox.data.ServiceStore null property error

[dustint]

There is a problem with the _processResults (line 229 of ServiceStore?.js) function in the dojox service store. Consider the following example: We have a json object passed from the server as follows:

{'foo': 'bar',

'description: null}

_processResults is recursively called on each attribute of our Json object, however, when _processResults is called on NULL the application errors out because results is null in the recursive call (and we try to get the length of NULL).

I propose that we check to make sure that the results[i] is not null before calling _processResults recursively.

BEFORE:

for (var i in results){

results[i] = this._processResults(results[i], deferred).items;

}

AFTER:

for (var i in results){

if(results[i] != null){

results[i] = this._processResults(results[i], deferred).items;

}

}

(Notice we don't need an else branch because we are simply writing back to the original object)

This is a problem in 1.2.3 and I still see it present in the trunk.

[Kris Zyp]

(In [18462]) Properly handle a top level null value in ServiceStore?._processResults, fixes #8949