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

Bug 506125

Summary: Semantic reordering does not trigger graphical reordering
Product: [Modeling] Sirius Reporter: Pierre-Charles David <pierre-charles.david>
Component: TreeAssignee: Project Inbox <sirius.tree-inbox>
Status: CLOSED FIXED QA Contact: Laurent Redor <laurent.redor>
Severity: normal    
Priority: P3 CC: axel.richard, esteban.dugueperoux, laurent.redor, pierre-charles.david, steve.monnier
Version: 3.1.0Keywords: triaged
Target Milestone: 3.1.7   
Hardware: PC   
OS: Windows 7   
See Also: https://git.eclipse.org/r/84003
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=3003c5b7b7ea0628ce2a01f94a63f28f49ab0962
https://bugs.eclipse.org/bugs/show_bug.cgi?id=504494
Whiteboard:
Bug Depends on: 481433, 506126    
Bug Blocks:    
Attachments:
Description Flags
TestCase_SortedBy none

Description Pierre-Charles David CLA 2016-10-18 04:04:44 EDT 
+++ This bug was initially created as a clone of Bug #481433 +++

On a tree representation representing an Ecore model, I defined a drop tool to change the position of a selected EClass among its siblings, in a Java Service. The semantic change is processed correctly, however there is no change on the tree itself. The element are still displayed in the same order. Closing and reopening does not update the tree. Creating a new tree will display the right order.

As a workaround I had to process the same reordering among the DTreeItem.
Comment 1 Pierre-Charles David CLA 2016-10-18 04:05:03 EDT 
Clone to report the fix on Sirius 3.1.7.
Comment 2 Eclipse Genie CLA 2016-10-27 04:28:57 EDT 
New Gerrit change created: https://git.eclipse.org/r/84003
Comment 4 Laurent Redor CLA 2016-11-04 05:39:01 EDT 
Validated on Sirius 3.1.7. But the session is not dirty with the scenario of bug #481433 comment 3. This is linked to bug #504494.
Comment 5 Pierre-Charles David CLA 2016-11-04 10:03:17 EDT 
Available in Sirius 3.1.7. See https://wiki.eclipse.org/Sirius/3.1.7.
Comment 6 Axel RICHARD CLA 2016-11-23 11:37:01 EST 
Created attachment 265549 [details]
TestCase_SortedBy

The sortedBy are not taking into account by the tree representation
Comment 7 Pierre-Charles David CLA 2016-11-24 04:38:23 EST 
(In reply to Axel RICHARD from comment #6)
> Created attachment 265549 [details]
> TestCase_SortedBy
> 
> The sortedBy are not taking into account by the tree representation

Seen with Axel: this was on 3.1.6 (so without the fix). On 3.1.7 with the fix, his use case works correctly.