SPIGOT-5294: Cancelled EntityPickupItemEvent and PlayerPickupItemEvent alter on-ground stacks

This commit is contained in:
md_5 2019-09-02 13:17:54 +10:00
parent 403ac18579
commit 763e560135

View file

@ -97,7 +97,7 @@
this.velocityChanged();
this.f = (int) ((float) this.f - f);
if (this.f <= 0) {
@@ -267,6 +289,38 @@
@@ -267,6 +289,40 @@
Item item = itemstack.getItem();
int i = itemstack.getCount();
@ -112,6 +112,7 @@
+ playerEvent.setCancelled(!entityhuman.canPickUpLoot);
+ this.world.getServer().getPluginManager().callEvent(playerEvent);
+ if (playerEvent.isCancelled()) {
+ itemstack.setCount(i); // SPIGOT-5294 - restore count
+ return;
+ }
+
@ -120,10 +121,11 @@
+ entityEvent.setCancelled(!entityhuman.canPickUpLoot);
+ this.world.getServer().getPluginManager().callEvent(entityEvent);
+ if (entityEvent.isCancelled()) {
+ itemstack.setCount(i); // SPIGOT-5294 - restore count
+ return;
+ }
+
+ itemstack.setCount(canHold + remaining);
+ itemstack.setCount(canHold + remaining); // = i
+
+ // Possibly < 0; fix here so we do not have to modify code below
+ this.pickupDelay = 0;
@ -136,7 +138,7 @@
if (this.pickupDelay == 0 && (this.owner == null || 6000 - this.age <= 200 || this.owner.equals(entityhuman.getUniqueID())) && entityhuman.inventory.pickup(itemstack)) {
entityhuman.receive(this, i);
if (itemstack.isEmpty()) {
@@ -309,7 +363,9 @@
@@ -309,7 +365,9 @@
}
public void setItemStack(ItemStack itemstack) {