Description Vaughn 2009-05-01 23:03:49 EDT

Created attachment 134143 [details]
Updated TreeItemUtil.js

Environment: Firefox 3/Internet Explorer 7/RAP 1.2M6

Steps to reproduce:
1.  Create a Tree widget (Note: not a TreeViewer) and populate with some TreeItems
2.  Select one of the items
3.  Remove the selected item and replace with another TreeItem

Once this happens, selection is no longer posted to the server and getSelection returns an empty array even when items are selected in the browser.

I have traced through the code and found the problem to be in the following file from org.eclipse.rap.rwt.q07_1.2.0.20090319-1159.jar:
org/eclipse/swt/TreeItemUtil.js

hijackTreeItemSelection should check if the TreeItem is disposed as follows:

hijackTreeItemSelection : function( item ) {
      item._applySelected = function(value, old)
        {
          // Ensure item is not disposed
          if (this.isDisposed())
               return;
          if (value)
          {
            this.addState("selected");
            this._labelObject.addState("selected");
          }
          else
          {
            this.removeState("selected");
            this._labelObject.removeState("selected");
        }

... etc