Skip to content

Commit

Permalink
remkop#441: Renamed showParamSyntax -> hideParamSyntax
Browse files Browse the repository at this point in the history
  • Loading branch information
@bbottema
bbottema committed Aug 18, 2018
1 parent 10b33a3 commit 48b23b3
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 10 deletions.
14 changes: 7 additions & 7 deletions src/main/java/picocli/CommandLine.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -3941,7 +3941,7 @@ public abstract static class ArgSpec {
// help-related fields
private final boolean hidden;
private final String paramLabel;
private final boolean showParamSyntax;
private final boolean hideParamSyntax;
private final String[] description;
private final Help.Visibility showDefaultValue;

Expand Down Expand Up @@ -3970,7 +3970,7 @@ private <T extends Builder<T>> ArgSpec(Builder<T> builder) {
description = builder.description == null ? new String[0] : builder.description;
splitRegex = builder.splitRegex == null ? "" : builder.splitRegex;
paramLabel = empty(builder.paramLabel) ? "PARAM" : builder.paramLabel;
showParamSyntax = builder.showParamSyntax;
hideParamSyntax = builder.hideParamSyntax;
converters = builder.converters == null ? new ITypeConverter<?>[0] : builder.converters;
showDefaultValue = builder.showDefaultValue == null ? Help.Visibility.ON_DEMAND : builder.showDefaultValue;
hidden = builder.hidden;
Expand Down Expand Up @@ -4076,7 +4076,7 @@ public String[] renderedDescription() {

/** Indicates whether paramLabel should be processed the regular way or that it should be rendered as-is.
* @see #paramLabel() {@link #paramLabel()} */
public boolean showParamSyntax() { return showParamSyntax; }
public boolean hideParamSyntax() { return hideParamSyntax; }

/** Returns auxiliary type information used when the {@link #type()} is a generic {@code Collection}, {@code Map} or an abstract class.
* @see Option#type() */
Expand Down Expand Up @@ -4245,7 +4245,7 @@ abstract static class Builder<T extends Builder<T>> {
private boolean required;
private boolean interactive;
private String paramLabel;
private boolean showParamSyntax = true;
private boolean hideParamSyntax;
private String splitRegex;
private boolean hidden;
private Class<?> type;
Expand All @@ -4271,7 +4271,7 @@ abstract static class Builder<T extends Builder<T>> {
setter = original.setter;
hidden = original.hidden;
paramLabel = original.paramLabel;
showParamSyntax = original.showParamSyntax;
hideParamSyntax = original.hideParamSyntax;
required = original.required;
interactive = original.interactive;
showDefaultValue = original.showDefaultValue;
Expand Down Expand Up @@ -4369,7 +4369,7 @@ abstract static class Builder<T extends Builder<T>> {

/** Indicates whether paramLabel should be processed the regular way or that it should be rendered as-is.
* @see #paramLabel() {@link #paramLabel()} */
public T showParamSyntax(boolean showParamSyntax) { this.showParamSyntax = showParamSyntax; return self(); }
public T hideParamSyntax(boolean hideParamSyntax) { this.hideParamSyntax = hideParamSyntax; return self(); }

/** Sets auxiliary type information, and returns this builder.
* @param types the element type(s) when the {@link #type()} is a generic {@code Collection} or a {@code Map};
Expand Down Expand Up @@ -7715,7 +7715,7 @@ public DefaultParamLabelRenderer(CommandSpec commandSpec) {
public Text renderParameterLabel(ArgSpec argSpec, Ansi ansi, List<IStyle> styles) {
Range capacity = argSpec.isOption() ? argSpec.arity() : ((PositionalParamSpec)argSpec).capacity();
if (capacity.max == 0) { return ansi.new Text(""); }
if (!argSpec.showParamSyntax()) { return ansi.apply(separator() + argSpec.paramLabel(), styles); }
if (argSpec.hideParamSyntax()) { return ansi.apply(separator() + argSpec.paramLabel(), styles); }

Text paramName = ansi.apply(argSpec.paramLabel(), styles);
String split = argSpec.splitRegex();
Expand Down
6 changes: 3 additions & 3 deletions src/test/java/picocli/CommandLineModelTest.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ public void testModelUsageHelp() throws Exception {
spec.addOption(OptionSpec.builder("-h", "--help").usageHelp(true).description("show help and exit").build());
spec.addOption(OptionSpec.builder("-V", "--version").versionHelp(true).description("show help and exit").build());
spec.addOption(OptionSpec.builder("-c", "--count").paramLabel("COUNT").arity("1").type(int.class).description("number of times to execute").build());
spec.addOption(OptionSpec.builder("-f", "--fix").paramLabel("FIXED(BOOLEAN)").arity("1").showParamSyntax(false).required(true).description("run with fixed option").build());
spec.addOption(OptionSpec.builder("-f", "--fix").paramLabel("FIXED(BOOLEAN)").arity("1").hideParamSyntax(true).required(true).description("run with fixed option").build());
CommandLine commandLine = new CommandLine(spec);
String actual = usageString(commandLine, Ansi.OFF);
String expected = String.format("" +
Expand All @@ -84,7 +84,7 @@ public void testModelUsageHelpWithCustomSeparator() throws Exception {
spec.addOption(OptionSpec.builder("-h", "--help").usageHelp(true).description("show help and exit").build());
spec.addOption(OptionSpec.builder("-V", "--version").versionHelp(true).description("show help and exit").build());
spec.addOption(OptionSpec.builder("-c", "--count").paramLabel("COUNT").arity("1").type(int.class).description("number of times to execute").build());
spec.addOption(OptionSpec.builder("-f", "--fix").paramLabel("FIXED(=BOOLEAN)").arity("1").showParamSyntax(false).required(true).description("run with fixed option").build());
spec.addOption(OptionSpec.builder("-f", "--fix").paramLabel("FIXED(=BOOLEAN)").arity("1").hideParamSyntax(true).required(true).description("run with fixed option").build());
CommandLine commandLine = new CommandLine(spec).setSeparator(" ");
String actual = usageString(commandLine, Ansi.OFF);
String expected = String.format("" +
Expand Down Expand Up @@ -517,7 +517,7 @@ public void testPositionalDefaultRequiredIsFalse() throws Exception {
}
@Test
public void testPositionalDefaultFixParamLabelIsFalse() throws Exception {
assertTrue(PositionalParamSpec.builder().build().showParamSyntax());
assertTrue(PositionalParamSpec.builder().build().hideParamSyntax());
}

@Test
Expand Down

0 comments on commit 48b23b3

Please sign in to comment.