Remove direction correction, as it does not seem to be effective.

NoCheatPlus · May 15, 2014 · c93edcb · c93edcb
1 parent 24bc281
commit c93edcb
Show file tree

Hide file tree

Showing 17 changed files with 134 additions and 149 deletions.
diff --git a/NCPCompatBukkit/src/main/java/fr/neatmonster/nocheatplus/compat/bukkit/MCAccessBukkit.java b/NCPCompatBukkit/src/main/java/fr/neatmonster/nocheatplus/compat/bukkit/MCAccessBukkit.java
@@ -233,9 +233,9 @@ public boolean hasGravity(final Material mat) {
 		}
 	}
 
-	@Override
-	public void correctDirection(Player player) {
-		// TODO: Consider using reflection (detect CraftPlayer, access EntityPlayer + check if possible (!), use flags for if valid or invalid.)
-	}
+//	@Override
+//	public void correctDirection(Player player) {
+//		// TODO: Consider using reflection (detect CraftPlayer, access EntityPlayer + check if possible (!), use flags for if valid or invalid.)
+//	}
 
 }
diff --git a/NCPCompatCB2512/src/main/java/fr/neatmonster/nocheatplus/compat/cb2512/MCAccessCB2512.java b/NCPCompatCB2512/src/main/java/fr/neatmonster/nocheatplus/compat/cb2512/MCAccessCB2512.java
@@ -18,7 +18,6 @@
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 
-import fr.neatmonster.nocheatplus.checks.moving.LocUtil;
 import fr.neatmonster.nocheatplus.compat.AlmostBoolean;
 import fr.neatmonster.nocheatplus.compat.MCAccess;
 import fr.neatmonster.nocheatplus.utilities.BlockCache;
@@ -163,13 +162,13 @@ public boolean hasGravity(final Material mat) {
 		}
 	}
 
-	@Override
-	public void correctDirection(final Player player) {
-		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
-		// Main direction.
-		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
-		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
-		// Consider setting the lastYaw here too.
-	}
+//	@Override
+//	public void correctDirection(final Player player) {
+//		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
+//		// Main direction.
+//		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
+//		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
+//		// Consider setting the lastYaw here too.
+//	}
 
 }
diff --git a/NCPCompatCB2545/src/main/java/fr/neatmonster/nocheatplus/compat/cb2545/MCAccessCB2545.java b/NCPCompatCB2545/src/main/java/fr/neatmonster/nocheatplus/compat/cb2545/MCAccessCB2545.java
@@ -18,7 +18,6 @@
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 
-import fr.neatmonster.nocheatplus.checks.moving.LocUtil;
 import fr.neatmonster.nocheatplus.compat.AlmostBoolean;
 import fr.neatmonster.nocheatplus.compat.MCAccess;
 import fr.neatmonster.nocheatplus.utilities.BlockCache;
@@ -163,13 +162,13 @@ public boolean hasGravity(final Material mat) {
 		}
 	}
 
-	@Override
-	public void correctDirection(final Player player) {
-		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
-		// Main direction.
-		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
-		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
-		// Consider setting the lastYaw here too.
-	}
+//	@Override
+//	public void correctDirection(final Player player) {
+//		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
+//		// Main direction.
+//		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
+//		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
+//		// Consider setting the lastYaw here too.
+//	}
 
 }
diff --git a/NCPCompatCB2602/src/main/java/fr/neatmonster/nocheatplus/compat/cb2602/MCAccessCB2602.java b/NCPCompatCB2602/src/main/java/fr/neatmonster/nocheatplus/compat/cb2602/MCAccessCB2602.java
@@ -18,7 +18,6 @@
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 
-import fr.neatmonster.nocheatplus.checks.moving.LocUtil;
 import fr.neatmonster.nocheatplus.compat.AlmostBoolean;
 import fr.neatmonster.nocheatplus.compat.MCAccess;
 import fr.neatmonster.nocheatplus.utilities.BlockCache;
@@ -165,13 +164,13 @@ public boolean hasGravity(final Material mat) {
 		}
 	}
 
-	@Override
-	public void correctDirection(final Player player) {
-		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
-		// Main direction.
-		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
-		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
-		// Consider setting the lastYaw here too.
-	}
+//	@Override
+//	public void correctDirection(final Player player) {
+//		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
+//		// Main direction.
+//		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
+//		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
+//		// Consider setting the lastYaw here too.
+//	}
 
 }
diff --git a/NCPCompatCB2645/src/main/java/fr/neatmonster/nocheatplus/compat/cb2645/MCAccessCB2645.java b/NCPCompatCB2645/src/main/java/fr/neatmonster/nocheatplus/compat/cb2645/MCAccessCB2645.java
@@ -18,7 +18,6 @@
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 
-import fr.neatmonster.nocheatplus.checks.moving.LocUtil;
 import fr.neatmonster.nocheatplus.compat.AlmostBoolean;
 import fr.neatmonster.nocheatplus.compat.MCAccess;
 import fr.neatmonster.nocheatplus.utilities.BlockCache;
@@ -165,13 +164,13 @@ public boolean hasGravity(final Material mat) {
 		}
 	}
 
-	@Override
-	public void correctDirection(final Player player) {
-		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
-		// Main direction.
-		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
-		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
-		// Consider setting the lastYaw here too.
-	}
+//	@Override
+//	public void correctDirection(final Player player) {
+//		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
+//		// Main direction.
+//		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
+//		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
+//		// Consider setting the lastYaw here too.
+//	}
 
 }
diff --git a/NCPCompatCB2691/src/main/java/fr/neatmonster/nocheatplus/compat/cb2691/MCAccessCB2691.java b/NCPCompatCB2691/src/main/java/fr/neatmonster/nocheatplus/compat/cb2691/MCAccessCB2691.java
@@ -18,7 +18,6 @@
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 
-import fr.neatmonster.nocheatplus.checks.moving.LocUtil;
 import fr.neatmonster.nocheatplus.compat.AlmostBoolean;
 import fr.neatmonster.nocheatplus.compat.MCAccess;
 import fr.neatmonster.nocheatplus.utilities.BlockCache;
@@ -167,13 +166,13 @@ public boolean hasGravity(final Material mat) {
 		}
 	}
 
-	@Override
-	public void correctDirection(final Player player) {
-		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
-		// Main direction.
-		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
-		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
-		// Consider setting the lastYaw here too.
-	}
+//	@Override
+//	public void correctDirection(final Player player) {
+//		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
+//		// Main direction.
+//		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
+//		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
+//		// Consider setting the lastYaw here too.
+//	}
 
 }
diff --git a/NCPCompatCB2763/src/main/java/fr/neatmonster/nocheatplus/compat/cb2763/MCAccessCB2763.java b/NCPCompatCB2763/src/main/java/fr/neatmonster/nocheatplus/compat/cb2763/MCAccessCB2763.java
@@ -18,7 +18,6 @@
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 
-import fr.neatmonster.nocheatplus.checks.moving.LocUtil;
 import fr.neatmonster.nocheatplus.compat.AlmostBoolean;
 import fr.neatmonster.nocheatplus.compat.MCAccess;
 import fr.neatmonster.nocheatplus.utilities.BlockCache;
@@ -166,13 +165,13 @@ public boolean hasGravity(final Material mat) {
 		}
 	}
 
-	@Override
-	public void correctDirection(final Player player) {
-		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
-		// Main direction.
-		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
-		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
-		// Consider setting the lastYaw here too.
-	}
+//	@Override
+//	public void correctDirection(final Player player) {
+//		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
+//		// Main direction.
+//		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
+//		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
+//		// Consider setting the lastYaw here too.
+//	}
 
 }
diff --git a/NCPCompatCB2794/src/main/java/fr/neatmonster/nocheatplus/compat/cb2794/MCAccessCB2794.java b/NCPCompatCB2794/src/main/java/fr/neatmonster/nocheatplus/compat/cb2794/MCAccessCB2794.java
@@ -18,7 +18,6 @@
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 
-import fr.neatmonster.nocheatplus.checks.moving.LocUtil;
 import fr.neatmonster.nocheatplus.compat.AlmostBoolean;
 import fr.neatmonster.nocheatplus.compat.MCAccess;
 import fr.neatmonster.nocheatplus.utilities.BlockCache;
@@ -160,13 +159,13 @@ public boolean hasGravity(final Material mat) {
 		return mat.hasGravity();
 	}
 
-	@Override
-	public void correctDirection(final Player player) {
-		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
-		// Main direction.
-		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
-		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
-		// Consider setting the lastYaw here too.
-	}
+//	@Override
+//	public void correctDirection(final Player player) {
+//		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
+//		// Main direction.
+//		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
+//		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
+//		// Consider setting the lastYaw here too.
+//	}
 
 }
diff --git a/NCPCompatCB2808/src/main/java/fr/neatmonster/nocheatplus/compat/cb2808/MCAccessCB2808.java b/NCPCompatCB2808/src/main/java/fr/neatmonster/nocheatplus/compat/cb2808/MCAccessCB2808.java
@@ -18,7 +18,6 @@
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 
-import fr.neatmonster.nocheatplus.checks.moving.LocUtil;
 import fr.neatmonster.nocheatplus.compat.AlmostBoolean;
 import fr.neatmonster.nocheatplus.compat.MCAccess;
 import fr.neatmonster.nocheatplus.utilities.BlockCache;
@@ -161,13 +160,13 @@ public boolean hasGravity(final Material mat) {
 		return mat.hasGravity();
 	}
 
-	@Override
-	public void correctDirection(final Player player) {
-		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
-		// Main direction.
-		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
-		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
-		// Consider setting the lastYaw here too.
-	}
+//	@Override
+//	public void correctDirection(final Player player) {
+//		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
+//		// Main direction.
+//		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
+//		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
+//		// Consider setting the lastYaw here too.
+//	}
 
 }
diff --git a/NCPCompatCB2882/src/main/java/fr/neatmonster/nocheatplus/compat/cb2882/MCAccessCB2882.java b/NCPCompatCB2882/src/main/java/fr/neatmonster/nocheatplus/compat/cb2882/MCAccessCB2882.java
@@ -18,7 +18,6 @@
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 
-import fr.neatmonster.nocheatplus.checks.moving.LocUtil;
 import fr.neatmonster.nocheatplus.compat.AlmostBoolean;
 import fr.neatmonster.nocheatplus.compat.MCAccess;
 import fr.neatmonster.nocheatplus.utilities.BlockCache;
@@ -162,13 +161,13 @@ public boolean hasGravity(final Material mat) {
 		return mat.hasGravity();
 	}
 
-	@Override
-	public void correctDirection(final Player player) {
-		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
-		// Main direction.
-		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
-		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
-		// Consider setting the lastYaw here too.
-	}
+//	@Override
+//	public void correctDirection(final Player player) {
+//		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
+//		// Main direction.
+//		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
+//		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
+//		// Consider setting the lastYaw here too.
+//	}
 
 }
diff --git a/NCPCompatCB2922/src/main/java/fr/neatmonster/nocheatplus/compat/cb2922/MCAccessCB2922.java b/NCPCompatCB2922/src/main/java/fr/neatmonster/nocheatplus/compat/cb2922/MCAccessCB2922.java
@@ -18,10 +18,8 @@
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 
-import fr.neatmonster.nocheatplus.checks.moving.LocUtil;
 import fr.neatmonster.nocheatplus.compat.AlmostBoolean;
 import fr.neatmonster.nocheatplus.compat.MCAccess;
-import fr.neatmonster.nocheatplus.compat.cb2922.BlockCacheCB2922;
 import fr.neatmonster.nocheatplus.utilities.BlockCache;
 import fr.neatmonster.nocheatplus.utilities.ReflectionUtil;
 
@@ -163,13 +161,13 @@ public boolean hasGravity(final Material mat) {
 		return mat.hasGravity();
 	}
 
-	@Override
-	public void correctDirection(final Player player) {
-		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
-		// Main direction.
-		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
-		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
-		// Consider setting the lastYaw here too.
-	}
+//	@Override
+//	public void correctDirection(final Player player) {
+//		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
+//		// Main direction.
+//		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
+//		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
+//		// Consider setting the lastYaw here too.
+//	}
 
 }
diff --git a/NCPCompatCB3026/src/main/java/fr/neatmonster/nocheatplus/compat/cb3026/MCAccessCB3026.java b/NCPCompatCB3026/src/main/java/fr/neatmonster/nocheatplus/compat/cb3026/MCAccessCB3026.java
@@ -18,7 +18,6 @@
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 
-import fr.neatmonster.nocheatplus.checks.moving.LocUtil;
 import fr.neatmonster.nocheatplus.compat.AlmostBoolean;
 import fr.neatmonster.nocheatplus.compat.MCAccess;
 import fr.neatmonster.nocheatplus.utilities.BlockCache;
@@ -162,13 +161,13 @@ public boolean hasGravity(final Material mat) {
 		return mat.hasGravity();
 	}
 
-	@Override
-	public void correctDirection(final Player player) {
-		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
-		// Main direction.
-		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
-		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
-		// Consider setting the lastYaw here too.
-	}
+//	@Override
+//	public void correctDirection(final Player player) {
+//		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
+//		// Main direction.
+//		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
+//		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
+//		// Consider setting the lastYaw here too.
+//	}
 
 }
diff --git a/NCPCompatCBDev/src/main/java/fr/neatmonster/nocheatplus/compat/cbdev/MCAccessCBDev.java b/NCPCompatCBDev/src/main/java/fr/neatmonster/nocheatplus/compat/cbdev/MCAccessCBDev.java
@@ -18,7 +18,6 @@
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 
-import fr.neatmonster.nocheatplus.checks.moving.LocUtil;
 import fr.neatmonster.nocheatplus.compat.AlmostBoolean;
 import fr.neatmonster.nocheatplus.compat.MCAccess;
 import fr.neatmonster.nocheatplus.utilities.BlockCache;
@@ -181,13 +180,13 @@ public boolean hasGravity(final Material mat) {
 		return mat.hasGravity();
 	}
 
-	@Override
-	public void correctDirection(final Player player) {
-		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
-		// Main direction.
-		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
-		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
-		// Consider setting the lastYaw here too.
-	}
+//	@Override
+//	public void correctDirection(final Player player) {
+//		final EntityPlayer mcPlayer = ((CraftPlayer) player).getHandle();
+//		// Main direction.
+//		mcPlayer.yaw = LocUtil.correctYaw(mcPlayer.yaw);
+//		mcPlayer.pitch = LocUtil.correctPitch(mcPlayer.pitch);
+//		// Consider setting the lastYaw here too.
+//	}
 
 }