| Summary: | Notational metamodel extension | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Modeling] Graphiti | Reporter: | Romain Raugi <romain.raugi> | ||||||
| Component: | Core | Assignee: | Michael Wenz <michael.wenz> | ||||||
| Status: | CLOSED FIXED | QA Contact: | |||||||
| Severity: | enhancement | ||||||||
| Priority: | P3 | CC: | michael.wenz, romain.raugi | ||||||
| Version: | 0.7.0 | Flags: | michael.wenz:
indigo+
|
||||||
| Target Milestone: | 0.8.0 | ||||||||
| Hardware: | All | ||||||||
| OS: | All | ||||||||
| See Also: | https://bugs.eclipse.org/bugs/show_bug.cgi?id=326733 | ||||||||
| Whiteboard: | Indigo M4 theme_round_offs | ||||||||
| Attachments: |
|
||||||||
|
Description
Romain Raugi
Created attachment 182932 [details]
A sample project to highlight the limitation.
The limitation in PictogramElementDelegate is removed by now (see https://bugs.eclipse.org/bugs/show_bug.cgi?id=326733). Removing the check in DiagramChangeListener could induce other issues since this check was introduced also for performance reasons, as far as I recall. Michael Hi Michael, Would it be possible to make customizable the list of packages the DiagramChangeListener use to filter (with these ones by default)? Thanks a lot, Regards, Romain Such a customization hook would be possible of course, but this would mean providing another method that can be used to configure something (e.g. in DiagramTypeProvider) and would increase complexity. I had another look onto this and found a probably easier solution: instead of only checking the package of the object itself it would be possible to (additionally if this check is false) check the packages of the super classes of the object to refresh. That makes at least your example work, does it completely solve the issue for you? (Please check the attached patch.) Thanks, Michael Created attachment 184943 [details]
Patch for an alternative solution
I have checked-in my fix (see patch) to have it available for Indigo M4. Just let me know in case there are issues. Michael Hi Michael, The fix works fine. Thanks a lot, Romain Updating state for M4 Part of Graphiti Indigo 0.8.0 |