-
-
Notifications
You must be signed in to change notification settings - Fork 815
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
Replace object type booleans with methods. This removes the need for … #2536
Replace object type booleans with methods. This removes the need for … #2536
Conversation
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Thanks for taking up work on this plugin. It actually deserves a thorough redesign! |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
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.
Otherwise it seems OK. Please rebase your PRs on master so that we can merge more easily.
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.
Thanks!
4746d7e
to
5ae4149
Compare
Rebased. I still need to test the rebased changes. |
…managing object type state and delegates it to the StelObjectMgr.
c40bb27
to
af2925f
Compare
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.
Yes, now we are there. Thanks!
Hello @colossatr0n! Please check the fresh version (development snapshot) of Stellarium: |
Hello @colossatr0n! Please check the fresh version (development snapshot) of Stellarium: |
Hello @colossatr0n! Please check the latest stable version of Stellarium: |
…managing object type state and delegates it to the StelObjectMgr.
Description
The Observability plugin manages object type state within the draw method. This is an error prone manual process that must be tracked by the developer when refactoring. This PR removes the need to manage the state and delegates it back to the StelObjectMgr by using methods that consume the object selection and return whether or not the object is the specified type.
Fixes # (issue)
Screenshots (if appropriate):
Type of change
How Has This Been Tested?
The changes in this PR are essentially a one-to-one conversion from the previous logic, with the exception of the introduction of
isSystemPlanet
.These changes were tested by running a pre-change and post-change stellarium instance, selecting the moon, sun, a planet, and an interstellar star, and comparing the output from each instance.
Test Configuration:
Checklist: