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

Bug 533870

Summary: ZestContentViewer should store a back reference to the original model elements
Product: [Tools] GEF Reporter: Matthias Wienand <matthias.wienand>
Component: GEF ZestAssignee: gef-inbox <gef-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: miklossy
Version: 5.0.1   
Target Milestone: 5.1.0 (2019-06)   
Hardware: All   
OS: All   
Whiteboard: low-hanging fruit

Description Matthias Wienand CLA 2018-04-20 07:57:39 EDT
Currently, ZestContentViewer only maintains a content-node-map, i.e. a map from content objects to GEF Graph Node objects. However, given a GEF Graph Node, you have to traverse this map in order to find the corresponding content object.

Instead, ZestContentViewer could store a back reference to the original content object in the created GEF Graph Node.
Comment 1 Tamas Miklossy CLA 2019-04-25 03:53:41 EDT
See also forum thread https://www.eclipse.org/forums/index.php/t/1092257/
Comment 2 Matthias Wienand CLA 2019-06-28 08:09:49 EDT
The back-references are now stored via the attributes of Graph, Node, and Edge objects. They are accessible via ZestProperties as of GEF 5.1.0. Therefore, I resolve this ticket as fixed.