-
Notifications
You must be signed in to change notification settings - Fork 15
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
Fix model loader crash and animation layer activation #199
Conversation
contributes to #197 Signed-off-by: Didier Vojtisek <didier.vojtisek@inria.fr>
consider both ActivatedLayers and ActivatedTransientLayers Signed-off-by: Didier Vojtisek <didier.vojtisek@inria.fr>
openEditorSubMonitor.newChild(1)); | ||
ToolManagement toolManagement = DiagramPlugin.getDefault().getToolManagement(diagram); | ||
toolManagement.addToolFilter(new ToolFilter() { | ||
if(representation instanceof DSemanticDiagram) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line is in fact almost the only real change introduced in the PR, correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right for item 1 of the description
for item 2 see my second comment
|| layerName.equalsIgnoreCase("Animation"); | ||
boolean mustBeActive = mustBeActiveForAnimation || mustBeActiveForDebug; | ||
hasADebugLayer = hasADebugLayer || mustBeActiveForDebug; | ||
if (mustBeActive && !diagram.getActivatedLayers().contains(l) && !diagram.getActivatedTransientLayers().contains(l)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is the second line that has changed (most other changes are due to the new indentation)
(added && !diagram.getActivatedTransientLayers().contains(l)
)
Description
(Actually, ignore other representations as we don't know how to animate them)
Contribution to issues
Fixes #197