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

Bug 316429

Summary: Comparing a model with resource with an in-memory model results in an empty right pane
Product: [Modeling] EMFCompare Reporter: Jan Reimann <janreimone>
Component: CoreAssignee: EMF Compare <emf.compare-inbox>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: cedric.brun, janreimone, laurent.goubet
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
solves the bug at two places laurent.goubet: iplog+

Description Jan Reimann CLA 2010-06-10 04:49:09 EDT 
Build Identifier: 

I want to compare two evolution steps of the same model. The original model comes from a file, i.e. it has a Resource. The second model was created by some transformations within the first model. That means it is in-memory and not contained in a Resource. When comparing those both models the right pane stays empty because the right model isn't contained in a resource.

Reproducible: Always

Steps to Reproduce:
1. just compare one resource model and an in-memory model
Comment 1 Jan Reimann CLA 2010-06-10 04:51:39 EDT 
I forgot to mention that I use Helios RC3 with build-ID 20100603-0907. And I checked out the plugins org.eclipse.emf.compare and org.eclipse.emf.compare.ui into my workspace.
Comment 2 Jan Reimann CLA 2010-06-10 04:55:12 EDT 
Created attachment 171616 [details]
solves the bug at two places

The patch solves the bug by using an Resource independent implementation
Comment 3 Jan Reimann CLA 2010-06-10 04:58:29 EDT 
will this be fixed with the Helios release?
Comment 4 Jan Reimann CLA 2010-06-10 05:03:34 EDT 
sorry for the accidentally inserted layout modifications in ModelContentMergeDiffTab.java. The modification in line 905 is the important change.
Comment 5 Cedric Brun CLA 2010-06-10 05:15:21 EDT 
Hi Jan,

thanks for your feedback and for your patch. I'm afraid it can't be integrated for the Helios release as it's in a freezed state and the intellectual property process is closed (we can't accept any contribution for this release).

As soon as Helios is officialy out, we'll try to integrate the fix and provides an integration build, the fix would be part of the Helios Service Release 1  this fall though.
Comment 6 Laurent Goubet CLA 2011-08-09 05:51:30 EDT 
Sorry about the delay. The patch was no longer applicable with the many developments that took place in the meantime, but some parts of it were still pointing to potential bugs. We have fixed this on 1.2 and master.