[FIXED][GXT 2.0.1] TreeGrid and toggle on double click

[VadimV1144]

Hi, this is not a bug, but a code consistency issue.

In the class TreeGrid :

Method

 public void toggle(M model) {..}

is called within the

protected void onClick(GridEvent<M> e){...}

to perform the tree expansion/collapse when the little arrow is single clicked.
However, the Double Click is performed the following method is called

 
protected void onDoubleClick(GridEvent<M> e) {
    super.onDoubleClick(e);
    M m = e.getModel();
    if (m != null) {
      TreeNode node = findNode(m);
      setExpanded(node.m, !node.expanded);
    }
  }

As you can see it totally duplicates the

 public void toggle(M model) {..}

and if I want to change the toggle behavior I have to override 2 methods instead of one.
Please change above method to following:

 
protected void onDoubleClick(GridEvent<M> e) {
    super.onDoubleClick(e);
    toggle(e.getModel());
  }

[darrellmeyer]

This bug is fixed in the releases/2.1 branch and is now available in the 2.1.1-beta release available here.