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

Bug 355030

Summary: WorkbenchStatusDialogManager.setSupportAreaProvider(AbstractStatusAreaProvider) doesn't work
Product: [Eclipse Project] Platform Reporter: Alexey Romanov <alexey.v.romanov>
Component: UIAssignee: Krzysztof Daniel <krzysztof.daniel>
Status: RESOLVED FIXED QA Contact: Krzysztof Daniel <krzysztof.daniel>
Severity: normal    
Priority: P3 CC: remy.suen
Version: 3.7   
Target Milestone: 4.2 M5   
Hardware: PC   
OS: Linux   
Whiteboard:
Attachments:
Description Flags
Patch proposition
none
mylyn/context/zip none

Description Alexey Romanov CLA 2011-08-18 03:09:45 EDT
Build Identifier: 

This method is defined as 

public void setSupportAreaProvider(AbstractStatusAreaProvider provider) {
	manager.setProperty(IStatusDialogConstants.CUSTOM_SUPPORT_PROVIDER,
		provider);
}

However, IStatusDialogConstants.CUSTOM_SUPPORT_PROVIDER isn't used anywhere outside this method (according to References > Workspace search). Presumably, it should be checked in SupportTray.getSupportProvider()

Reproducible: Always

Steps to Reproduce:
Create an Eclipse application with a custom status handler and call WorkbenchStatusDialogManager.setSupportAreaProvider(AbstractStatusAreaProvider) in your configureStatusDialog(WorkbenchStatusDialogManager) method.
Comment 1 Krzysztof Daniel CLA 2011-12-30 15:15:01 EST
Created attachment 208881 [details]
Patch proposition

This is a patch that solves this problem (introduced by fix for bug 298748).
Comment 2 Krzysztof Daniel CLA 2011-12-30 15:15:05 EST
Created attachment 208882 [details]
mylyn/context/zip
Comment 3 Krzysztof Daniel CLA 2011-12-30 15:18:31 EST
Alexey,
would you be able to verify that this patch works for you?

Note that you can workaround the problem by setting the support provider via org.eclipse.jface.util.Policy#setErrorSupportProvider.
Comment 4 Krzysztof Daniel CLA 2011-12-31 06:58:48 EST
I have released the fix and tests
commit d1162a612d74a6fe538ab781e184394cda644b2c
Comment 5 Alexey Romanov CLA 2011-12-31 07:12:48 EST
Yes, it works for me.