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

Bug 344241

Summary:	illegal API use in org.eclipse.gmt.modisco.infra.discoverymanager.ui.launch.utils.BooleanCellEditor
Product:	[Modeling] MoDisco	Reporter:	Nicolas Bros <nicolas.bros>
Component:	Infrastructure	Assignee:	Nicolas Bros <nicolas.bros>
Status:	CLOSED FIXED	QA Contact:
Severity:	normal
Priority:	P1	CC:	fabien.giquel, gdupe, hugo.bruneliere, modisco.web-inbox
Version:	0.9.0
Target Milestone:	0.9.1 RC2
Hardware:	All
OS:	All
URL:	https://bugs.eclipse.org/bugs/show_bug.cgi?id=343840
Whiteboard:
Bug Depends on:	334669
Bug Blocks:

Description Nicolas Bros

2011-04-29 04:46:48 EDT

org.eclipse.gmt.modisco.infra.discoverymanager.ui.launch.utils.BooleanCellEditor extends org.eclipse.jface.viewers.ComboBoxCellEditor, which is marked "@noextend This class is not intended to be subclassed by clients."

Comment 1 Fabien Giquel

2011-04-29 06:05:58 EDT

The oegm.infra.discoverymanager.ui.launch.utils.BooleanCellEditor is deprecated and will disappear. I had the dependency to the "[Deprecated]" reminder bugzilla.

Comment 2 Nicolas Bros

2011-06-08 10:24:39 EDT

I fixed the warning in revision 4468 by copying code from the superclass, and I added a unit test.
But this SWTBot test is not yet launched during the build because SWTBot tests are not run during MoDisco builds yet.

Comment 3 Fabien Giquel

2011-06-09 05:22:20 EDT

I remove the 343840 dependency since the issue has directly been fixed.

Comment 4 Nicolas Bros

2011-08-30 05:01:42 EDT

The SWTBot unit test is now run during the Hudson builds, and it passes successfully.

Comment 5 Hugo Bruneliere

2012-05-29 08:28:15 EDT

Issue solved and tested.