Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 316429 - Comparing a model with resource with an in-memory model results in an empty right pane
Summary: Comparing a model with resource with an in-memory model results in an empty r...
Status: CLOSED FIXED
Alias: None
Product: EMFCompare
Classification: Modeling
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: EMF Compare CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-10 04:49 EDT by Jan Reimann CLA
Modified: 2012-05-22 09:46 EDT (History)
3 users (show)

See Also:


Attachments
solves the bug at two places (4.41 KB, patch)
2010-06-10 04:55 EDT, Jan Reimann CLA
laurent.goubet: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.