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

fix(Bouncer/SendTileRectHandler): MatchRemoval incorrect check range

Browse source
This commit is contained in:
LaoSparrow 2025-02-28 23:01:40 +08:00
parent e4e28cb1b5
commit 1e23785a04
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
TShockAPI/Handlers/SendTileRectHandler.cs
View file

@ -327,7 +327,7 @@ namespace TShockAPI.Handlers
private bool MatchRemoval(TSPlayer player, TileRect rect) private bool MatchRemoval(TSPlayer player, TileRect rect)
{ {
for (int x = rect.X; x < rect.Y + rect.Width; x++) for (int x = rect.X; x < rect.X + rect.Width; x++)
{ {
for (int y = rect.Y; y < rect.Y + rect.Height; y++) for (int y = rect.Y; y < rect.Y + rect.Height; y++)
{ {
@ -910,7 +910,7 @@ namespace TShockAPI.Handlers
} }
} }
/* /*
* This is a copy of the `WorldGen.Convert` method with the following precise changes: * This is a copy of the `WorldGen.Convert` method with the following precise changes:
* - Added a `MockTile tile` parameter * - Added a `MockTile tile` parameter
* - Changed the `i` and `j` parameters to `k` and `l` * - Changed the `i` and `j` parameters to `k` and `l`
@ -921,7 +921,7 @@ namespace TShockAPI.Handlers
* - Removed the ifs checking the bounds of the tile and wall types * - Removed the ifs checking the bounds of the tile and wall types
* - Removed branches that would call `WorldGen.KillTile` * - Removed branches that would call `WorldGen.KillTile`
* - Changed branches depending on randomness to instead set the property to both values after one another * - Changed branches depending on randomness to instead set the property to both values after one another
* *
* This overall leads to a method that can be called on a MockTile and real-world coordinates and will spit out the proper conversion changes into the MockTile. * This overall leads to a method that can be called on a MockTile and real-world coordinates and will spit out the proper conversion changes into the MockTile.
*/ */