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

Bug 246632

Summary: [CommonNavigator] Why is getLinkHelperService private?
Product: [Eclipse Project] Platform Reporter: Rob Stryker <stryker>
Component: UIAssignee: Francis Upton IV <francisu>
Status: RESOLVED DUPLICATE QA Contact:
Severity: enhancement    
Priority: P3 CC: francisu
Version: 3.4   
Target Milestone: ---   
Hardware: PC   
OS: Linux   
Whiteboard:

Description Rob Stryker CLA 2008-09-08 14:21:45 EDT
It is impossible (or difficult) to subclass some parts of the CommonNavigator because getLinkHelperService() is private.

For example:
	protected ActionGroup createCommonActionGroup() {
		return new CommonNavigatorActionGroup(this, commonViewer, getLinkHelperService());
	}


createCommonActionGroup is protected and overrideable, but CommonNavigatorActionGroup, and probably any replacement for it, would require access to the link helper service.  A subclass has no way to gain access to this variable at all.

As the link helper service itself is pretty safe and contains mostly getters, I think making CommonNavigator.getLinkHelperService() a protected method rather than a private one.
Comment 1 Francis Upton IV CLA 2009-01-15 04:21:11 EST

*** This bug has been marked as a duplicate of bug 250198 ***