Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 363286 - GenericDiffEngine#checkMoves(DiffGroup, Match2Elements) should not create move operation for left and right roots
Summary: GenericDiffEngine#checkMoves(DiffGroup, Match2Elements) should not create mov...
Status: CLOSED WORKSFORME
Alias: None
Product: EMFCompare
Classification: Modeling
Component: Core (show other bugs)
Version: 1.1   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: EMF Compare CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 365285
  Show dependency tree
 
Reported: 2011-11-09 06:11 EST by Ali AKAR CLA
Modified: 2012-05-30 03:33 EDT (History)
1 user (show)

See Also:

Attachments
A patch to resolve the issue (4.44 KB, text/plain)
2011-11-09 06:11 EST, Ali AKAR CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ali AKAR CLA 2011-11-09 06:11:41 EST 
Created attachment 206668 [details]
A patch to resolve the issue

When trying to merge automatically two objects which are not the root of the resources an exception occurs since EMF Compare try to move the left and right roots.
I created a patch to resolve this issue.
Comment 1 Laurent Goubet CLA 2011-11-21 04:50:02 EST 
Ali,

I am not the only dev on EMF Compare, and the emf.compare-inbox default assignee is there in order for all of us (along with anyone who wants to follow "all bugs on compare") to see what's going on. Please do not change the assignee of a bug.

Laurent
Comment 2 Laurent Goubet CLA 2011-12-12 09:19:21 EST 
Ali,

Could you provide us with a sample model which merging result in the issue you describe here? I did not manage to reproduce a problematic "move".
Comment 3 Laurent Goubet CLA 2012-03-06 06:01:38 EST 
Ali,

Any news on this? This bug cannot be reproduced by the EMF Compare team. Do you have sample problematic models you could provide us?
Comment 4 Laurent Goubet CLA 2012-05-30 03:33:26 EDT 
Cannot be reproduced and no input from reporter.