Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 191010 - Proxy resolve from label update which causes a resource to load fails to update the viewer
Summary: Proxy resolve from label update which causes a resource to load fails to upda...
Status: VERIFIED FIXED
Alias: None
Product: EMF
Classification: Modeling
Component: Edit (show other bugs)
Version: 2.3.0   Edit
Hardware: PC Windows XP
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Ed Merks CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-05 08:23 EDT by Ed Merks CLA
Modified: 2008-01-28 16:40 EST (History)
3 users (show)

See Also:


Attachments
This patch avoids the problem by changing only the item providers for Resource and ResourceSet (2.42 KB, patch)
2007-06-05 12:37 EDT, Ed Merks CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ed Merks CLA 2007-06-05 08:23:32 EDT
Because of the changes added by https://bugs.eclipse.org/bugs/show_bug.cgi?id=154571 the Ecore editor doesn't update properly when cross document references are resolved as part of computing a label.  A fix is to change NotifyChangedToViewerRefresh.handleNotifyChanged to asyncExec the update in this case, and since we can't tell if a viewer is busy, we can't know if we must do a post or not.  So it seems we need to change the behavior for all clients, even though this might have a potential to impact them adversely if they are relying on an immediate update taking place.
Comment 1 Kenn Hussey CLA 2007-06-05 09:21:17 EDT
I'm seeing the same issue in UML2 as well.
Comment 2 Ed Merks CLA 2007-06-05 12:37:30 EDT
Created attachment 70170 [details]
This patch avoids the problem by changing only the item providers for Resource and ResourceSet
Comment 3 Ed Merks CLA 2007-06-05 14:21:20 EDT
The changes are committed to CVS. 
Comment 4 Nick Boldt CLA 2007-06-07 03:18:53 EDT
Fixed in I200706061821.
Comment 5 Nick Boldt CLA 2008-01-28 16:40:59 EST
Move to verified as per bug 206558.