Sureshbabu pointed out something interesting in an earlier post.  The
TreeBacker class has the boolean values for leaf reversed.  So all of
the branch nodes have setLeaf(true) and visa versa.

I will fix this in CVS now, but there is another interesting point
here.  It doesn't matter to any of the simple examples!  This is
because the getType method is used to determine the facet (and hence
what type of icon to use).  The navigation icons (the plus/minus
icons) are determined by the child count.

IMO the getChildCount method and the getType methods could cover
pretty much every scenario that you would need isLeaf for.  I have
opened a JIRA issue (MYFACES-215) to suggest we deprecate the is Leaf
method (and eventually eliminate it.)  Please add comments there if
you have feelings on this one way or another.

sean

Reply via email to