[FIXED] GridSelectionModel firing Select TWICE with SelectionMode= Multi

[joshb]

Line 232 of GridSelectionModel,

I beleive it should be else if( !isSelected(sel)....
This is how handleMouseDown does it. Now, doSelect(...) gets fired once for mouseDown, once for mouseClick.

@SuppressWarnings("unchecked")
  protected void handleMouseClick(GridEvent<M> e) {
    if (isLocked()) {
      return;
    }
    if (!e.isRightClick() && selectionMode == SelectionMode.MULTI) {
      GridView view = grid.getView();
      M sel = listStore.getAt(e.getRowIndex());
      if (e.isControlKey() && isSelected(sel)) {
        doDeselect(Arrays.asList(sel), false);
      } else if (e.isControlKey()) {
        doSelect(Arrays.asList(sel), true, false);
        view.focusCell(e.getRowIndex(), e.getColIndex(), true);
      } else if(isSelected(sel) && !e.isShiftKey()){
        doSelect(Arrays.asList(sel), false, false);
        view.focusCell(e.getRowIndex(), e.getColIndex(), true);
      }
    }
  }

[darrellmeyer]

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