dukeboard/kevoree-modeling-framework

Wrong cardinality in KMF MM generated from EMF MM

Closed this issue · 7 comments

pascalpoizat commented

error:
I have a .ecore file with a 1/-1 (1..*) cardinality. It is transformed in a 1/1 cardinality in the .mm file generated from it.

environment:
.ecore file is created under Eclipse. Then copied in an IntelliJ workspace.
Right click on the .ecore file -> convert to meta model DSL.
IntelliJ IDEA 14.0.2 139.659, Dec 6th, 2014
Kevoree plugin V16

files: (should I add piece of code through Gist ?)

EMF

KMF

class pif.Workflow {
@contained
nodes : pif.WorkflowNode
initialNode : pif.InitialEvent
finalNodes : pif.EndEvent[0,]
@contained
sequenceFlows : pif.SequenceFlow[0,]
}

dukeboard commented

Hello,
thanks for the bug report, I check immediately if i can reproduce.

dukeboard commented

Hi again, after a quick check in our code its quite difficult to understand how the upper bound (int value) of a reference can be null into the Ecore AST (which is used for the conversion). Can you provide us the ecore used ? By advance thanks.

dukeboard commented

I've pushed a potential fix in the version 17 of the plugin.
Can you update and confirme me the correction in V17 please ?

Thanks by advance

pascalpoizat commented

Merci François

Le 9 janv. 2015 à 14:13, Fouquet François notifications@github.com a écrit :

Hello,
thanks for the bug report, I check immediately if i can reproduce.

—
Reply to this email directly or view it on GitHub.

Pascal Poizat
Université Paris Ouest Nanterre la Défense
LIP6 UMR 7606 CNRS
http://lip6.fr/Pascal.Poizat

pascalpoizat commented

Here you are

Le 9 janv. 2015 à 14:22, Fouquet François notifications@github.com a écrit :

Hi again, after a quick check in our code its quite difficult to understand how the upper bound (int value) of a reference can be null into the Ecore AST (which is used for the conversion). Can you provide us the ecore used ? By advance thanks.

—
Reply to this email directly or view it on GitHub.

Pascal Poizat
Université Paris Ouest Nanterre la Défense
LIP6 UMR 7606 CNRS
http://lip6.fr/Pascal.Poizat

pascalpoizat commented

I have updated the plugin (using IntelliJ update mechanism, didn’t know how to do it with github).
It seems to work (at least for the bug we found). I will check it more carefully in 7 days.

Le 9 janv. 2015 à 15:07, Fouquet François notifications@github.com a écrit :

I've pushed a potential fix in the version 17 of the plugin.
Can you update and confirme me the correction in V17 please ?

Thanks by advance

—
Reply to this email directly or view it on GitHub.

Pascal Poizat
Université Paris Ouest Nanterre la Défense
LIP6 UMR 7606 CNRS
http://lip6.fr/Pascal.Poizat

dukeboard commented

Ok perfect, the update process of IDEA is fine. I close this ticket.
I have modified the generation process for 1..* relationships, so it should address the problem.
If you face another issue, don't hesitate to open a ticket again.

Best Regards,
François