Bug 201125 - External Discouraged Access warnings in o.e.d.connectivity.ui
Summary: External Discouraged Access warnings in o.e.d.connectivity.ui
Status: RESOLVED FIXED
Alias: None
Product: Data Tools
Classification: Tools
Component: Connectivity (show other bugs)
Version: 1.5   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: 1.7M5   Edit
Assignee: Brian Fitzpatrick CLA
QA Contact:
URL:
Whiteboard:
Keywords: api
Depends on: 208830
Blocks: 201136
  Show dependency tree
 
Reported: 2007-08-24 15:42 EDT by John Graham CLA
Modified: 2009-01-06 15:44 EST (History)
2 users (show)

See Also:


Attachments
Patch (6.31 KB, patch)
2008-12-23 21:57 EST, Linda Chan CLA
no flags Details | Diff
Revised Patch (6.68 KB, patch)
2009-01-06 15:42 EST, Linda Chan CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description John Graham CLA 2007-08-24 15:42:42 EDT
org/eclipse/datatools/connectivity/ui/actions/ViewPropertyAction.java
---------------------------------------------------------------------
line 109, The constructor PropertyDialog(Shell, PreferenceManager, ISelection) is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 100, The constructor PropertyPageManager() is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 106, The method contribute(PropertyPageManager, Object) from the type PropertyPageContributorManager is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 82, The method getApplicableContributors(Object) from the type PropertyPageContributorManager is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 82, The method getManager() from the type PropertyPageContributorManager is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 106, The method getManager() from the type PropertyPageContributorManager is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 46, The type PropertyDialog is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 109, The type PropertyDialog is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 109, The type PropertyDialog is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 47, The type PropertyPageContributorManager is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 82, The type PropertyPageContributorManager is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 106, The type PropertyPageContributorManager is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 48, The type PropertyPageManager is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 100, The type PropertyPageManager is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
	line 100, The type PropertyPageManager is not accessible due to restriction on required library org.eclipse.ui.workbench_3.3.0.I20070608-1100.jar
Comment 1 Brian Fitzpatrick CLA 2007-10-25 12:42:31 EDT
Until the platform team does something to make some of these internal classes public, we really can't do much about the warnings in this particular plug-in. Sorry.
Comment 2 Brian Fitzpatrick CLA 2007-11-05 11:35:14 EST
No feedback from platform at this point. Pushing off to M4
Comment 3 Linda Chan CLA 2007-11-19 22:33:04 EST
Extracted all related discouraged access calls to a new utility class ProfileUIManager.  Any additional calls to the Platform UI internal API should be added here as well. The warnings are still there, but at least encapsulated within an utility class for ease of future updates.
Comment 4 Brian Fitzpatrick CLA 2008-01-08 11:56:42 EST
This appears to be as fixed as we can get it for Ganymede.
Comment 5 Linda Chan CLA 2008-02-21 23:42:33 EST
Related utility API methods have been added in org.eclipse.ui.dialogs.PreferencesUtil (see bug 208830) to support the usage in DTP.
Re-opening this bug to use those new platform-ui methods.  
Comment 6 Linda Chan CLA 2008-02-22 00:32:59 EST
DTP 1.6 needs to provide backward compatibility support of Eclipse platform 3.3, and the new API methods are added in 3.4 only.
So for 1.6, updated DTP internal utility class ProfileUIManager to use a copy of the platform-ui methods instead.  
Leaving this bug open for the next major release, when DTP no longer needs to be backward compatible with 3.3.  By then, we can directly use the API methods in o.e.ui.dialogs.PreferencesUtil, and completely remove the use of internal platform-ui package.
Comment 7 Linda Chan CLA 2008-12-23 21:57:29 EST
Created attachment 121195 [details]
Patch

Since backward compatibility to Platform version 3.3 is no longer supported in DTP 1.7, this patch switches from using internal Platform UI packages to Platform UI API methods added in 3.4 (Bugzilla 208830).
This also updates the minimum version of a couple of plugin dependencies.
Comment 8 Linda Chan CLA 2008-12-23 22:02:09 EST
Brian,
Can you please take a look at the patch, especially the updates to the minimum version of plugin dependencies?
It seems that some other plugin dependencies should have their version tolerance updated as well.
Comment 9 Brian Fitzpatrick CLA 2009-01-06 10:16:05 EST
Hi Linda... 

Your patch looks fine to me. I want to talk today about your question regarding minimum versions. 

But go ahead and deliver this to Galileo if you don't mind. Thanks for making sure this got resolved!

--Fitz
Comment 10 Linda Chan CLA 2009-01-06 15:42:06 EST
Created attachment 121688 [details]
Revised Patch

Revised patch to increase the o.e.d.c.ui plugin version from 1.1.3 to 1.1.4.
Comment 11 Linda Chan CLA 2009-01-06 15:44:20 EST
Delivered latest patch to HEAD branch, tagged with v200901070440.