mirror of
https://github.com/PaperMC/Paper.git
synced 2024-11-22 07:08:47 +01:00
ac554ad46d
Updated Upstream (Bukkit/CraftBukkit) Upstream has released updates that appear to apply and compile correctly. This update has not been tested by PaperMC and as with ANY update, please do your own testing Bukkit Changes: fa99e752 PR-1007: Add ItemMeta#getAsComponentString() 94a91782 Fix copy-pasted BlockType.Typed documentation 9b34ac8c Largely restore deprecated PotionData API 51a6449b PR-1008: Deprecate ITEMS_TOOLS, removed in 1.20.5 702d15fe Fix Javadoc reference 42f6cdf4 PR-919: Add internal ItemType and BlockType, delegate Material methods to them 237bb37b SPIGOT-1166, SPIGOT-7647: Expose Damager BlockState in EntityDamageByBlockEvent 035ea146 SPIGOT-6993: Allow #setVelocity to change the speed of a fireball and add a note to #setDirection about it 8c7880fb PR-1004: Improve field rename handling and centralize conversion between bukkit and string more 87c90e93 SPIGOT-7650: Add DamageSource for EntityDeathEvent and PlayerDeathEvent CraftBukkit Changes: 4af0f22e8 SPIGOT-7664: Item meta should prevail over block states c2ccc46ec SPIGOT-7666: Fix access to llama and horse special slot 124ac66d7 SPIGOT-7665: Fix ThrownPotion#getEffects() implementation only bringing custom effects 66f1f439a Restore null page behaviour of signed books even though not strictly allowed by API 6118e5398 Fix regression listening to minecraft:brand custom payloads c1a26b366 Fix unnecessary and potential not thread-safe chat visibility check 12360a7ec Remove unused imports 147b098b4 PR-1397: Add ItemMeta#getAsComponentString() 428aefe0e Largely restore deprecated PotionData API afe5b5ee9 PR-1275: Add internal ItemType and BlockType, delegate Material methods to them 8afeafa7d SPIGOT-1166, SPIGOT-7647: Expose Damager BlockState in EntityDamageByBlockEvent 4e7d749d4 SPIGOT-6993: Allow #setVelocity to change the speed of a fireball and add a note to #setDirection about it 441880757 Support both entity_data and bucket_entity_data on axolotl/fish buckets 0e22fdd1e Fix custom direct BlockState being not correctly set in DamageSource f2182ed47 SPIGOT-7659: TropicalFishBucketMeta should use BUCKET_ENTITY_DATA 2a6207fe1 PR-1393: Improve field rename handling and centralize conversion between bukkit and string more c024a5039 SPIGOT-7650: Add DamageSource for EntityDeathEvent and PlayerDeathEvent 741b84480 PR-1390: Improve internal handling of damage sources 0364df4e1 SPIGOT-7657: Error when loading angry entities
241 lines
8.3 KiB
Diff
241 lines
8.3 KiB
Diff
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
From: Jake Potrebic <jake.m.potrebic@gmail.com>
|
|
Date: Sat, 21 May 2022 20:59:56 -0700
|
|
Subject: [PATCH] Add BlockLockCheckEvent
|
|
|
|
|
|
diff --git a/src/main/java/io/papermc/paper/block/LockableTileState.java b/src/main/java/io/papermc/paper/block/LockableTileState.java
|
|
new file mode 100644
|
|
index 0000000000000000000000000000000000000000..f309961e0e96b6baacc4fe6d80dabd6c7c5d2e1d
|
|
--- /dev/null
|
|
+++ b/src/main/java/io/papermc/paper/block/LockableTileState.java
|
|
@@ -0,0 +1,11 @@
|
|
+package io.papermc.paper.block;
|
|
+
|
|
+import org.bukkit.Nameable;
|
|
+import org.bukkit.block.Lockable;
|
|
+import org.bukkit.block.TileState;
|
|
+
|
|
+/**
|
|
+ * Interface for tile entities that are lockable.
|
|
+ */
|
|
+public interface LockableTileState extends TileState, Lockable, Nameable {
|
|
+}
|
|
diff --git a/src/main/java/io/papermc/paper/event/block/BlockLockCheckEvent.java b/src/main/java/io/papermc/paper/event/block/BlockLockCheckEvent.java
|
|
new file mode 100644
|
|
index 0000000000000000000000000000000000000000..4c335212f6a5ad4a304c3c0f66ffd7e0974c0572
|
|
--- /dev/null
|
|
+++ b/src/main/java/io/papermc/paper/event/block/BlockLockCheckEvent.java
|
|
@@ -0,0 +1,186 @@
|
|
+package io.papermc.paper.event.block;
|
|
+
|
|
+import com.google.common.base.Preconditions;
|
|
+import io.papermc.paper.block.LockableTileState;
|
|
+import net.kyori.adventure.sound.Sound;
|
|
+import net.kyori.adventure.text.Component;
|
|
+import org.bukkit.block.Block;
|
|
+import org.bukkit.entity.Player;
|
|
+import org.bukkit.event.HandlerList;
|
|
+import org.bukkit.event.block.BlockEvent;
|
|
+import org.bukkit.inventory.ItemStack;
|
|
+import org.jetbrains.annotations.ApiStatus;
|
|
+import org.jetbrains.annotations.NotNull;
|
|
+import org.jetbrains.annotations.Nullable;
|
|
+
|
|
+import java.util.Objects;
|
|
+
|
|
+/**
|
|
+ * Called when the server tries to check the lock on a lockable block entity.
|
|
+ * <br>
|
|
+ * See {@link #setResult(Result)} to change behavior
|
|
+ */
|
|
+public class BlockLockCheckEvent extends BlockEvent {
|
|
+
|
|
+ private static final HandlerList HANDLER_LIST = new HandlerList();
|
|
+
|
|
+ private final LockableTileState state;
|
|
+ private final Player player;
|
|
+ private Component lockedMessage;
|
|
+ private Sound lockedSound;
|
|
+ private ItemStack itemStack;
|
|
+ private Result result = Result.DEFAULT;
|
|
+
|
|
+ @ApiStatus.Internal
|
|
+ public BlockLockCheckEvent(final @NotNull Block block, final @NotNull LockableTileState state, final @NotNull Player player, final @NotNull Component lockedMessage, final @NotNull Sound lockedSound) {
|
|
+ super(block);
|
|
+ this.state = state;
|
|
+ this.player = player;
|
|
+ this.lockedMessage = lockedMessage;
|
|
+ this.lockedSound = lockedSound;
|
|
+ }
|
|
+
|
|
+ /**
|
|
+ * Gets the snapshot {@link LockableTileState} of the block entity
|
|
+ * whose lock is being checked.
|
|
+ *
|
|
+ * @return the snapshot block state.
|
|
+ */
|
|
+ public @NotNull LockableTileState getBlockState() {
|
|
+ return this.state;
|
|
+ }
|
|
+
|
|
+ /**
|
|
+ * Get the player involved this lock check.
|
|
+ *
|
|
+ * @return the player
|
|
+ */
|
|
+ public @NotNull Player getPlayer() {
|
|
+ return this.player;
|
|
+ }
|
|
+
|
|
+ /**
|
|
+ * Gets the itemstack that will be used as the key itemstack. Initially
|
|
+ * this will be the item in the player's main hand but an override can be set
|
|
+ * with {@link #setKeyItem(ItemStack)}. Use {@link #isUsingCustomKeyItemStack()}
|
|
+ * to check if a custom key stack has been set.
|
|
+ *
|
|
+ * @return the item being used as the key item
|
|
+ * @see #isUsingCustomKeyItemStack()
|
|
+ */
|
|
+ public @NotNull ItemStack getKeyItem() {
|
|
+ return Objects.requireNonNullElseGet(this.itemStack, this.player.getInventory()::getItemInMainHand);
|
|
+ }
|
|
+
|
|
+ /**
|
|
+ * Sets the itemstack that will be used as the key item.
|
|
+ *
|
|
+ * @param stack the stack to use as a key
|
|
+ * @see #resetKeyItem() to clear a custom key item
|
|
+ */
|
|
+ public void setKeyItem(@NotNull ItemStack stack) {
|
|
+ Preconditions.checkArgument(stack != null, "stack cannot be null");
|
|
+ this.itemStack = stack;
|
|
+ }
|
|
+
|
|
+ /**
|
|
+ * Reset the key stack to the default (the player's main hand).
|
|
+ */
|
|
+ public void resetKeyItem() {
|
|
+ this.itemStack = null;
|
|
+ }
|
|
+
|
|
+ /**
|
|
+ * Checks if a custom key stack has been set.
|
|
+ *
|
|
+ * @return {@code true} if a custom key itemstack has been set
|
|
+ */
|
|
+ public boolean isUsingCustomKeyItemStack() {
|
|
+ return this.itemStack != null;
|
|
+ }
|
|
+
|
|
+ /**
|
|
+ * Gets the result of this event.
|
|
+ *
|
|
+ * @return the result
|
|
+ * @see #setResult(Result)
|
|
+ */
|
|
+ public @NotNull Result getResult() {
|
|
+ return this.result;
|
|
+ }
|
|
+
|
|
+ /**
|
|
+ * Gets the result of this event. {@link Result#DEFAULT} is the default
|
|
+ * allowing the vanilla logic to check the lock of this block. Set to {@link Result#ALLOW}
|
|
+ * or {@link Result#DENY} to override that behavior.
|
|
+ * <p>
|
|
+ * Setting this to {@link Result#ALLOW} bypasses the spectator check.
|
|
+ *
|
|
+ * @param result the result of this event
|
|
+ */
|
|
+ public void setResult(@NotNull Result result) {
|
|
+ this.result = result;
|
|
+ }
|
|
+
|
|
+ /**
|
|
+ * Shorthand method to set the {@link #getResult()} to {@link Result#DENY},
|
|
+ * the locked message and locked sound.
|
|
+ *
|
|
+ * @param lockedMessage the message to show if locked (or {@code null} for none)
|
|
+ * @param lockedSound the sound to play if locked (or {@code null} for none)
|
|
+ */
|
|
+ public void denyWithMessageAndSound(@Nullable Component lockedMessage, @Nullable Sound lockedSound) {
|
|
+ this.result = Result.DENY;
|
|
+ this.lockedMessage = lockedMessage;
|
|
+ this.lockedSound = lockedSound;
|
|
+ }
|
|
+
|
|
+ /**
|
|
+ * Gets the locked message that will be sent if the
|
|
+ * player cannot open the block.
|
|
+ *
|
|
+ * @return the locked message (or {@code null} if none)
|
|
+ */
|
|
+ public @Nullable Component getLockedMessage() {
|
|
+ return this.lockedMessage;
|
|
+ }
|
|
+
|
|
+ /**
|
|
+ * Sets the locked message that will be sent if the
|
|
+ * player cannot open the block.
|
|
+ *
|
|
+ * @param lockedMessage the locked message (or {@code null} for none)
|
|
+ */
|
|
+ public void setLockedMessage(@Nullable Component lockedMessage) {
|
|
+ this.lockedMessage = lockedMessage;
|
|
+ }
|
|
+
|
|
+ /**
|
|
+ * Gets the locked sound that will play if the
|
|
+ * player cannot open the block.
|
|
+ *
|
|
+ * @return the locked sound (or {@code null} if none)
|
|
+ */
|
|
+ public @Nullable Sound getLockedSound() {
|
|
+ return this.lockedSound;
|
|
+ }
|
|
+
|
|
+ /**
|
|
+ * Sets the locked sound that will play if the
|
|
+ * player cannot open the block.
|
|
+ *
|
|
+ * @param lockedSound the locked sound (or {@code null} for none)
|
|
+ */
|
|
+ public void setLockedSound(@Nullable Sound lockedSound) {
|
|
+ this.lockedSound = lockedSound;
|
|
+ }
|
|
+
|
|
+ @Override
|
|
+ public @NotNull HandlerList getHandlers() {
|
|
+ return HANDLER_LIST;
|
|
+ }
|
|
+
|
|
+ public static @NotNull HandlerList getHandlerList() {
|
|
+ return HANDLER_LIST;
|
|
+ }
|
|
+}
|
|
diff --git a/src/main/java/org/bukkit/block/Beacon.java b/src/main/java/org/bukkit/block/Beacon.java
|
|
index 7d212c409035ccb8b22d4ffc322b4a1aea367627..79c04b840adb768f7a38e95a82f79287f42681f5 100644
|
|
--- a/src/main/java/org/bukkit/block/Beacon.java
|
|
+++ b/src/main/java/org/bukkit/block/Beacon.java
|
|
@@ -11,7 +11,7 @@ import org.jetbrains.annotations.Nullable;
|
|
/**
|
|
* Represents a captured state of a beacon.
|
|
*/
|
|
-public interface Beacon extends TileState, Lockable, Nameable {
|
|
+public interface Beacon extends io.papermc.paper.block.LockableTileState { // Paper
|
|
|
|
/**
|
|
* Returns the list of players within the beacon's range of effect.
|
|
diff --git a/src/main/java/org/bukkit/block/Container.java b/src/main/java/org/bukkit/block/Container.java
|
|
index bc06199f0a1cc43e0bdfd5b11fa170badd46e180..a67ee0cb0cd2cbab8dab375e2fe44168c250bcb7 100644
|
|
--- a/src/main/java/org/bukkit/block/Container.java
|
|
+++ b/src/main/java/org/bukkit/block/Container.java
|
|
@@ -8,7 +8,7 @@ import org.jetbrains.annotations.NotNull;
|
|
/**
|
|
* Represents a captured state of a container block.
|
|
*/
|
|
-public interface Container extends TileState, BlockInventoryHolder, Lockable, Nameable {
|
|
+public interface Container extends io.papermc.paper.block.LockableTileState, BlockInventoryHolder { // Paper
|
|
|
|
/**
|
|
* Gets the inventory of the block represented by this block state.
|