Don't allow setting build + passthrough in the flag helper.

It masks the warning message and too many people mess this up.
EngineHub · Jun 22, 2019 · 6d25513 · 6d25513
1 parent 96b6174
commit 6d25513
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/worldguard-core/src/main/java/com/sk89q/worldguard/commands/region/FlagHelperBox.java b/worldguard-core/src/main/java/com/sk89q/worldguard/commands/region/FlagHelperBox.java
@@ -200,7 +200,7 @@ private <V> void appendValueChoices(TextComponent.Builder builder, Flag<V> flag,
             if (maySet) {
                 if (isExplicitSet) {
                     hoverTexts.add(TextComponent.of("Click to unset", TextColor.GOLD));
-                } else {
+                } else if (flag != Flags.BUILD && flag != Flags.PASSTHROUGH) {
                     hoverTexts.add(TextComponent.of("Click to set", TextColor.GOLD));
                 }
             }
@@ -221,7 +221,7 @@ private <V> void appendValueChoices(TextComponent.Builder builder, Flag<V> flag,
                         HoverEvent.of(HoverEvent.Action.SHOW_TEXT, hoverBuilder.build()));
             }
 
-            if (maySet) {
+            if (maySet && (isExplicitSet || flag != Flags.BUILD && flag != Flags.PASSTHROUGH)) {
                 builder.append(choiceComponent.clickEvent(ClickEvent.of(ClickEvent.Action.RUN_COMMAND,
                         makeCommand(flag, isExplicitSet ? "" : choice))));
             } else {