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

Make NewAbstractAaxlHandler publicly visible #2705

Closed
AaronGreenhouse opened this issue Aug 27, 2021 · 3 comments · Fixed by #2706
Closed

Make NewAbstractAaxlHandler publicly visible #2705

AaronGreenhouse opened this issue Aug 27, 2021 · 3 comments · Fixed by #2706
Assignees
Milestone

Comments

@AaronGreenhouse
Copy link
Contributor

Class org.osate.analysis.resource.budgets.handlers.NewAbstractAaxlHandler is currently package visible, and thus only usable internally to the org.osate.analysis.resource.budgets plug-in. This class is intended to replace the existing org.osate.ui.handlers.AbstractAaxlHandler in org.osate.ui. It needs to be moved to org.osate.ui and made public so that new analyses can use it.

Class should be renamed to something other than NewAbstractAaxlHandler, but I cannot used AbstractAaxlHandler because we are not ready to remove the existing AbstractAaxlHandler.

@AaronGreenhouse AaronGreenhouse added this to the 2.10.0 milestone Aug 27, 2021
@AaronGreenhouse AaronGreenhouse self-assigned this Aug 27, 2021
@AaronGreenhouse
Copy link
Contributor Author

Renaming to AbstractAnalysisHandler.

@AaronGreenhouse
Copy link
Contributor Author

This is being done in the context of creating a mixed trust scheduling analysis. This class may be further updated as needed to support this 2nd subclass, and to better abstract the capabilities of this class.

@AaronGreenhouse
Copy link
Contributor Author

Also added addError() and addWarning() and addInfo() methods to ResultUtil

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