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

Bug 459359

Summary: Operation history gets polluted with a lot of ResizeRelocate operations.
Product: [Tools] GEF Reporter: Alexander Nyßen <nyssen>
Component: GEF ZestAssignee: Matthias Wienand <matthias.wienand>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: matthias.wienand
Version: unspecified   
Target Milestone: 3.10.0 (Mars) M7   
Hardware: All   
OS: All   
Whiteboard:

Description Alexander Nyßen CLA 2015-02-07 06:38:34 EST 
Even without any user interaction (i.e. when opening the Zest Graph example view), the operation history already contains several operations (that are not related to user interactions). This continues to happen while the user interacts with the viewer. We need to ensure that operations are properly related only direct user interactions.
Comment 1 Alexander Nyßen CLA 2015-02-08 13:02:15 EST 
The cause for this is that NodeLayoutPolicy performed the application of layout to the visuals via operations on the global operation history. 

Fixed it by ensuring that application of the layout to the visuals is now performed within NodeLayoutBehavior directly, executing the respective operation locally.

Renamed test case from NodeLayoutPolicyTests to NodeLayoutBehaviorTests. But these do not seem to be valid so far, so set them both to ignore. We will need to adjust the tests before resolving this.
Comment 2 Matthias Wienand CLA 2015-04-27 07:41:14 EDT 
As the tests have been updated in the meantime, I resolve this ticket as fixed for 3.10.0M7.