I agree with Sean that removing the if statement would make the tree really slow with server-side toggle. The code is correct here.
With client-side toggle we may have to change this ... but if we get a ajax tree working that would be the best solution.
