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

Bug 304715

Summary: A problem with org.eclipse.emf.common.ui.celleditor.ExtendedComboBoxCellEditor
Product: [Modeling] EMF Reporter: Lidija Grahek <lgrahek>
Component: EditAssignee: Project Inbox <emf.emfindex-inbox>
Status: CLOSED DUPLICATE QA Contact:
Severity: normal    
Priority: P3 CC: ahunter.eclipse, Ed.Merks
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Patch none

Description Lidija Grahek CLA 2010-03-04 13:15:23 EST 
Created attachment 160989 [details]
Patch

This defect is created after discussion in Bug 193757. On focus lost, it wipes its selection list (the one that is created by filtering original list based on what user types in). Any further attempts to access the selected value fail, even though the selected value is still present and should be accessible.

I attached the patch that could solve this issue.
Comment 1 Ed Merks CLA 2010-03-04 14:09:10 EST 
Why would we cache the index in the original list rather than just the value that's already been determined?  This all feels a bit hacky.  I think the fix in https://bugs.eclipse.org/bugs/show_bug.cgi?id=290555 is better because I suspect it fixes this problem as well as the problem reported there. Could you confirm?  Note that even with that fix, I still had some flaky problems that bothered me and made me want to revisit the solution...
Comment 2 Lidija Grahek CLA 2010-03-04 15:01:35 EST 
Well, hacky sticks to hacky :) 
This just seemed the least risky to me. Anyways, it looks like this defect, as well as GMF's Bug 193757 are duplicates of Bug 290555. Whichever solution you adopt is fine with me.
Comment 3 Lidija Grahek CLA 2010-03-04 15:11:52 EST 

*** This bug has been marked as a duplicate of bug 290555 ***