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

Bug 327628

Summary:	ComboRidget: do not fire event on UI modification when output only
Product:	[RT] Riena	Reporter:	Elias Volanakis <elias>
Component:	ridget	Assignee:	Elias Volanakis <elias>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	unspecified
Target Milestone:	2.1.0.M1
Hardware:	PC
OS:	Windows Vista
Whiteboard:

Description Elias Volanakis

2010-10-13 01:40:25 EDT

To reproduce:

0. Create a ComboRidget
1. Add listener to selection event
2. Make read only
3. Change from UI (the change will be reverted)

However two selection events are fired. They should not be fired.

Note that we still want selection events to be fired when the selection is changed via API (i.e. setSelection(...)).

Comment 1 Sabine Achilles

2010-11-19 03:56:23 EST

I have made some changes to the AbstractComboRidget to deal with another Bug. The result of these changes was that this bug here is fixed as well. I commented in the test you wrote for it.