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

Bug 514014

Summary:	ReadOnlySetWrapperEx does not override #getReadOnlyProperty(), therefore, listeners are not notified
Product:	[Tools] GEF	Reporter:	Matthias Wienand <matthias.wienand>
Component:	GEF Common	Assignee:	gef-inbox <gef-inbox>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	1.1.0
Target Milestone:	5.0.0 (Oxygen) M7
Hardware:	All
OS:	All
Whiteboard:

Description Matthias Wienand

2017-03-21 13:52:39 EDT

The ReadOnlySetWrapperEx class does define a ReadOnlyPropertyImpl that should be constructed/returned within #getReadOnlyProperty(). However, as that method is not overridden, the super class's #getReadOnlyProperty() method will construct and return its own ReadOnlyPropertyImpl. As the listener notification is correctly overridden by ReadOnlySetWrapperEx, the listeners registered at the read-only-view of the property will never be notified.

Comment 1 Matthias Wienand

2017-03-22 08:14:42 EDT

I implemented #getReadOnlyProperty() as outlined and added a test for the functionality. The code is published on the master branch, therefore, I resolve this ticket as fixed for 5.0.0 M7.