[FIXED] GXT B4: Trees + TreeStore

I think I have found a bug, and have spent an hour or so going through source. Actually two bugs - but please correct me if I'm wrong.

Bug #1: TreeBuilder does not bind the BaseTreeModel to the TreeItem. Perhaps it is not supposed to, but I don't see why this would be a problem, and given that you can pass a TreeModel into it, it would be the expected behaviour (at least by me!)

Bug #2: TreeStore + Binder (but no TreeLoader) - Does not add children of children of Model.
I've created a BaseTreeLoader, and bound it using a TreeBinder to a tree. I then do an RPC call to fetch the TreeModel, at which point I add the Root to the store.

This method cycles through the children and inserts the children of the Model into the tree, but it does not recursively go through its children. I'm pretty sure this is an oversight, and wouldn't be picked up in any of the Explorer examples. As a result, the Tree is only one level deep when I believe it should build it all out, as TreeBuilder does (and as I said, I tried to use TreeBuilder but found that it loses the relationship with the Model.

1. TreeBuilder now calls item.setData(model) on newly created tree items.
2. The 4 add and 4 insert methods in TreeStore have a new addChildren param. When set to true, the children of the models are added recursively.