From 85ae59a767378b9bb7f916e6a04ef755217be855 Mon Sep 17 00:00:00 2001 From: CraftBukkit/Spigot Date: Wed, 19 Jun 2024 07:10:02 +1000 Subject: [PATCH] SPIGOT-7777: Speed attribute stays after death; missing EntityPotionEffectEvent call By: md_5 --- .../minecraft/world/entity/EntityLiving.patch | 94 ++++++++++--------- 1 file changed, 51 insertions(+), 43 deletions(-) diff --git a/paper-server/nms-patches/net/minecraft/world/entity/EntityLiving.patch b/paper-server/nms-patches/net/minecraft/world/entity/EntityLiving.patch index 63d030a940..a2989ee3aa 100644 --- a/paper-server/nms-patches/net/minecraft/world/entity/EntityLiving.patch +++ b/paper-server/nms-patches/net/minecraft/world/entity/EntityLiving.patch @@ -129,7 +129,15 @@ this.brain.clearMemories(); } -@@ -757,6 +817,17 @@ +@@ -698,6 +758,7 @@ + mobeffect.onMobRemoved(this, entity_removalreason); + } + ++ this.removeAllEffects(org.bukkit.event.entity.EntityPotionEffectEvent.Cause.DEATH); // CraftBukkit + this.activeEffects.clear(); + } + +@@ -757,6 +818,17 @@ } } @@ -147,7 +155,7 @@ if (nbttagcompound.contains("Health", 99)) { this.setHealth(nbttagcompound.getFloat("Health")); } -@@ -795,9 +866,32 @@ +@@ -795,9 +867,32 @@ } @@ -180,7 +188,7 @@ try { while (iterator.hasNext()) { Holder holder = (Holder) iterator.next(); -@@ -807,6 +901,12 @@ +@@ -807,6 +902,12 @@ this.onEffectUpdated(mobeffect, true, (Entity) null); })) { if (!this.level().isClientSide) { @@ -193,7 +201,7 @@ iterator.remove(); this.onEffectRemoved(mobeffect); } -@@ -817,6 +917,17 @@ +@@ -817,6 +918,17 @@ } catch (ConcurrentModificationException concurrentmodificationexception) { ; } @@ -211,7 +219,7 @@ if (this.effectsDirty) { if (!this.level().isClientSide) { -@@ -932,7 +1043,13 @@ +@@ -932,7 +1044,13 @@ this.entityData.set(EntityLiving.DATA_EFFECT_PARTICLES, List.of()); } @@ -225,7 +233,7 @@ if (this.level().isClientSide) { return false; } else { -@@ -941,7 +1058,14 @@ +@@ -941,7 +1059,14 @@ boolean flag; for (flag = false; iterator.hasNext(); flag = true) { @@ -241,7 +249,7 @@ iterator.remove(); } -@@ -970,20 +1094,50 @@ +@@ -970,20 +1095,50 @@ return this.addEffect(mobeffect, (Entity) null); } @@ -293,7 +301,7 @@ flag = true; } -@@ -1014,13 +1168,39 @@ +@@ -1014,13 +1169,39 @@ return this.getType().is(TagsEntity.INVERTED_HEALING_AND_HARM); } @@ -334,7 +342,7 @@ if (mobeffect != null) { this.onEffectRemoved(mobeffect); -@@ -1118,20 +1298,55 @@ +@@ -1118,20 +1299,55 @@ } @@ -391,7 +399,7 @@ this.entityData.set(EntityLiving.DATA_HEALTH_ID, MathHelper.clamp(f, 0.0F, this.getMaxHealth())); } -@@ -1145,7 +1360,7 @@ +@@ -1145,7 +1361,7 @@ return false; } else if (this.level().isClientSide) { return false; @@ -400,7 +408,7 @@ return false; } else if (damagesource.is(DamageTypeTags.IS_FIRE) && this.hasEffect(MobEffects.FIRE_RESISTANCE)) { return false; -@@ -1156,10 +1371,11 @@ +@@ -1156,10 +1372,11 @@ this.noActionTime = 0; float f1 = f; @@ -414,7 +422,7 @@ this.hurtCurrentlyUsedShield(f); f2 = f; f = 0.0F; -@@ -1180,7 +1396,8 @@ +@@ -1180,7 +1397,8 @@ f *= 5.0F; } @@ -424,7 +432,7 @@ this.hurtHelmet(damagesource, f); f *= 0.75F; } -@@ -1188,18 +1405,27 @@ +@@ -1188,18 +1406,27 @@ this.walkAnimation.setSpeed(1.5F); boolean flag1 = true; @@ -456,7 +464,7 @@ this.hurtDuration = 10; this.hurtTime = this.hurtDuration; } -@@ -1245,7 +1471,7 @@ +@@ -1245,7 +1472,7 @@ this.level().broadcastDamageEvent(this, damagesource); } @@ -465,7 +473,7 @@ this.markHurt(); } -@@ -1265,7 +1491,7 @@ +@@ -1265,7 +1492,7 @@ d1 = damagesource.getSourcePosition().z() - this.getZ(); } @@ -474,7 +482,7 @@ if (!flag) { this.indicateDamage(d0, d1); } -@@ -1284,7 +1510,7 @@ +@@ -1284,7 +1511,7 @@ this.playHurtSound(damagesource); } @@ -483,7 +491,7 @@ if (flag2) { this.lastDamageSource = damagesource; -@@ -1318,7 +1544,7 @@ +@@ -1318,7 +1545,7 @@ } protected void blockedByShield(EntityLiving entityliving) { @@ -492,7 +500,7 @@ } private boolean checkTotemDeathProtection(DamageSource damagesource) { -@@ -1329,19 +1555,32 @@ +@@ -1329,19 +1556,32 @@ EnumHand[] aenumhand = EnumHand.values(); int i = aenumhand.length; @@ -529,7 +537,7 @@ EntityPlayer entityplayer = (EntityPlayer) this; entityplayer.awardStat(StatisticList.ITEM_USED.get(Items.TOTEM_OF_UNDYING)); -@@ -1350,14 +1589,16 @@ +@@ -1350,14 +1590,16 @@ } this.setHealth(1.0F); @@ -551,7 +559,7 @@ } } -@@ -1464,14 +1705,22 @@ +@@ -1464,14 +1706,22 @@ IBlockData iblockdata = Blocks.WITHER_ROSE.defaultBlockState(); if (this.level().getBlockState(blockposition).isAir() && iblockdata.canSurvive(this.level(), blockposition)) { @@ -576,7 +584,7 @@ this.level().addFreshEntity(entityitem); } } -@@ -1482,26 +1731,41 @@ +@@ -1482,26 +1732,41 @@ protected void dropAllDeathLoot(WorldServer worldserver, DamageSource damagesource) { boolean flag = this.lastHurtByPlayerTime > 0; @@ -621,7 +629,7 @@ } protected void dropCustomDeathLoot(WorldServer worldserver, DamageSource damagesource, boolean flag) {} -@@ -1540,9 +1804,14 @@ +@@ -1540,9 +1805,14 @@ } public void knockback(double d0, double d1, double d2) { @@ -638,7 +646,7 @@ Vec3D vec3d; -@@ -1552,7 +1821,14 @@ +@@ -1552,7 +1822,14 @@ Vec3D vec3d1 = (new Vec3D(d1, 0.0D, d2)).normalize().scale(d0); @@ -654,7 +662,7 @@ } } -@@ -1613,6 +1889,28 @@ +@@ -1613,6 +1890,28 @@ return itemstack.getEatingSound(); } @@ -683,7 +691,7 @@ public Optional getLastClimbablePos() { return this.lastClimbablePos; } -@@ -1666,9 +1964,14 @@ +@@ -1666,9 +1965,14 @@ int i = this.calculateFallDamage(f, f1); if (i > 0) { @@ -699,7 +707,7 @@ return true; } else { return flag; -@@ -1738,7 +2041,7 @@ +@@ -1738,7 +2042,7 @@ protected float getDamageAfterArmorAbsorb(DamageSource damagesource, float f) { if (!damagesource.is(DamageTypeTags.BYPASSES_ARMOR)) { @@ -708,7 +716,7 @@ f = CombatMath.getDamageAfterAbsorb(this, f, damagesource, (float) this.getArmorValue(), (float) this.getAttributeValue(GenericAttributes.ARMOR_TOUGHNESS)); } -@@ -1749,7 +2052,8 @@ +@@ -1749,7 +2053,8 @@ if (damagesource.is(DamageTypeTags.BYPASSES_EFFECTS)) { return f; } else { @@ -718,7 +726,7 @@ int i = (this.getEffect(MobEffects.DAMAGE_RESISTANCE).getAmplifier() + 1) * 5; int j = 25 - i; float f1 = f * (float) j; -@@ -1792,16 +2096,125 @@ +@@ -1792,16 +2097,125 @@ } } @@ -852,7 +860,7 @@ if (f2 > 0.0F && f2 < 3.4028235E37F) { Entity entity = damagesource.getEntity(); -@@ -1812,13 +2225,47 @@ +@@ -1812,13 +2226,47 @@ } } @@ -902,7 +910,7 @@ } public CombatTracker getCombatTracker() { -@@ -1843,8 +2290,18 @@ +@@ -1843,8 +2291,18 @@ } public final void setArrowCount(int i) { @@ -922,7 +930,7 @@ public final int getStingerCount() { return (Integer) this.entityData.get(EntityLiving.DATA_STINGER_COUNT_ID); -@@ -2086,6 +2543,12 @@ +@@ -2086,6 +2544,12 @@ public abstract ItemStack getItemBySlot(EnumItemSlot enumitemslot); @@ -935,7 +943,7 @@ public abstract void setItemSlot(EnumItemSlot enumitemslot, ItemStack itemstack); public Iterable getHandSlots() { -@@ -2336,6 +2799,7 @@ +@@ -2336,6 +2800,7 @@ } if (this.onGround() && !this.level().isClientSide) { @@ -943,7 +951,7 @@ this.setSharedFlag(7, false); } } else { -@@ -2506,7 +2970,7 @@ +@@ -2506,7 +2971,7 @@ } } @@ -952,7 +960,7 @@ if (this.tickCount % 20 == 0) { this.getCombatTracker().recheckStatus(); } -@@ -2610,7 +3074,7 @@ +@@ -2610,7 +3075,7 @@ } @@ -961,7 +969,7 @@ Map map = this.collectEquipmentChanges(); if (map != null) { -@@ -2949,6 +3413,7 @@ +@@ -2949,6 +3414,7 @@ } if (!this.level().isClientSide) { @@ -969,7 +977,7 @@ this.setSharedFlag(7, flag); } -@@ -3141,14 +3606,21 @@ +@@ -3141,14 +3607,21 @@ @Override public boolean isPickable() { @@ -993,7 +1001,7 @@ @Override public float getYHeadRot() { return this.yHeadRot; -@@ -3346,7 +3818,26 @@ +@@ -3346,7 +3819,26 @@ } else { if (!this.useItem.isEmpty() && this.isUsingItem()) { this.triggerItemUseEffects(this.useItem, 16); @@ -1021,7 +1029,7 @@ if (itemstack != this.useItem) { this.setItemInHand(enumhand, itemstack); -@@ -3424,6 +3915,12 @@ +@@ -3424,6 +3916,12 @@ } public boolean randomTeleport(double d0, double d1, double d2, boolean flag) { @@ -1034,7 +1042,7 @@ double d3 = this.getX(); double d4 = this.getY(); double d5 = this.getZ(); -@@ -3448,16 +3945,41 @@ +@@ -3448,16 +3946,41 @@ } if (flag2) { @@ -1079,7 +1087,7 @@ } else { if (flag) { world.broadcastEntityEvent(this, (byte) 46); -@@ -3469,7 +3991,7 @@ +@@ -3469,7 +3992,7 @@ entitycreature.getNavigation().stop(); } @@ -1088,7 +1096,7 @@ } } -@@ -3562,7 +4084,7 @@ +@@ -3562,7 +4085,7 @@ } public void stopSleeping() { @@ -1097,7 +1105,7 @@ World world = this.level(); java.util.Objects.requireNonNull(world); -@@ -3596,7 +4118,7 @@ +@@ -3596,7 +4119,7 @@ @Nullable public EnumDirection getBedOrientation() { @@ -1106,7 +1114,7 @@ return blockposition != null ? BlockBed.getBedOrientation(this.level(), blockposition) : null; } -@@ -3633,7 +4155,7 @@ +@@ -3633,7 +4156,7 @@ FoodInfo.b foodinfo_b = (FoodInfo.b) iterator.next(); if (this.random.nextFloat() < foodinfo_b.probability()) {