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 attempted bed spawn when no bed is registered #99

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

Conversation

ItsOnlyBinary
Copy link

papermc seems to no longer fire a NullPointerException when attempting to spawn a player who does not have a bed, it instead fires IllegalArgumentException

so i have switched it to checking for null before attempting a bed spawn

@Narimm
Copy link
Member

Narimm commented Oct 26, 2019

Mate could you post a stack trace of why this is required and under what versions you have had it occur

@ItsOnlyBinary
Copy link
Author

[17:54:37] [Server thread/INFO]: Player7777 issued server command: /spawn
[17:54:40] [Server thread/WARN]: [geSuitSpawn] Task #710724 for geSuitSpawn v1.0.0-SNAPSHOT- generated an exception
java.lang.IllegalArgumentException: location
	at com.google.common.base.Preconditions.checkArgument(Preconditions.java:122) ~[patched_1.14.4.jar:git-Paper-207]
	at org.bukkit.craftbukkit.v1_14_R1.entity.CraftPlayer.teleport(CraftPlayer.java:741) ~[patched_1.14.4.jar:git-Paper-207]
	at org.bukkit.craftbukkit.v1_14_R1.entity.CraftEntity.teleport(CraftEntity.java:489) ~[patched_1.14.4.jar:git-Paper-207]
	at net.cubespace.geSuiteSpawn.managers.SpawnManager.sendPlayerToSpawn(SpawnManager.java:198) ~[?:?]
	at net.cubespace.geSuiteSpawn.commands.SpawnCommand.lambda$onCommand$0(SpawnCommand.java:70) ~[?:?]
	at org.bukkit.craftbukkit.v1_14_R1.scheduler.CraftTask.run(CraftTask.java:84) ~[patched_1.14.4.jar:git-Paper-207]
	at org.bukkit.craftbukkit.v1_14_R1.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:452) ~[patched_1.14.4.jar:git-Paper-207]
	at net.minecraft.server.v1_14_R1.MinecraftServer.b(MinecraftServer.java:1171) ~[patched_1.14.4.jar:git-Paper-207]
	at net.minecraft.server.v1_14_R1.DedicatedServer.b(DedicatedServer.java:417) ~[patched_1.14.4.jar:git-Paper-207]
	at net.minecraft.server.v1_14_R1.MinecraftServer.a(MinecraftServer.java:1098) ~[patched_1.14.4.jar:git-Paper-207]
	at net.minecraft.server.v1_14_R1.MinecraftServer.run(MinecraftServer.java:925) ~[patched_1.14.4.jar:git-Paper-207]
	at java.lang.Thread.run(Thread.java:748) [?:1.8.0_222]

This server is running Paper version git-Paper-207 (MC: 1.14.4) (Implementing API version 1.14.4-R0.1-SNAPSHOT)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants