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

createExplosion with no world in location throws exception #6481

Closed
TheLimeGlass opened this issue Mar 5, 2024 · 0 comments
Closed

createExplosion with no world in location throws exception #6481

TheLimeGlass opened this issue Mar 5, 2024 · 0 comments
Labels
bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. completed The issue has been fully resolved and the change will be in the next Skript update. patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version. PR available Issues which have a yet-to-be merged PR resolving it

Comments

@TheLimeGlass
Copy link
Collaborator

TheLimeGlass commented Mar 5, 2024

Skript/Server Version

2.8.3
Spigot Paper 1.20.1

Bug Description

[14:41:47] [Server thread/ERROR]: #!#! Stack trace:
[14:41:47] [Server thread/ERROR]: #!#! java.lang.NullPointerException: Cannot invoke "org.bukkit.World.createExplosion(org.bukkit.Location, float, boolean)" because the return value of "org.bukkit.Location.getWorld()" is null
[14:41:47] [Server thread/ERROR]: #!#!     at Skript.jar//ch.njol.skript.effects.EffExplosion.execute(EffExplosion.java:85)
[14:41:47] [Server thread/ERROR]: #!#!     at Skript.jar//ch.njol.skript.lang.Effect.run(Effect.java:49)
[14:41:47] [Server thread/ERROR]: #!#!     at Skript.jar//ch.njol.skript.lang.TriggerItem.walk(TriggerItem.java:61)
[14:41:47] [Server thread/ERROR]: #!#!     at Skript.jar//ch.njol.skript.lang.TriggerItem.walk(TriggerItem.java:88)
[14:41:47] [Server thread/ERROR]: #!#!     at Skript.jar//ch.njol.skript.effects.Delay.lambda$walk$0(Delay.java:107)
[14:41:47] [Server thread/ERROR]: #!#!     at org.bukkit.craftbukkit.v1_20_R1.scheduler.CraftTask.run(CraftTask.java:101)
[14:41:47] [Server thread/ERROR]: #!#!     at org.bukkit.craftbukkit.v1_20_R1.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:480)
[14:41:47] [Server thread/ERROR]: #!#!     at net.minecraft.server.MinecraftServer.b(MinecraftServer.java:1479)
[14:41:47] [Server thread/ERROR]: #!#!     at net.minecraft.server.dedicated.DedicatedServer.b(DedicatedServer.java:446)
[14:41:47] [Server thread/ERROR]: #!#!     at net.minecraft.server.MinecraftServer.a(MinecraftServer.java:1393)
[14:41:47] [Server thread/ERROR]: #!#!     at net.minecraft.server.MinecraftServer.w(MinecraftServer.java:1170)
[14:41:47] [Server thread/ERROR]: #!#!     at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:317)
[14:41:47] [Server thread/ERROR]: #!#!     at java.base/java.lang.Thread.run(Thread.java:833)
[14:41:47] [Server thread/ERROR]: #!#! 
[14:41:47] [Server thread/ERROR]: #!#! Version Information:
[14:41:47] [Server thread/ERROR]: #!#!   Skript: (custom version)
[14:41:47] [Server thread/ERROR]: #!#!     Flavor: selfbuilt-unknown
[14:41:47] [Server thread/ERROR]: #!#!     Date: unknown
[14:41:47] [Server thread/ERROR]: #!#!   Bukkit: 1.20.1-R0.1-SNAPSHOT
[14:41:47] [Server thread/ERROR]: #!#!   Minecraft: 1.20.1
[14:41:47] [Server thread/ERROR]: #!#!   Java: 17.0.8.1 (OpenJDK 64-Bit Server VM 17.0.8.1+8-LTS)
[14:41:47] [Server thread/ERROR]: #!#!   OS: Linux amd64 5.10.0-24-amd64
[14:41:47] [Server thread/ERROR]: #!#! 
[14:41:47] [Server thread/ERROR]: #!#! Server platform: Paper
[14:41:47] [Server thread/ERROR]: #!#! 
[14:41:47] [Server thread/ERROR]: #!#! Current node: null
[14:41:47] [Server thread/ERROR]: #!#! Current item: create explosion effect {_vector} (as ch.njol.skript.util.Direction) {5::dam} offset by vector(0, 20, 0)
[14:41:47] [Server thread/ERROR]: #!#! Current trigger: function breakDam (simple event) (5/5.sk, line 21)

Expected Behavior

To not throw an exception

Steps to Reproduce

Run the EffExplosion effect with a location that has no world.

@AyhamAl-Ali AyhamAl-Ali added the bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. label Mar 6, 2024
@Moderocky Moderocky added the PR available Issues which have a yet-to-be merged PR resolving it label Apr 19, 2024
@Moderocky Moderocky added completed The issue has been fully resolved and the change will be in the next Skript update. patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version. labels Apr 26, 2024
@sovdeeth sovdeeth closed this as completed May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. completed The issue has been fully resolved and the change will be in the next Skript update. patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version. PR available Issues which have a yet-to-be merged PR resolving it
Projects
None yet
Development

No branches or pull requests

4 participants