dijit.tree.TreeStoreModel.onNewItem uses parentInfo in the wrong way

[coa]

The last three lines:

this.getChildren(parentInfo.item, dojo.hitch(this, function(children){
	this.onChildrenChange(parentInfo.item, children);
}));

...should be changed into:

this.getChildren(parentInfo.parent, dojo.hitch(this, function(children){
	this.onChildrenChange(parentInfo.parent, children);
}));

[bill]

Hmm, I'm not so sure. Can you provide a test case? Our existing tree tests are working.

[coa]

Ah, interesting.

The form of parentInfo in newItem is

{parent, attribute}

but most stores (e.g. ItemFileWriteStore?, AndOrWriteStore?, XmlStore?) change it into

{item, attribute}

before passing it to onNew

(JsonRestStore? ignores it completely and sends null - see #10706, #9801)

I suppose this ticket should be marked as invalid.

[bill]

OK, that explains it then, sorry it's confusing like that.