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

Revert "Bouncer: Handle case where prefix < 1"

Browse source 
This reverts commit 3f79a904da.

If prefix is < 1 and we block this event, clients can no longer
delete picked up items. This is what caused what Joshwoo reported.
This commit is contained in:
Lucas Nicodemus 2017-12-17 12:25:23 -07:00
parent c1de974e16
commit 263c0bc402
1 changed files with 4 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
TShockAPI/Bouncer.cs
View file

@ -359,8 +359,10 @@ namespace TShockAPI
return;
}
//make sure the prefix is a legit value
if (prefix > PrefixID.Count || prefix < 1)
// make sure the prefix is a legit value
// Note: Not checking if prefix is less than 1 because if it is, this check
// will break item pickups on the client.
if (prefix > PrefixID.Count)
{
args.Player.SendData(PacketTypes.ItemDrop, "", id);
args.Handled = true;