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

Bug 360267

Summary: R4E disable filter generated an error
Product: z_Archived Reporter: Jacques Bouthillier <lmcbout>
Component: MylynAssignee: Sebastien Dubois <sebastien.dubois>
Status: RESOLVED FIXED QA Contact:
Severity: minor    
Priority: P3 CC: alvaro.sanchez-leon
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows Vista   
Whiteboard:

Description Jacques Bouthillier CLA 2011-10-07 14:12:17 EDT 
Occurs when I set a review disable, then I selected the preferences and wanted to see the disabled element. They showed in the view, but generated the following error:

java.lang.IllegalStateException: Need an underlying widget to be able to set the input.(Has the widget been disposed?)
	at org.eclipse.jface.viewers.StructuredViewer.setInput(StructuredViewer.java:1681)
	at org.eclipse.mylyn.reviews.r4e.ui.internal.navigator.ReviewNavigatorView.resetInput(ReviewNavigatorView.java:306)
	at org.eclipse.mylyn.reviews.r4e.ui.internal.navigator.ReviewNavigatorView.preferenceChange(ReviewNavigatorView.java:697)
	at org.eclipse.core.internal.preferences.EclipsePreferences$2.run(EclipsePreferences.java:754)
	at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:42)
	at org.eclipse.core.internal.preferences.EclipsePreferences.firePreferenceEvent(EclipsePreferences.java:757)
	at org.eclipse.core.internal.preferences.EclipsePreferences.putBoolean(EclipsePreferences.java:784)
	at org.eclipse.ui.preferences.ScopedPreferenceStore.setValue(ScopedPreferenceStore.java:810)
	at org.eclipse.mylyn.reviews.r4e.ui.internal.preferences.R4EPreferencePage.performOk(R4EPreferencePage.java:618)
	at org.eclipse.jface.preference.PreferencePage.performApply(PreferencePage.java:432)
	at org.eclipse.jface.preference.PreferencePage$2.widgetSelected(PreferencePage.java:281)
	at org.eclipse.swt.widgets.TypedListener.handleEvent(TypedListener.java:240)
	at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
Comment 1 Sebastien Dubois CLA 2011-10-21 14:58:32 EDT 
I don't see how this can happen, but I've put guards to prevent the situation from happening.  Implemented and tested