| Summary: | [StateMachine Diagram] Changing the property "kind" of a pseudoState, only the outline is changed | ||
|---|---|---|---|
| Product: | [Modeling] Papyrus | Reporter: | Fabien Gautreault <fabien.gautreault.ext> |
| Component: | Diagram | Assignee: | Camille Letavernier <cletavernier> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | charles, jeremie.tatibouet, papyrus-bugs |
| Version: | 1.1.0 | ||
| Target Milestone: | --- | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | SR3 | ||
|
Description
Fabien Gautreault
True. In fact, the property "kind" should not be changeable through the property view. It is prone to error as pseudostates vary a lot e.g. from a EntryPoint which sits on the side and is owned by the statemachine to an initial pseudostate which is owned by the containing region. Should be corrected in the property view generated for pseudostate. Camille, can you please constrain changing the "kind" attribute of the pseudo state. There are still some changes that would be useful, for instance toggling between deep and shallow history state, but for the moment I propose to disable changing the kind completely. The bug is still there. If I click on an ExitPoint that is on the edge of a composite state I can change the kind of this latter to "junction". This affects the diagram as well as the property view. Both views become inconsistent while the semantic model (the abstract tree) is still correct. If the model is closed and re-open, the diagram and the property view are still both invalid. The StateMachine Diagram doesn't support the change of Kind for pseudo-states. This can't be easily fixed The easiest (partial) fix would be to prevent the user from changing the Kind of a pseudo-state, but we cannot remove this possibility from e.g. the Advanced tab of the properties view. We should also ensure that it is possible to create all kinds of PseudoState from the ModelExplorer, since it wouldn't be easily possible to change their kind afterwards Pseudostate#kind is now read-only in the properties view Commit 9af3696 (Master) and 8fd8c23 (Mars) |