Only log sign base color changes when we log SIGNTEXT for this world

LogBlock · May 27, 2019 · 424ef3b · 424ef3b
1 parent 4fda020
commit 424ef3b
Showing 1 changed file with 12 additions and 10 deletions.
diff --git 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 void onPlayerInteract(PlayerInteractEvent event) {
                 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);
+                                }
                             }
                         }
                     }