I owe you one!

One problem I see off the bat is that your code doesn’t apply “Rule 2: Always return the same identical object from a call to getChildren() for a given value of the node argument.” Because you’re calling new ArrayCollection() every time, this would tend to throw off the bookkeeping inside Tree.

It would be easier for you if the collections inside your nodes (.circulars, .pages, etc.) were already instances of ArrayCollection. Then you wouldn’t have to wrap them in anything.

You weren’t specific about what “previously loaded” data meant, but the other thing I’d do is make absolutely sure that the events that fire when your loading operation completes are passing back the correct, newly loaded data set. There might be a problem in there somewhere. That’s beyond the scope of what I could help with though.

public function getChildren(node:Object, model:Object=null):ICollectionView
{
if (node is EnvironmentVO)
return ICollectionView(new ArrayCollection(EnvironmentVO(node).circulars));
if (node is CircularVO)
return ICollectionView(new ArrayCollection(CircularVO(node).pages));
if (node is PageVO)
return ICollectionView(new ArrayCollection(PageVO(node).items));
return null;
}
public function hasChildren(node:Object, model:Object=null):Boolean
{
if (node is EnvironmentVO) return true;
if (node is CircularVO)
{
if (CircularVO(node).pages.length==0){
CircularVO(node).pages.push(new PageVO({name:’loading’}));
var cmd:CmdEvent = new CmdEvent(Controller.EVENT_DataLoadPageList);
cmd.data = CircularVO(node);
Controller.getInstance().executeCommand(cmd);
return false;
}else if (CircularVO(node).pages.length==1){
if (PageVO(CircularVO(node).pages[0]).name==”loading”)
return false;
}
return true;
}
if (node is PageVO) return true; // Got To Get The Top Level Working First!!
if (node is ItemVO) return false;
return false;
}
public function isBranch(node:Object, model:Object=null):Boolean
{
if (node is EnvironmentVO) return true;
if (node is CircularVO) return true;
if (node is PageVO) return true;
if (node is ItemVO) return false;
return false;
}

I am trying to create a dinamic tree, but i just can get it to work. Can someone please post the code here, not everything but just the way i can handle results and insert them to my tree component. Btw, i got it to work with addChildAt, but it works if my result only has one XML row like “”.

thx in advanced

I did have to add a call to invalidateList() to get the expanded branch to refresh the display, revealing the newly added node(s):

Tree(event.target).dataDescriptor.addChildAt(selectedResultNode,{label:”item1 child”},0);
Tree(event.target).invalidateList();

You are using XMLListCollection, and we are using an ArrayCollection (or something like it). Perhaps that explains why you need your “lonely node” for the tree to work correctly. We didn’t seem to need such a node.

There are still some bugs in Tree that can occur when scrolling around as data is added. Overall it is still in a delicate condition and I don’t think this control is completely solid yet.

thanks for this post! I fully understand the theory behind that, but I don’t really get it how for example rule 1 could be translated to code. That what you described is in my problem exactly the case. isBranch returns true and hasChildren returns false. How do I force the ITreeDataDescriptor to return true on hasChildren? Do I have to write my own custom datadescriptor? Sorry for that dumb question but I really don’t get it.. Maybe you can post a little code snippet?
Thanks in advance!

I post about the SVN idea in flexcoders and Matt Chotin responded that they were thinking in do something like that, but they want to think a little about how to manage the support.

I think that Adobe will surprise again to their developers if they show such innovative way to do their support and I don’t think it will be bad for them, it should be very positive and make people trust them and go the flex way.

I’m afraid that the conservative way (give full updates from time to time) will be very bad, since people stuck and frustrade will drop flex and go other ways they know better and have less surprises.