-
Notifications
You must be signed in to change notification settings - Fork 48
ST6RI-907: Cannot correctly render nested inherited connections with SHOWINHERITED style (PlantUML) #730
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
makeInheritKeyForReferer() to properly set up PCNamespace() (PCFlowEnd): Renamed from PCItemFromEnd (PCNamespace): Added. (addContextForFeature, addContextForFeatureChainExpression, addContextForFeatureReferenceExpression): Use makeRefPCForReferer(). (addContextForFlowEnd): Renamed from addContextForItemFlowEnd(), and use makeRefPCForReferer()
|
I also found that in the model below, the flow ends of So I traced the chain of the redefined features to find out the root redefined features to find out source and target. |
The flow ends of |
|
@seidewitz thank you for the explanation. I read KerML 8.4.4.10.2 to understand the rule but I could only found the phrase.
So I interpreted this implies BTW, I confirmed it also implicitly redefines |
That is not really what it says in what you quoted from the specification. What the spec says is that the FlowEnd has an owned Feature redefining In your example, the redefined flow The redefining flow is then parsed like: Because of the This was a particularly subtle aspect of redefinition that we had to carefully work out when we formalized the computation of inherited memberships! |

The model below could not be rendered correctly with SHOWINHERITED. This is caused by the inappropriate matching against inherited feature, in this case
a0. The current code wrongly assumes the first feature of feature chains directly inherited and fails to find a nested feature.