From 424ef3b02bf0af190836def8ad7187783a8500eb Mon Sep 17 00:00:00 2001 From: Brokkonaut Date: Mon, 27 May 2019 23:53:50 +0200 Subject: [PATCH] Only log sign base color changes when we log SIGNTEXT for this world --- .../LogBlock/listeners/InteractLogging.java | 22 ++++++++++--------- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/src/main/java/de/diddiz/LogBlock/listeners/InteractLogging.java b/src/main/java/de/diddiz/LogBlock/listeners/InteractLogging.java index 743c39e..e3dacc1 100644 --- a/src/main/java/de/diddiz/LogBlock/listeners/InteractLogging.java +++ b/src/main/java/de/diddiz/LogBlock/listeners/InteractLogging.java @@ -221,16 +221,18 @@ public class InteractLogging extends LoggingListener { case JUNGLE_WALL_SIGN: case ACACIA_WALL_SIGN: case DARK_OAK_WALL_SIGN: - ItemStack stack = event.getItem(); - if (stack != null && BukkitUtils.isDye(stack.getType())) { - final BlockState before = event.getClickedBlock().getState(); - if (before instanceof Sign) { - DyeColor newColor = BukkitUtils.dyeToDyeColor(stack.getType()); - Sign signBefore = (Sign) before; - if (newColor != null && signBefore.getColor() != newColor) { - final Sign signAfter = (Sign) event.getClickedBlock().getState(); - signAfter.setColor(newColor); - consumer.queueBlockReplace(Actor.actorFromEntity(player), signBefore, signAfter); + if (wcfg.isLogging(Logging.SIGNTEXT) && event.getAction() == Action.RIGHT_CLICK_BLOCK) { + ItemStack stack = event.getItem(); + if (stack != null && BukkitUtils.isDye(stack.getType())) { + final BlockState before = event.getClickedBlock().getState(); + if (before instanceof Sign) { + DyeColor newColor = BukkitUtils.dyeToDyeColor(stack.getType()); + Sign signBefore = (Sign) before; + if (newColor != null && signBefore.getColor() != newColor) { + final Sign signAfter = (Sign) event.getClickedBlock().getState(); + signAfter.setColor(newColor); + consumer.queueBlockReplace(Actor.actorFromEntity(player), signBefore, signAfter); + } } } }