Skip to content
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

Missing error/warning markers in AADL navigator view #2609

Closed
lwrage opened this issue Mar 5, 2021 · 2 comments · Fixed by #2614
Closed

Missing error/warning markers in AADL navigator view #2609

lwrage opened this issue Mar 5, 2021 · 2 comments · Fixed by #2614
Assignees
Milestone

Comments

@lwrage
Copy link
Contributor

lwrage commented Mar 5, 2021

Summary
The AADL Navigator no longer shows error markers on elements in expanded AADL model files.

This used to work in previous versions.

Steps To Reproduce

  1. Paste model below into AADL editor and save it as P.aadl.
  2. Expand P.aadl file in AADL navigator
  3. The file contains error markers on flow implementations for fl
  4. No error markers on flows in the AADL navigator
package P
public

	thread T
		features
			i: in data port;
			o: out data port;
		flows
			fl: flow path i -> o;
	end T;

	thread implementation T.i
		calls
			seq: {
				cl: subprogram SP;
			};
		connections
			pc1: parameter i -> cl.pi;
			pc2: parameter cl.po -> o;
		flows
			-- cannot specify flow through called subprogram
			-- both of the following are not allowed
			fl: flow path i -> pc1 -> cl -> pc2 -> o;
			fl: flow path i -> pc1 -> cl.fp -> pc2 -> o;
	end T.i;

	subprogram SP
		features
			pi: in parameter;
			po: out parameter;
		flows
			fp: flow path pi -> po;
	end SP;
end P;

Screenshots
OSATE 2.9.0:
image

OSATE 2.9.1:
image

Desktop (please complete the following information):

  • OSATE Version: 2.9.1
  • Operating System: Linux, Windows
@lwrage lwrage added this to the 2.9.2 milestone Mar 5, 2021
@lwrage lwrage assigned lwrage and unassigned lwrage Mar 5, 2021
@lwrage
Copy link
Contributor Author

lwrage commented Mar 5, 2021

This is caused by the fix to #2430.

@AaronGreenhouse
Copy link
Contributor

Problem wasn't the imaging caching. I thought maybe I wasn't including the decorations correctly, but I was. The problem was that the original EObject represented by the wrapper was coming from a different ResourceSetImpl than the was being used when looking at the markers of any children. That meant the EcoreUtil.isAncestor() predicate was always returning false. I changed the factory to be able to share the resource set used by the caller in AadlElementContentProvider.getChildren().

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants