From 7487afafb660be8fb908599a2da383af3590a91c Mon Sep 17 00:00:00 2001
From: Patrikkk <patrikveres@hotmail.com>
Date: Tue, 2 Jun 2020 19:26:25 +0200
Subject: [PATCH] SyncTilePickingHandler - Remove redundant check. Properly
 document method.

---
 TShockAPI/Handlers/SyncTilePickingHandler.cs | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/TShockAPI/Handlers/SyncTilePickingHandler.cs b/TShockAPI/Handlers/SyncTilePickingHandler.cs
index a03bec66..d7ad1e13 100644
--- a/TShockAPI/Handlers/SyncTilePickingHandler.cs
+++ b/TShockAPI/Handlers/SyncTilePickingHandler.cs
@@ -11,19 +11,12 @@ namespace TShockAPI.Handlers
 	class SyncTilePickingHandler : IPacketHandler<SyncTilePickingEventArgs>
 	{
 		/// <summary>
-		/// Invoked when player damages a tile.
+		/// Invoked when player damages a tile. Rejects the packet if its out of world bounds.
 		/// </summary>
 		/// <param name="sender"></param>
 		/// <param name="args"></param>
 		public void OnReceive(object sender, SyncTilePickingEventArgs args)
 		{
-			if (args.PlayerIndex != args.Player.Index)
-			{
-				TShock.Log.ConsoleDebug($"SyncTilePickingHandler: SyncTilePicking packet rejected for ID spoofing. Expected {args.Player.Index}, received {args.PlayerIndex} from {args.Player.Name}.");
-				args.Handled = true;
-				return;
-			}
-
 			if (args.TileX > Main.maxTilesX || args.TileX < 0
 			   || args.TileY > Main.maxTilesY || args.TileY < 0)
 			{