--- a/net/minecraft/world/entity/animal/EntityBee.java +++ b/net/minecraft/world/entity/animal/EntityBee.java @@ -89,6 +89,12 @@ import net.minecraft.world.level.pathfinder.PathType; import net.minecraft.world.phys.Vec3D; +// CraftBukkit start +import org.bukkit.craftbukkit.event.CraftEventFactory; +import org.bukkit.event.entity.EntityPotionEffectEvent; +import org.bukkit.event.entity.EntityTargetEvent; +// CraftBukkit end + public class EntityBee extends EntityAnimal implements IEntityAngerable, EntityBird { public static final float FLAP_DEGREES_PER_TICK = 120.32113F; @@ -194,12 +200,19 @@ @Override public void addAdditionalSaveData(NBTTagCompound nbttagcompound) { + // CraftBukkit start - selectively save data + addAdditionalSaveData(nbttagcompound, true); + } + + @Override + public void addAdditionalSaveData(NBTTagCompound nbttagcompound, boolean includeAll) { + // CraftBukkit end super.addAdditionalSaveData(nbttagcompound); - if (this.hasHive()) { + if (includeAll && this.hasHive()) { // CraftBukkit - selectively save hive nbttagcompound.put("hive_pos", GameProfileSerializer.writeBlockPos(this.getHivePos())); } - if (this.hasSavedFlowerPos()) { + if (includeAll && this.hasSavedFlowerPos()) { // CraftBukkit - selectively save flower nbttagcompound.put("flower_pos", GameProfileSerializer.writeBlockPos(this.getSavedFlowerPos())); } @@ -219,8 +232,8 @@ this.ticksWithoutNectarSinceExitingHive = nbttagcompound.getInt("TicksSincePollination"); this.stayOutOfHiveCountdown = nbttagcompound.getInt("CannotEnterHiveTicks"); this.numCropsGrownSincePollination = nbttagcompound.getInt("CropsGrownSincePollination"); - this.hivePos = (BlockPosition) GameProfileSerializer.readBlockPos(nbttagcompound, "hive_pos").orElse((Object) null); - this.savedFlowerPos = (BlockPosition) GameProfileSerializer.readBlockPos(nbttagcompound, "flower_pos").orElse((Object) null); + this.hivePos = (BlockPosition) GameProfileSerializer.readBlockPos(nbttagcompound, "hive_pos").orElse(null); // CraftBukkit - decompile error + this.savedFlowerPos = (BlockPosition) GameProfileSerializer.readBlockPos(nbttagcompound, "flower_pos").orElse(null); // CraftBukkit - decompile error this.readPersistentAngerSaveData(this.level(), nbttagcompound); } @@ -244,7 +257,7 @@ } if (b0 > 0) { - entityliving.addEffect(new MobEffect(MobEffects.POISON, b0 * 20, 0), this); + entityliving.addEffect(new MobEffect(MobEffects.POISON, b0 * 20, 0), this, EntityPotionEffectEvent.Cause.ATTACK); // CraftBukkit } } @@ -502,7 +515,7 @@ @Nullable TileEntityBeehive getBeehiveBlockEntity() { - return this.hivePos == null ? null : (this.isTooFarAway(this.hivePos) ? null : (TileEntityBeehive) this.level().getBlockEntity(this.hivePos, TileEntityTypes.BEEHIVE).orElse((Object) null)); + return this.hivePos == null ? null : (this.isTooFarAway(this.hivePos) ? null : (TileEntityBeehive) this.level().getBlockEntity(this.hivePos, TileEntityTypes.BEEHIVE).orElse(null)); // CraftBukkit - decompile error } boolean isHiveValid() { @@ -638,8 +651,14 @@ if (this.isInvulnerableTo(worldserver, damagesource)) { return false; } else { + // CraftBukkit start - Only stop pollinating if entity was damaged + boolean result = super.hurtServer(worldserver, damagesource, f); + if (!result) { + return result; + } + // CraftBukkit end this.beePollinateGoal.stopPollinating(); - return super.hurtServer(worldserver, damagesource, f); + return result; // CraftBukkit } } @@ -1048,7 +1067,7 @@ e() { super(); - this.travellingTicks = EntityBee.this.level().random.nextInt(10); + this.travellingTicks = EntityBee.this.random.nextInt(10); // CraftBukkit - SPIGOT-7495: Give Bees another chance and let them use their own random, avoid concurrency issues this.blacklistedTargets = Lists.newArrayList(); this.setFlags(EnumSet.of(PathfinderGoal.Type.MOVE)); } @@ -1162,7 +1181,7 @@ f() { super(); - this.travellingTicks = EntityBee.this.level().random.nextInt(10); + this.travellingTicks = EntityBee.this.random.nextInt(10); // CraftBukkit - SPIGOT-7495: Give Bees another chance and let them use their own random, avoid concurrency issues this.setFlags(EnumSet.of(PathfinderGoal.Type.MOVE)); } @@ -1267,7 +1286,7 @@ } } - if (iblockdata1 != null) { + if (iblockdata1 != null && CraftEventFactory.callEntityChangeBlockEvent(EntityBee.this, blockposition, iblockdata1)) { // CraftBukkit EntityBee.this.level().levelEvent(2011, blockposition, 15); EntityBee.this.level().setBlockAndUpdate(blockposition, iblockdata1); EntityBee.this.incrementNumCropsGrownSincePollination(); @@ -1344,7 +1363,7 @@ @Override protected void alertOther(EntityInsentient entityinsentient, EntityLiving entityliving) { if (entityinsentient instanceof EntityBee && this.mob.hasLineOfSight(entityliving)) { - entityinsentient.setTarget(entityliving); + entityinsentient.setTarget(entityliving, EntityTargetEvent.TargetReason.TARGET_ATTACKED_ENTITY, true); // CraftBukkit - reason } } @@ -1353,7 +1372,7 @@ private static class c extends PathfinderGoalNearestAttackableTarget { c(EntityBee entitybee) { - Objects.requireNonNull(entitybee); + // Objects.requireNonNull(entitybee); // CraftBukkit - decompile error super(entitybee, EntityHuman.class, 10, true, false, entitybee::isAngryAt); }