Skip to content

Add precondition to spawn location setter in spawn events#13830

Merged
electronicboy merged 1 commit intoPaperMC:mainfrom
masmc05:spawn-preconditions
Apr 27, 2026
Merged

Add precondition to spawn location setter in spawn events#13830
electronicboy merged 1 commit intoPaperMC:mainfrom
masmc05:spawn-preconditions

Conversation

@masmc05
Copy link
Copy Markdown
Contributor

@masmc05 masmc05 commented Apr 27, 2026

No description provided.

@masmc05 masmc05 requested a review from a team as a code owner April 27, 2026 15:42
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Apr 27, 2026
@mbax
Copy link
Copy Markdown
Contributor

mbax commented Apr 27, 2026

Example of issue to fix with this.

java.lang.NullPointerException: Cannot invoke "org.bukkit.craftbukkit.CraftWorld.getHandle()" because the return value of "org.bukkit.Location.getWorld()" is null
	at net.minecraft.server.network.config.PrepareSpawnTask$Preparing.tick(PrepareSpawnTask.java:233) ~[paper-26.1.2.jar:26.1.2-49-7799bf2]
	at net.minecraft.server.network.config.PrepareSpawnTask.tick(PrepareSpawnTask.java:123) ~[paper-26.1.2.jar:26.1.2-49-7799bf2]
	at net.minecraft.server.network.ServerConfigurationPacketListenerImpl.tick(ServerConfigurationPacketListenerImpl.java:218) ~[paper-26.1.2.jar:26.1.2-49-7799bf2]
	at net.minecraft.network.Connection.tick(Connection.java:573) ~[paper-26.1.2.jar:26.1.2-49-7799bf2]
	at net.minecraft.server.network.ServerConnectionListener.tick(ServerConnectionListener.java:246) ~[paper-26.1.2.jar:26.1.2-49-7799bf2]
	at net.minecraft.server.MinecraftServer.tickConnection(MinecraftServer.java:1909) ~[paper-26.1.2.jar:26.1.2-49-7799bf2]
	at net.minecraft.server.dedicated.DedicatedServer.tickConnection(DedicatedServer.java:582) ~[paper-26.1.2.jar:26.1.2-49-7799bf2]
	at net.minecraft.server.MinecraftServer.tickChildren(MinecraftServer.java:1872) ~[paper-26.1.2.jar:26.1.2-49-7799bf2]
	at net.minecraft.server.MinecraftServer.tickServer(MinecraftServer.java:1655) ~[paper-26.1.2.jar:26.1.2-49-7799bf2]
	at net.minecraft.server.dedicated.DedicatedServer.tickServer(DedicatedServer.java:449) ~[paper-26.1.2.jar:26.1.2-49-7799bf2]
	at net.minecraft.server.MinecraftServer.processPacketsAndTick(MinecraftServer.java:1713) ~[paper-26.1.2.jar:26.1.2-49-7799bf2]
	at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1383) ~[paper-26.1.2.jar:26.1.2-49-7799bf2]
	at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:304) ~[paper-26.1.2.jar:26.1.2-49-7799bf2]
	at java.base/java.lang.Thread.run(Thread.java:1474) ~[?:?]

@electronicboy electronicboy merged commit 17524d0 into PaperMC:main Apr 27, 2026
5 checks passed
@github-project-automation github-project-automation Bot moved this from Awaiting review to Merged in Paper PR Queue Apr 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

3 participants