joergreichert / spray

Automatically exported from code.google.com/p/spray
1 stars 2 forks source link

References should only be allowed to containment features #78

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

example domain model:

model
  cont. states
     ref. action
  cont actions 

When you then use actions, 

class State: ...
reference actions attribute name

the add of the action will fail in the diagram, as it is not contained in a 
resource. So either the DSL should validate, that a containment reference is 
used or in the generated diagram editor code it should only be allowed to add 
elements that have been already created in their container.

model
  cont. states
     cont action

Would work with the existing Spray.

If you model it in the Spray DSL like that

class State: ...
reference actions attribute name

class Action ...

and than create the action first as node and then try to add an action as 
attribute reference to the state node, this also fails in the current Spray.

Original issue reported on code.google.com by de.abg.r...@gmail.com on 10 Nov 2011 at 5:56

GoogleCodeExporter commented 9 years ago
This looks very much like the same problem as 
http://code.google.com/a/eclipselabs.org/p/spray/issues/detail?id=60 .  Non 
containment references could be allowed if we have the ability to specify their 
container,  this would b the same solution as described in the other issue.

Original comment by joswar...@gmail.com on 11 Nov 2011 at 2:52

GoogleCodeExporter commented 9 years ago
I'll take that, since I'm working on issue#60. I am about to implement the 
scoping rules for this.

Original comment by karsten....@googlemail.com on 15 Nov 2011 at 12:15

GoogleCodeExporter commented 9 years ago
Right Karsten, if you resolve #60,  this one will be resolved as well.

Original comment by joswar...@gmail.com on 15 Nov 2011 at 12:37

GoogleCodeExporter commented 9 years ago
This issue was closed by revision a8a3457b2f59.

Original comment by karsten....@googlemail.com on 16 Nov 2011 at 7:35