Attachment #148138 for bug #290555

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




  {
    CCombo combo = (CCombo)getControl();
    if (combo != null && (!combo.isDisposed()))
    {
      Object previousValue = getValue();
      String[] items = createItems(list = new ArrayList<Object>(originalList), labelProvider, filter, sorted);
      combo.setItems(items);
      // If the previous value is valid, keep it selected
      if (list.contains(previousValue))
      {
        setValue(previousValue);
      }
      else if (items.length > 0)
      {
        combo.select(0);
      }
    }

Return to bug 290555

Lines 211-221 Link Here

(-)org.eclipse.emf/plugins/org.eclipse.emf.common.ui/src/org/eclipse/emf/common/ui/celleditor/ExtendedComboBoxCellEditor.java (-1 / +7 lines)
211	{	211	{
212	CCombo combo = (CCombo)getControl();	212	CCombo combo = (CCombo)getControl();
213	if (combo != null && (!combo.isDisposed()))	213	if (combo != null && (!combo.isDisposed()))
214	{	214	{
		215	Object previousValue = getValue();
215	String[] items = createItems(list = new ArrayList<Object>(originalList), labelProvider, filter, sorted);	216	String[] items = createItems(list = new ArrayList<Object>(originalList), labelProvider, filter, sorted);
216	combo.setItems(items);	217	combo.setItems(items);
217	if (items.length > 0)	218	// If the previous value is valid, keep it selected
		219	if (list.contains(previousValue))
		220	{
		221	setValue(previousValue);
		222	}
		223	else if (items.length > 0)
218	{	224	{
219	combo.select(0);	225	combo.select(0);
220	}	226	}
221	}	227	}