From 4b08d616733fa6350442d2b46ce161820c6c0a9d Mon Sep 17 00:00:00 2001
From: Lucas Nicodemus <shank@shanked.me>
Date: Mon, 11 Dec 2017 00:17:58 -0700
Subject: [PATCH] Change null check to return true in HandlePlayerUpdate

After confirming with @QuiCM on this, we couldn't figure out why
this was set to false in the first place. As a result, we changed
it to true to conform with usual logic (bad stuff? reject it.).
---
 TShockAPI/GetDataHandlers.cs | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/TShockAPI/GetDataHandlers.cs b/TShockAPI/GetDataHandlers.cs
index 2545a2f0..0ef703af 100644
--- a/TShockAPI/GetDataHandlers.cs
+++ b/TShockAPI/GetDataHandlers.cs
@@ -2076,10 +2076,7 @@ namespace TShockAPI
 		{
 			if (args.Player == null || args.TPlayer == null || args.Data == null)
 			{
-				// Is this really the best option?
-				// If we're getting a packet that doesn't have a player or a TPlayer or data...
-				// Should we really let it through?
-				return false;
+				return true;
 			}
 
 			byte plr = args.Data.ReadInt8();