Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 322279

Summary:

[Tree] CheckedStateListeners broken for new tree

Product:

[RT] RAP

Reporter:

Cole Markham <cole>

Component:

RWT

Assignee:

Project Inbox <rap-inbox>

Status:

RESOLVED FIXED

QA Contact:

Severity:

blocker

Priority:

CC:

ivan, tuzcsaba

Version:

1.4

Target Milestone:

1.4 M2

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
Patch to fix the problem temporarily.	none

Description Cole Markham

2010-08-10 15:34:53 EDT

Checked state is not being propagated from the client to the server using the new tree from CVS head.

Comment 1 Austin Riddle

2010-08-10 15:40:45 EDT

Created attachment 176273 [details]
Patch to fix the problem temporarily.

The TreeItemLCA does not seem to be receiving checked events.

I couldn't quite figure out where the most appropriate place to propagate the event back to the LCA is. We needed this fixed today so here is a patch to add a hack that sends the event back from setChecked().

I am sure this is not the correct way or place, but we couldn't figure out how the update event scheme was intended to work.

Comment 2 Ivan Furnadjiev

2010-08-11 08:49:47 EDT

Fixed in CVS HEAD.

Comment 3 Tűz Csaba

2010-09-08 08:07:49 EDT

(In reply to comment #2)
> Fixed in CVS HEAD.

We got Eclipse RAP Tooling yesterday from repository, that's 08.11 15:?? build, but we still have this issue.
As I could figured out, SelectionEvent doesn't occurs on JS checkbox state change.
I suggest reopen

Comment 4 Ivan Furnadjiev

2010-09-08 08:18:09 EDT

Yes... it's possible to have this problem with 1.4M1 if you use "standard" "Client-side Library Variant" (check your launch configuration). The problem does not exist with 1.4M1 and "debug" variant. With CVS HEAD (and upcoming 1.4M2) this works in both library variants.

Comment 5 Ivan Furnadjiev

2010-09-08 09:18:22 EDT

I will change the target milestone to 1.4M2 to avoid such situations.