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

Bug 481836

Summary:	BorderSizeComputationExpression migration is done only on top level containers mapping
Product:	[Modeling] Sirius	Reporter:	Maxime Porhel <maxime.porhel>
Component:	Diagram	Assignee:	Maxime Porhel <maxime.porhel>
Status:	CLOSED FIXED	QA Contact:
Severity:	major
Priority:	P3	CC:	florian.barbin, maxime.porhel, pierre-charles.david, yann.mortier
Version:	3.1.0	Keywords:	triaged
Target Milestone:	4.0.0M4
Hardware:	PC
OS:	Windows 7
See Also:	https://git.eclipse.org/r/60191 https://git.eclipse.org/r/60236 https://git.eclipse.org/r/60189 https://git.eclipse.org/r/60305 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=881e92f6c06a0429d223da3bb32eaf01c3a37938 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=a17462443176eeabf555e85cfa86d0634f6a33c5
Whiteboard:
Bug Depends on:	465211, 481119
Bug Blocks:

Description Maxime Porhel

2015-11-10 08:28:13 EST

+++ This bug was initially created as a clone of Bug #481119 +++

BorderSizeComputionExpression migration is done only on top level containers mapping

Steps to reproduce
 * Open an odesign that have at least one container mapping contained into another container mapping made with Sirius 3.0.0 with a Sirius 3.1.0
 * Do a modification in order to save the migrated odesign.
 * Check the style of the container mapping contained into another container mapping
 * See that Border Size Computation is 0 but it should be 1 because 0 means no border since Sirius 3.1.0

Comment 1 Eclipse Genie

2015-11-12 05:46:44 EST

New Gerrit change created: https://git.eclipse.org/r/60191

Comment 2 Eclipse Genie

2015-11-12 11:12:36 EST

New Gerrit change created: https://git.eclipse.org/r/60236

Comment 3 Eclipse Genie

2015-11-13 05:51:39 EST

New Gerrit change created: https://git.eclipse.org/r/60305

Comment 4 Eclipse Genie

2015-11-13 10:06:45 EST

Gerrit change https://git.eclipse.org/r/60191 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=881e92f6c06a0429d223da3bb32eaf01c3a37938

Comment 5 Eclipse Genie

2015-11-13 10:08:10 EST

Gerrit change https://git.eclipse.org/r/60236 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=a17462443176eeabf555e85cfa86d0634f6a33c5

Comment 6 Maxime Porhel

2015-11-13 10:09:05 EST

Corrected on master for 4.0.0 M4

Comment 7 Florian Barbin

2016-05-20 08:43:06 EDT

Verified using the junit use case.

Comment 8 Pierre-Charles David

2016-06-24 08:02:19 EDT

Available in Sirius 4.0.0.