-
-
Notifications
You must be signed in to change notification settings - Fork 368
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
Inventory Click event-value null check #6578
Conversation
Ah nuts I switched it to the wrong branch, give me a few minutes to fix the mess 😬 |
i was so confused when it suddenly switched to 23 commits |
d43a389
to
e11549a
Compare
Okay now I shall try to press the right button 🤣 |
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.
wee woo wee woo brace police
src/main/java/ch/njol/skript/classes/data/BukkitEventValues.java
Outdated
Show resolved
Hide resolved
id get arrested if i didnt Co-authored-by: sovdee <10354869+sovdeeth@users.noreply.github.com>
* null check * Update src/main/java/ch/njol/skript/classes/data/BukkitEventValues.java id get arrested if i didnt Co-authored-by: sovdee <10354869+sovdeeth@users.noreply.github.com> --------- Co-authored-by: Moderocky <admin@moderocky.com> Co-authored-by: sovdee <10354869+sovdeeth@users.noreply.github.com>
where can I download the latest skript.jar that contains this fix? |
checks tab -> java 17 -> artifacts -> download the zip |
sorry im not too familiar with github, could you send a link or further guidiance? apologies. |
Description
adds a null check to the event-slot of the inventory click event
I tested it, it just returns air when clicking outside the inventory, which should be fine (probably)
Target Minecraft Versions: any
Requirements: none
Related Issues: #6458