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

Bug 361357

Summary: [RBD] Dojo RadioGroup: Remove and Move dwon buttons should be disabled by default in options popup window
Product: z_Archived Reporter: Thomas Wu <wxwu>
Component: EDTAssignee: pf yu <pfyu>
Status: NEW --- QA Contact:
Severity: minor    
Priority: P3 CC: chenzhh, hjiyong, huozz
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
screenshot none

Description Thomas Wu CLA 2011-10-19 05:06:40 EDT
Build Identifier: 20111018

There is the same issue in RBD.
In options window of Dojo RadioGroup, the Remove and Move down buttons are enabled by default. But no item has been selected yet, so they should be disabled. If user clicks any of them, an exception will be raised in error log view. 

Reproducible: Always

Steps to Reproduce:
0.Open error log view in EDT.
1.DD a Dojo RadioGroup into a handler.
2.Open its properties view, then click "..." button beside the options property. The options window popups.
3.Click Remove button or Move down button directly.

Actual results: The following exception is raised in error log. These two buttons should be disabled by default because user has not select anyone yet.

java.lang.IllegalArgumentException: Index out of bounds
	at org.eclipse.swt.SWT.error(SWT.java:4263)
	at org.eclipse.swt.SWT.error(SWT.java:4197)
	at org.eclipse.swt.SWT.error(SWT.java:4168)
	at org.eclipse.swt.widgets.Widget.error(Widget.java:468)
	at org.eclipse.swt.widgets.List.remove(List.java:811)
	at org.eclipse.edt.ide.rui.visualeditor.internal.util.StringArrayDialog.widgetSelected(StringArrayDialog.java:188)
	at org.eclipse.swt.widgets.TypedListener.handleEvent(TypedListener.java:240)
	at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
	at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1053)
	at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:4165)
	at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3754)
	at org.eclipse.jface.window.Window.runEventLoop(Window.java:825)
	at org.eclipse.jface.window.Window.open(Window.java:801)
	at org.eclipse.edt.ide.rui.visualeditor.internal.properties.PropertyEditorStringArray.widgetSelected(PropertyEditorStringArray.java:141)
	at org.eclipse.swt.widgets.TypedListener.handleEvent(TypedListener.java:240)
	at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
	at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1053)
	at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:4165)
	at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3754)
	at org.eclipse.ui.internal.Workbench.runEventLoop(Workbench.java:2696)
	at org.eclipse.ui.internal.Workbench.runUI(Workbench.java:2660)
	at org.eclipse.ui.internal.Workbench.access$4(Workbench.java:2494)
	at org.eclipse.ui.internal.Workbench$7.run(Workbench.java:674)
	at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:332)
	at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:667)
	at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:149)
	at org.eclipse.ui.internal.ide.application.IDEApplication.start(IDEApplication.java:123)
	at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:196)
	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:110)
	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:79)
	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:344)
	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:179)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.lang.reflect.Method.invoke(Unknown Source)
	at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:622)
	at org.eclipse.equinox.launcher.Main.basicRun(Main.java:577)
	at org.eclipse.equinox.launcher.Main.run(Main.java:1410)
Comment 1 Thomas Wu CLA 2011-10-19 05:08:08 EDT
Created attachment 205485 [details]
screenshot
Comment 2 Huang Ji Yong CLA 2011-10-21 01:45:59 EDT
This is a VE property view problem, route to Forest
Comment 3 Lisa Lasher CLA 2011-11-08 13:05:56 EST
This is a low priority defect that also occurs in RBD, so I am deferring the fix to 1.0.