TreeNodeTask `check()` incorrectly removes `tree-node-child-unchecked` class from ancestor Nodes

• Type: Bug
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 2.0.x, 1.5.x, 1.0.x, master, master-deprecated, 2.5.x, 3.0.x
• Fix Version/s: 2.0.x, 1.5.x, master, master-deprecated, 3.0.x, 2.0.0.31-deprecated, 1.5.2.24, 3.1.x
• Component/s: Tree
• Labels: None
• Fix Priority: 3
• Git Pull Request: https://github.com/eduardolundgren/alloy-ui/pull/791

Note: The check behavior in 1.0.x is significantly different; I'm unable to confirm the same bug.

TreeNodeTask check() incorrectly removes tree-node-child-unchecked class from ancestor Nodes.

This happens when a descendant is checked while also having unchecked children.

Steps to Replicate:

1. Build a TreeView using Example Tree (below).
2. Check GreatGrandParent (descendants will be checked as well)
3. Uncheck ParentA (descendants will be unchecked as well)
4. Reckeck ChildA (descendants will be checked as well)

Expected Results:
GreatGrandParent should be [ - ]

Actual Results:
GreatGrandParent is [ x ]

State Key:

• [ _ ] Unckecked
• [ x ] Ckecked
• [ - ] Ckecked and some descendants unchecked (could be grayed out or a dash)

Example Tree:

• GreatGrandParent
  • GrandParentA
    • ParentA
      • ChildA
        • GrandChildA
      • ChildB
  • GrandParentB
    • ParentB