The Bug:
The freezing effect isn't immediately removed upon switching into spectator mode.
Steps to Reproduce:
Switch into creative mode and stand inside of some powder snow.
Wait until the freezing overlay completely covers your screen and then switch to spectator mode.
Take note as to whether or not the freezing effect is immediately removed upon switching into spectator mode.
Observed Behavior:
The freezing effect isn't immediately removed upon switching into spectator mode.
Expected Behavior:
The freezing effect would be immediately removed upon switching into spectator mode.
Code Analysis:
Code analysis by @unknown can be found below.
The following is based on a decompiled version of Minecraft 1.19.2 using MCP-Reborn.
net.minecraft.server.level.ServerPlayer.java
public class ServerPlayer extends Player {
...
public boolean setGameMode(GameType gameType) {
if (!this.gameMode.changeGameModeForPlayer(gameType)) {
return false;
} else {
this.connection.send(new ClientboundGameEventPacket(ClientboundGameEventPacket.CHANGE_GAME_MODE, (float)gameType.getId()));
if (gameType == GameType.SPECTATOR) {
this.removeEntitiesOnShoulder();
this.stopRiding();
} else {
this.setCamera(this);
}
this.onUpdateAbilities();
this.updateEffectVisibility();
return true;
}
}
...
If we look at the above class, we can see that only two methods are called when a player switches into spectator mode. These methods are removeEntitiesOnShoulder()
and stopRiding()
. This basically means that when a player changes into spectator mode, any entities that are riding on their shoulders will be dismounted, along with the player themselves being dismounted if they are riding any entity. As a result of only these two methods being called, the freezing effect isn't immediately removed upon switching into spectator mode.
Fix:
Simply calling the setTicksFrozen()
method where appropriate within this piece of code will immediately remove the freezing effect upon switching into spectator mode, thus resolving this problem. The following line code can be used to fix this issue.
this.setTicksFrozen(0);
Linked issues
Attachments
Comments 2
Following on from my code analysis, I've double-checked my proposed fix and I can confidently confirm that it's fully functioning and works as expected, so I've attached two screenshots to this report, one of which shows the current code and the other that shows the fixed code. I feel this information may be quite insightful hence my reasoning for providing it. 🙂
[media][media]
Can confirm