silverstripe/silverstripe-framework — pull request #3234 by stephenmcm (2014-06-25 05:19:27+00:00)
Title:  FIX: Hierarchy markOpened incorrectly applying

Moved call to `markingClasses()` to after resolution of the nodes children has happened. This fixes the jstree-open applying before the tree has been checked.

Previously nodes with children would get `jstree-open` applied before checking for if the tree is open or not has completed.

Also changed `$node_threshold_total` from 50 to 30, as it's changed to 30 multiple times in the file. This should should be refactored to be more consistently calling the `$node_threshold_total`.

LeftAndMain was missing an argument with it's call to `getChildrenAsUL()`, this has been added though it should be affecting the `markOpened()` call.