Description Stefan Lötscher 2005-07-01 18:19:23 EDT

If you want to load data in TreeViewer in a dynamic way e.g. lazy loading from a server in RCP 
applications, the implementation of hasChildren() in BaseWorkbenchContentProvider is not optimal. 
Actually it is implemented as follows:

public boolean hasChildren(Object element) {
    return getChildren(element).length > 0;
}

better would be:

public boolean hasChildren(Object element) {
        IExtendedWorkbenchAdapter adapter = (IExtendedWorkbenchAdapter) getAdapter(element);
        if (adapter != null) {
            return adapter.hasChildren(element);
        }
        return getChildren(element).length > 0;
    }

This gives us the possibility to load data dynamically only if the node is really expanded. Additionally, it 
looks inconsistent if that method is not adaptable but all others are.