Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 344241 - illegal API use in org.eclipse.gmt.modisco.infra.discoverymanager.ui.launch.utils.BooleanCellEditor
Summary: illegal API use in org.eclipse.gmt.modisco.infra.discoverymanager.ui.launch.u...
Status: CLOSED FIXED
Alias: None
Product: MoDisco
Classification: Modeling
Component: Infrastructure (show other bugs)
Version: 0.9.0   Edit
Hardware: All All
: P1 normal (vote)
Target Milestone: 0.9.1 RC2   Edit
Assignee: Nicolas Bros CLA
QA Contact:
URL: https://bugs.eclipse.org/bugs/show_bu...
Whiteboard:
Keywords:
Depends on: 334669
Blocks:
  Show dependency tree
 
Reported: 2011-04-29 04:46 EDT by Nicolas Bros CLA
Modified: 2012-05-29 08:28 EDT (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Nicolas Bros CLA 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 CLA 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 CLA 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 CLA 2011-06-09 05:22:20 EDT 
I remove the 343840 dependency since the issue has directly been fixed.
Comment 4 Nicolas Bros CLA 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 CLA 2012-05-29 08:28:15 EDT 
Issue solved and tested.