ItemFileReadStore.getValues should use slice

[Millennium]

Today I was wondering the Reverse Reference Map wasn't set for an Item I inserted in the parent with the following code:

var tarRef = store.getValues(tarItem, "children");
tarRef.splice(tarRef.length - 1, 0, item)
store.setValues(tarItem, "children", tarRef);

Because the ItemFileReadStore? uses getValues in his _setValueOrValue function. (Which I altered to the new situation with the above splice code.) And so the following Itemfilewritestore code fails to update the RRM.

var map = {};
					dojo.forEach(oldValues, function(possibleItem){
						if(this.isItem(possibleItem)){
							var id = this.getIdentity(possibleItem);
							map[id.toString()] = true;
						}
					}, this);
					dojo.forEach(newValueArray, function(possibleItem){
						if(this.isItem(possibleItem)){
							var id = this.getIdentity(possibleItem);
							if(map[id.toString()]){
								delete map[id.toString()];
							}else{
								this._addReferenceToMap(possibleItem, item, attribute); 
							}
						}
					}, this);
					for(var rId in map){
						var removedItem;
						if(this._itemsByIdentity){
							removedItem = this._itemsByIdentity[rId];
						}else{
							removedItem = this._arrayOfAllItems[rId];
						}
						this._removeReferenceFromMap(removedItem, item, attribute);
					}

Because OldValues? and New Values are the same array pointer....

[Jared Jurkiewicz]

(In [20989]) Committing in minor tweak. fixes #10474 \!strict

[Jared Jurkiewicz]

(In [20990]) Committing in minor tweak. refs #10474 \!strict

[nic]

[20990] broke dojox/data/demos/demo_LazyLoad.html
Open the demo page then click on "North America", then "Mexico" or "Canada", the children aren't loaded.
Any ideas?
Thanks!

[Jared Jurkiewicz]

(In [21043]) Fixing problem with lazy-load demo caused by change to use slice in getValues. refs #10474 \!strict