From c10cda979b59b0e4ce2c1264c28c66ff95c0aa10 Mon Sep 17 00:00:00 2001 From: mcmonkey4eva Date: Fri, 18 Aug 2017 01:53:46 -0700 Subject: [PATCH] workaround a spigot dumb-dumb, thanks Patrons! --- .../events/player/PlayerClicksBlockScriptEvent.java | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/plugin/src/main/java/net/aufdemrand/denizen/events/player/PlayerClicksBlockScriptEvent.java b/plugin/src/main/java/net/aufdemrand/denizen/events/player/PlayerClicksBlockScriptEvent.java index 0d9cf90cae..4364fe3ee5 100644 --- a/plugin/src/main/java/net/aufdemrand/denizen/events/player/PlayerClicksBlockScriptEvent.java +++ b/plugin/src/main/java/net/aufdemrand/denizen/events/player/PlayerClicksBlockScriptEvent.java @@ -13,6 +13,7 @@ import net.aufdemrand.denizencore.scripts.containers.ScriptContainer; import net.aufdemrand.denizencore.utilities.CoreUtilities; import org.bukkit.Bukkit; +import org.bukkit.event.Event; import org.bukkit.event.EventHandler; import org.bukkit.event.Listener; import org.bukkit.event.block.Action; @@ -62,13 +63,10 @@ private boolean couldMatchIn(String lower) { if (index == -1) return true; String in = CoreUtilities.getXthArg(index + 1, lower); - if (in.equals("notable") - || dWorld.matches(in) - || dCuboid.matches(in) - || dEllipsoid.matches(in)) { - return true; + if (in.equals("inventory")) { + return false; } - return false; + return true; } private boolean runUsingCheck(ScriptContainer scriptContainer, String s, String lower) { @@ -208,7 +206,7 @@ public void playerClicksBlock(PlayerInteractEvent event) { location = event.hasBlock() ? new dLocation(event.getClickedBlock().getLocation()) : null; relative = event.hasBlock() ? new dLocation(event.getClickedBlock().getRelative(event.getBlockFace()).getLocation()) : null; click_type = new Element(event.getAction().name()); - cancelled = event.isCancelled(); + cancelled = event.isCancelled() && event.useItemInHand() == Event.Result.DENY; // Spigot is dumb! this.event = event; fire(); event.setCancelled(cancelled);