Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 364594 - R4Eclipse needs protection against wrong preferences selection
Summary: R4Eclipse needs protection against wrong preferences selection
Status: RESOLVED INVALID
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows Vista
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Sebastien Dubois CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-23 09:21 EST by Jacques Bouthillier CLA
Modified: 2012-02-23 10:41 EST (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jacques Bouthillier CLA 2011-11-23 09:21:46 EST 
When modifying the preferences, if by mistake a user add a "Rule Set" inside the group preference, it generates the following "NULL pointer" exception:

java.lang.NullPointerException
	at org.eclipse.mylyn.reviews.r4e.core.model.serial.impl.RModelFactoryExtImpl.loadUsrReviews(RModelFactoryExtImpl.java:177)
	at org.eclipse.mylyn.reviews.r4e.core.model.serial.impl.RModelFactoryExtImpl.openR4EReviewGroup(RModelFactoryExtImpl.java:146)
	at org.eclipse.mylyn.reviews.r4e.ui.internal.model.R4EUIModelController.loadReviewGroups(R4EUIModelController.java:319)
	at org.eclipse.mylyn.reviews.r4e.ui.internal.navigator.ReviewNavigatorView.preferenceChange(ReviewNavigatorView.java:675)
	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.put(EclipsePreferences.java:770)
	at org.eclipse.ui.preferences.ScopedPreferenceStore.setValue(ScopedPreferenceStore.java:789)
	at org.eclipse.jface.preference.ListEditor.doStore(ListEditor.java:250)
	at org.eclipse.jface.preference.FieldEditor.store(FieldEditor.java:680)
	at org.eclipse.jface.preference.FieldEditorPreferencePage.performOk(FieldEditorPreferencePage.java:343)
	at org.eclipse.mylyn.reviews.r4e.ui.internal.preferences.R4EPreferencePage.performOk(R4EPreferencePage.java:732)
	at org.eclipse.jface.preference.PreferenceDialog$13.run(PreferenceDialog.java:964)
	at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:42)
	at org.eclipse.ui.internal.JFaceUtil$1.run(JFaceUtil.java:49)
	at org.eclipse.jface.util.SafeRunnable.run(SafeRunnable.java:175)
	at org.eclipse.jface.preference.PreferenceDialog.okPressed(PreferenceDialog.java:944)
	at org.eclipse.ui.internal.dialogs.FilteredPreferenceDialog.okPressed(FilteredPreferenceDialog.java:447)
	at org.eclipse.ui.internal.dialogs.WorkbenchPreferenceDialog.okPressed(WorkbenchPreferenceDialog.java:171)
	at org.eclipse.jface.preference.PreferenceDialog.buttonPressed(PreferenceDialog.java:233)
Comment 1 Jacques Bouthillier CLA 2012-02-23 10:41:41 EST 
Not reproducible