Remove unnecessary repeating logic in SpectatorModeClient.java

Roadhog360 · Aug 26, 2023 · 862ead5 · 862ead5
1 parent cca3146
commit 862ead5
Showing 1 changed file with 5 additions and 8 deletions.
diff --git a/src/main/java/ganymedes01/etfuturum/spectator/SpectatorModeClient.java b/src/main/java/ganymedes01/etfuturum/spectator/SpectatorModeClient.java
@@ -7,6 +7,7 @@
 import cpw.mods.fml.relauncher.Side;
 import cpw.mods.fml.relauncher.SideOnly;
 import net.minecraft.client.Minecraft;
+import net.minecraft.client.entity.EntityClientPlayerMP;
 import net.minecraft.client.model.ModelBiped;
 import net.minecraft.client.renderer.entity.RenderManager;
 import net.minecraft.entity.player.EntityPlayer;
@@ -126,14 +127,10 @@ public void onBlockHighlight(DrawBlockHighlightEvent event) {
 	@SubscribeEvent
 	@SideOnly(Side.CLIENT)
 	public void onClientTick(TickEvent.ClientTickEvent event) {
-		EntityPlayer player = FMLClientHandler.instance().getClientPlayerEntity();
-		if (player != null && Minecraft.getMinecraft().playerController != null && event.phase == TickEvent.Phase.START) {
-			if (Minecraft.getMinecraft().playerController.currentGameType == SpectatorMode.SPECTATOR_GAMETYPE) {
-				if (!player.capabilities.isFlying) {
-					player.capabilities.isFlying = true;
-					player.sendPlayerAbilities();
-				}
-			}
+		EntityClientPlayerMP player = FMLClientHandler.instance().getClientPlayerEntity();
+		if (isSpectator(player) && !player.capabilities.isFlying) {
+			player.capabilities.isFlying = true;
+			player.sendPlayerAbilities();
 		}
 	}
 }