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.

Sign up for GitHub

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

Jump to bottom

Prevent NPE from querying toolbelt before capability initialization #2721

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Prevent NPE from querying toolbelt before capability initialization #2721

wants to merge 1 commit into from

Conversation

M-W-K
Copy link
Contributor

@M-W-K M-W-K commented Feb 12, 2025

What

Fixes NPEs from querying a toolbelt stack before its handler capability has been initialized.

Implementation Details

Added a static fallback instance in case a handler cannot be found on a toolbelt stack.

Outcome

Mods that query a toolbelt who's capabilities are being initialized for information won't throw an NPE

@M-W-K M-W-K added type: refactor Suggestion to refactor a section of code ignore changelog PR should not be added to the changelog. labels Feb 12, 2025
@M-W-K M-W-K requested a review from a team as a code owner February 12, 2025 02:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ignore changelog PR should not be added to the changelog. type: refactor Suggestion to refactor a section of code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@M-W-K