Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 66375 - [Dialogs] TaskPropertiesDialog and DialogMarkerProperties need a common superclass
Summary: [Dialogs] TaskPropertiesDialog and DialogMarkerProperties need a common super...
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P4 normal (vote)
Target Milestone: 3.3 M2   Edit
Assignee: Susan McCourt CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-06-09 14:36 EDT by Tod Creasey CLA
Modified: 2006-09-19 13:00 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tod Creasey CLA 2004-06-09 14:36:51 EDT
RC1

Despite these two classes being pretty much identical they share almost no 
code. TaskPropertiesDialog should be a subclass a DialogMarkerProperties.

As this is an API change post 3.0
Comment 1 Tod Creasey CLA 2004-06-09 15:26:25 EDT
Not for 3.0
Comment 2 Tod Creasey CLA 2004-06-28 11:28:02 EDT
Reopening now that 3.0 has shipped
Comment 3 Susan McCourt CLA 2006-07-06 19:28:54 EDT
marking for 3.3.
This came up while integrating marker undo...I implemented the undo in DialogMarkerProperties and then realized there was a duplicate API class that JDT and others were using.  This will get cleaned up as part of the undo marker work.

Tod, is there any history I should know about..ie, why there is an internal dialog and an API dialog doing the same thing?
Comment 4 Susan McCourt CLA 2006-08-18 18:06:00 EDT
this will go in for M2 as part of marker operation undo support.
Comment 5 Susan McCourt CLA 2006-08-22 19:09:02 EDT
Fixed in HEAD, >20060822.
The DialogMarkerProperties hierarchy was reorganized so that all marker dialogs inherit this implementation rather than copying/changing the implementation.  This means that some public classes, such as TaskPropertiesDialog and BookmarkPropertiesDialog, are inheriting API implementations from a private superclass.

Per discussions with Boris, in these cases, the API methods are overridden in the public subclasses, calling the super implementations.  The internal superclasses are documented to emphasize that public API classes are dependent on their implementation, and the specific methods in question are also specially documented.
Comment 6 Susan McCourt CLA 2006-09-19 13:00:51 EDT
verified through source code inspection,  I20060919-0010