mirror of
https://github.com/PaperMC/Paper.git
synced 2024-12-04 22:14:40 +01:00
c98d61bfcd
Also use correct block position for spawning hanging entities
85 lines
3.8 KiB
Diff
85 lines
3.8 KiB
Diff
--- a/net/minecraft/server/EntityItemFrame.java
|
|
+++ b/net/minecraft/server/EntityItemFrame.java
|
|
@@ -52,15 +52,29 @@
|
|
@Override
|
|
protected void updateBoundingBox() {
|
|
if (this.direction != null) {
|
|
+ // CraftBukkit start code moved in to calculateBoundingBox
|
|
+ this.a(calculateBoundingBox(this, this.blockPosition, this.direction, this.getHangingWidth(), this.getHangingHeight()));
|
|
+ // CraftBukkit end
|
|
+ }
|
|
+ }
|
|
+
|
|
+ // CraftBukkit start - break out BB calc into own method
|
|
+ public static AxisAlignedBB calculateBoundingBox(@Nullable Entity entity, BlockPosition blockPosition, EnumDirection direction, int width, int height) {
|
|
+ {
|
|
double d0 = 0.46875D;
|
|
|
|
- this.locX = (double) this.blockPosition.getX() + 0.5D - (double) this.direction.getAdjacentX() * 0.46875D;
|
|
- this.locY = (double) this.blockPosition.getY() + 0.5D - (double) this.direction.getAdjacentY() * 0.46875D;
|
|
- this.locZ = (double) this.blockPosition.getZ() + 0.5D - (double) this.direction.getAdjacentZ() * 0.46875D;
|
|
- double d1 = (double) this.getHangingWidth();
|
|
- double d2 = (double) this.getHangingHeight();
|
|
- double d3 = (double) this.getHangingWidth();
|
|
- EnumDirection.EnumAxis enumdirection_enumaxis = this.direction.k();
|
|
+ double locX = (double) blockPosition.getX() + 0.5D - (double) direction.getAdjacentX() * 0.46875D;
|
|
+ double locY = (double) blockPosition.getY() + 0.5D - (double) direction.getAdjacentY() * 0.46875D;
|
|
+ double locZ = (double) blockPosition.getZ() + 0.5D - (double) direction.getAdjacentZ() * 0.46875D;
|
|
+ if (entity != null) {
|
|
+ entity.locX = locX;
|
|
+ entity.locY = locY;
|
|
+ entity.locZ = locZ;
|
|
+ }
|
|
+ double d1 = (double) width;
|
|
+ double d2 = (double) height;
|
|
+ double d3 = (double) width;
|
|
+ EnumDirection.EnumAxis enumdirection_enumaxis = direction.k();
|
|
|
|
switch (enumdirection_enumaxis) {
|
|
case X:
|
|
@@ -76,9 +90,10 @@
|
|
d1 /= 32.0D;
|
|
d2 /= 32.0D;
|
|
d3 /= 32.0D;
|
|
- this.a(new AxisAlignedBB(this.locX - d1, this.locY - d2, this.locZ - d3, this.locX + d1, this.locY + d2, this.locZ + d3));
|
|
+ return new AxisAlignedBB(locX - d1, locY - d2, locZ - d3, locX + d1, locY + d2, locZ + d3);
|
|
}
|
|
}
|
|
+ // CraftBukkit end
|
|
|
|
@Override
|
|
public boolean survives() {
|
|
@@ -108,6 +123,11 @@
|
|
return false;
|
|
} else if (!damagesource.isExplosion() && !this.getItem().isEmpty()) {
|
|
if (!this.world.isClientSide) {
|
|
+ // CraftBukkit start - fire EntityDamageEvent
|
|
+ if (org.bukkit.craftbukkit.event.CraftEventFactory.handleNonLivingEntityDamageEvent(this, damagesource, f, false) || this.dead) {
|
|
+ return true;
|
|
+ }
|
|
+ // CraftBukkit end
|
|
this.b(damagesource.getEntity(), false);
|
|
this.a(SoundEffects.ENTITY_ITEM_FRAME_REMOVE_ITEM, 1.0F, 1.0F);
|
|
}
|
|
@@ -193,6 +213,12 @@
|
|
}
|
|
|
|
public void setItem(ItemStack itemstack, boolean flag) {
|
|
+ // CraftBukkit start
|
|
+ this.setItem(itemstack, flag, true);
|
|
+ }
|
|
+
|
|
+ public void setItem(ItemStack itemstack, boolean flag, boolean playSound) {
|
|
+ // CraftBukkit end
|
|
if (!itemstack.isEmpty()) {
|
|
itemstack = itemstack.cloneItemStack();
|
|
itemstack.setCount(1);
|
|
@@ -200,7 +226,7 @@
|
|
}
|
|
|
|
this.getDataWatcher().set(EntityItemFrame.ITEM, itemstack);
|
|
- if (!itemstack.isEmpty()) {
|
|
+ if (!itemstack.isEmpty() && playSound) { // CraftBukkit
|
|
this.a(SoundEffects.ENTITY_ITEM_FRAME_ADD_ITEM, 1.0F, 1.0F);
|
|
}
|
|
|