fix (dependencies): compatibility to recent TShock & OTAPI builds

Author: Stealownz

Implementation/ChestManager.cs

@@ -349,11 +349,11 @@ public IChest PlaceChest(ushort tileType, int style, DPoint placeLocation) {
       if (tileType == TileID.Containers2)
         storageType = 4;
 
-      TSPlayer.All.SendData(PacketTypes.TileKill, string.Empty, storageType, placeLocation.X, placeLocation.Y, style, chestIndex);
+      TSPlayer.All.SendData(PacketTypes.PlaceChest, string.Empty, storageType, placeLocation.X, placeLocation.Y, style, chestIndex);
       // The client will always show open / close animations for the latest chest index. But when there are multiple chests with id 999
       // this will look awkard, so instead tell the client about another 999 chest on a location where the animation will never be noticed by the player.
       if (chestIndex == ChestManager.DummyChestIndex)
-        TSPlayer.All.SendData(PacketTypes.TileKill, string.Empty, storageType, 0, 0, style, chestIndex);
+        TSPlayer.All.SendData(PacketTypes.PlaceChest, string.Empty, storageType, 0, 0, style, chestIndex);
 
       return chest;
     }
@@ -445,7 +445,7 @@ public void DestroyChest(IChest chest) {
         }
 
         WorldGen.KillTile(chest.Location.X, chest.Location.Y);
-        TSPlayer.All.SendData(PacketTypes.TileKill, string.Empty, 1, chest.Location.X, chest.Location.Y, 0, chestIndex);
+        TSPlayer.All.SendData(PacketTypes.PlaceChest, string.Empty, 1, chest.Location.X, chest.Location.Y, 0, chestIndex);
       }
     }
 

Implementation/ProtectionManager.cs

@@ -259,7 +259,7 @@ public ProtectionEntry CreateProtection(
       if (checkIfBlockTypeProtectableByConfig && !this.Config.ManuallyProtectableTiles[tile.type])
         throw new InvalidBlockTypeException(blockType);
 
-      if (checkTShockBuildAndRegionAccess && TShock.CheckTilePermission(player, tileLocation.X, tileLocation.Y))
+      if (checkTShockBuildAndRegionAccess && !player.HasBuildPermission(tileLocation.X, tileLocation.Y))
         throw new TileProtectedException(tileLocation);
 
       if (

Implementation/ProtectorPlugin.cs

@@ -340,7 +340,7 @@ private void TShock_PlayerSpawn(object sender, GetDataHandlers.SpawnEventArgs e)
       if (this.isDisposed || !this.hooksEnabled || e.Handled)
         return;
 
-      TSPlayer player = TShock.Players[e.Player];
+      TSPlayer player = e.Player;
       if (player == null || !player.IsLoggedIn)
         return;
 

Implementation/UserInteractionHandler.cs

@@ -1168,7 +1168,7 @@ private void ShareGroupCommand_Exec(CommandArgs args) {
         groupName = args.ParamsToSingleString();
       }
 
-      if (TShock.Utils.GetGroup(groupName) == null) {
+      if (TShock.Groups.GetGroupByName(groupName) == null) {
         args.Player.SendErrorMessage($"The group \"{groupName}\" does not exist.");
 
         return;
@@ -3568,7 +3568,7 @@ public bool TrySwapChestData(TSPlayer player, DPoint anyChestTileLocation, out I
           //TSPlayer.All.SendData(PacketTypes.ChestName, string.Empty, chest.Index, chestLocation.X, chestLocation.Y);
 
           // Tell the client to remove the chest with the given index from its own chest array.
-          TSPlayer.All.SendData(PacketTypes.TileKill, string.Empty, 1, chestLocation.X, chestLocation.Y, 0, chest.Index);
+          TSPlayer.All.SendData(PacketTypes.PlaceChest, string.Empty, 1, chestLocation.X, chestLocation.Y, 0, chest.Index);
           TSPlayer.All.SendTileSquare(chestLocation.X, chestLocation.Y, 2);
           player?.SendWarningMessage("This chest is now a Protector chest.");
         }
@@ -3928,7 +3928,7 @@ private void DestroyBlockOrObject(DPoint tileLocation) {
         this.ChestManager.DestroyChest(tileLocation);
       } else {
         WorldGen.KillTile(tileLocation.X, tileLocation.Y, false, false, true);
-        TSPlayer.All.SendData(PacketTypes.TileKill, string.Empty, 0, tileLocation.X, tileLocation.Y, 0, -1);
+        TSPlayer.All.SendData(PacketTypes.PlaceChest, string.Empty, 0, tileLocation.X, tileLocation.Y, 0, -1);
       }
     }