Bug 447849 - ContainerMappingImport importing a ListContainerMapping does not reuse its children mappings
Summary: ContainerMappingImport importing a ListContainerMapping does not reuse its ch...
Status: NEW
Alias: None
Product: Sirius
Classification: Modeling
Component: Diagram (show other bugs)
Version: 1.0.1   Edit
Hardware: PC Mac OS X
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2014-10-20 03:49 EDT by Maxime Porhel CLA
Modified: 2014-11-25 05:01 EST (History)
3 users (show)

See Also:

Attachments
Reproduction case (3.33 KB, application/zip)
2014-11-25 04:03 EST, Maxime Porhel 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 Maxime Porhel CLA 2014-10-20 03:49:19 EDT 
Steps to reproduce: 
 . Get the attached project (VSM+ecore+aird)
 . Open the session
 . Open the existing diagram

Its description declare an import mapping to import a list ContainerMapping from another diagram in the VSM: the hideSubMappings attribute is not enabled, we should see some list elements and a border node.
Comment 1 Esteban DUGUEPEROUX CLA 2014-11-13 12:00:36 EST 
It miss a project in attachement.
Comment 2 Maxime Porhel CLA 2014-11-25 04:03:58 EST 
Created attachment 248891 [details]
Reproduction case

The diagram showing errors is 447849 - Import mapping with issue
Comment 3 Florian Barbin CLA 2014-11-25 05:01:03 EST 
Reproduced on Sirius master. The bug seems there for a while.

the call of org.eclipse.sirius.diagram.business.internal.componentization.mappings.DiagramMappingsManagerImpl.getNodeMappings(DNodeList, boolean) within org.eclipse.sirius.diagram.business.internal.componentization.mappings.DiagramMappingsManagerImpl.getNodeMappings(DNodeList) is done with takeAlsoChildrenOfAllImportedMappings=false. But there is also an other issue since the descriptionMappings list is empty (l.190 of DiagramMappingsManagerImpl)