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

Fix EffActionBar NullPointException when message is null #5833

Merged

Conversation

Fusezion
Copy link
Contributor

Description

This PR aims to fix the NullPointException caused when message is null in EffActionBar
in addition I've made to %players% optional as players have been asking for this for awhile now and is a minor change


Target Minecraft Versions: any
Requirements: any
Related Issues: #5776

- Fixed NullPointerException
- Made to player optional
@AyhamAl-Ali AyhamAl-Ali added bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. enhancement Feature request, an issue about something that could be improved, or a PR improving something. labels Jul 14, 2023
Ah oops

Co-authored-by: Ayham Al Ali <[email protected]>
Fusezion and others added 3 commits July 14, 2023 08:07
- Some moldy cheese picked one me
@TheLimeGlass TheLimeGlass added the 2.7 Targeting a 2.7.X version release label Jul 17, 2023
@TheLimeGlass TheLimeGlass merged commit ebc0929 into SkriptLang:master Jul 17, 2023
TheLimeGlass pushed a commit that referenced this pull request Jul 17, 2023
@Fusezion Fusezion deleted the bugfix/nullpointer_effactionbar branch July 17, 2023 06:03
TheLimeGlass added a commit that referenced this pull request Jul 26, 2023
…ll (#5840)

Fix EffActionBar NullPointException when message is null (#5833)

(cherry picked from commit ebc0929)

Co-authored-by: Fusezion <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.7 Targeting a 2.7.X version release bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. enhancement Feature request, an issue about something that could be improved, or a PR improving something.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants