themaster1970sf/TShock
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

Bouncer OnTileData - Add more checks on ReplaceTile/ReplaceWall

Browse source
This commit is contained in:
Patrikkk 2020-06-09 00:39:04 +02:00
parent 0d0641c8c0
commit 6c22cfd57c
2 changed files with 9 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

6
CHANGELOG.md
View file

@ -4,7 +4,11 @@ This is the rolling changelog for TShock for Terraria. Use past tense when addin
## Upcoming changes ## Upcoming changes
* Installed new sprinklers! * Installed new sprinklers!
* Fix issue where players could replace tiles with banned tiles without permission. (@Patrikkk) * Fix multiple holes in Bouncer OnTileData. (@Patrikkk, @hakusaro)
* Issue where players could replace tiles with banned tiles without permission.
* Including replace action in TilePlace threshold incrementation, so players cannot bypass the threshold while replacing tiles/walls.
* Including check for maxTileSets when player is replacing tiles, so players cannot send invalid tile data through the replace tile action.
* Including a check for ReplaceWall when the tile is a Breakable/CutTile.
## TShock 4.4.0 (Pre-release 11) ## TShock 4.4.0 (Pre-release 11)
* New permission `tshock.tp.pylon` to enable teleporting via Teleportation Pylons (@QuiCM) * New permission `tshock.tp.pylon` to enable teleporting via Teleportation Pylons (@QuiCM)

8
TShockAPI/Bouncer.cs
View file

@ -330,7 +330,7 @@ namespace TShockAPI
return; return;
} }
} }
else if (action == EditAction.PlaceTile || action == EditAction.PlaceWall) else if (action == EditAction.PlaceTile || action == EditAction.ReplaceTile || action == EditAction.PlaceWall || action == EditAction.ReplaceWall)
{ {
if ((action == EditAction.PlaceTile && TShock.Config.PreventInvalidPlaceStyle) && if ((action == EditAction.PlaceTile && TShock.Config.PreventInvalidPlaceStyle) &&
(MaxPlaceStyles.ContainsKey(editData) && style > MaxPlaceStyles[editData]) && (MaxPlaceStyles.ContainsKey(editData) && style > MaxPlaceStyles[editData]) &&
@ -437,7 +437,7 @@ namespace TShockAPI
} }
if (TShock.Config.AllowCutTilesAndBreakables && Main.tileCut[tile.type]) if (TShock.Config.AllowCutTilesAndBreakables && Main.tileCut[tile.type])
{ {
if (action == EditAction.KillWall) if (action == EditAction.KillWall || action == EditAction.ReplaceWall)
{ {
TShock.Log.ConsoleDebug("Bouncer / OnTileEdit rejected from sts allow cut from {0} {1} {2}", args.Player.Name, action, editData); TShock.Log.ConsoleDebug("Bouncer / OnTileEdit rejected from sts allow cut from {0} {1} {2}", args.Player.Name, action, editData);
args.Player.SendTileSquare(tileX, tileY, 1); args.Player.SendTileSquare(tileX, tileY, 1);
@ -536,7 +536,7 @@ namespace TShockAPI
return; return;
} }
if ((action == EditAction.PlaceTile || action == EditAction.ReplaceTile || action == EditAction.PlaceWall) && !args.Player.HasPermission(Permissions.ignoreplacetiledetection)) if ((action == EditAction.PlaceTile || action == EditAction.ReplaceTile || action == EditAction.PlaceWall || action == EditAction.ReplaceWall) && !args.Player.HasPermission(Permissions.ignoreplacetiledetection))
{ {
args.Player.TilePlaceThreshold++; args.Player.TilePlaceThreshold++;
var coords = new Vector2(tileX, tileY); var coords = new Vector2(tileX, tileY);
@ -545,7 +545,7 @@ namespace TShockAPI
args.Player.TilesCreated.Add(coords, Main.tile[tileX, tileY]); args.Player.TilesCreated.Add(coords, Main.tile[tileX, tileY]);
} }
if ((action == EditAction.KillTile || action == EditAction.KillTileNoItem || action == EditAction.KillWall) && Main.tileSolid[Main.tile[tileX, tileY].type] && if ((action == EditAction.KillTile || action == EditAction.KillTileNoItem || action == EditAction.ReplaceTile || action == EditAction.KillWall || action == EditAction.ReplaceWall) && Main.tileSolid[Main.tile[tileX, tileY].type] &&
!args.Player.HasPermission(Permissions.ignorekilltiledetection)) !args.Player.HasPermission(Permissions.ignorekilltiledetection))
{ {
args.Player.TileKillThreshold++; args.Player.TileKillThreshold++;