fix: display required only if no default value

Author: Nicola Di Falco

spring-shell-core/src/main/resources/org/springframework/shell/component/string-input-default.stg

@@ -15,22 +15,20 @@ info(model) ::= <%
   <if(model.maskedInput)>
     <model.maskedInput>
   <else>
-    <if(model.required)>
-      <("[Required]"); format="style-value">
-    <endif>
     <if(model.defaultValue)>
       <("[Default "); format="style-value"><model.defaultValue; format="style-value"><("]"); format="style-value">
+    <elseif(model.required)>
+      <("[Required]"); format="style-value">
     <endif>
   <endif>
 <else>
   <if(model.input)>
     <model.input>
   <else>
-    <if(model.required)>
-      <("[Required]"); format="style-value">
-    <endif>
     <if(model.defaultValue)>
       <("[Default "); format="style-value"><model.defaultValue; format="style-value"><("]"); format="style-value">
+    <elseif(model.required)>
+      <("[Required]"); format="style-value">
     <endif>
   <endif>
 <endif>

spring-shell-samples/src/main/java/org/springframework/shell/samples/standard/ComponentCommands.java

@@ -39,14 +39,15 @@
 import org.springframework.shell.standard.AbstractShellComponent;
 import org.springframework.shell.standard.ShellComponent;
 import org.springframework.shell.standard.ShellMethod;
+import org.springframework.shell.standard.ShellOption;
 import org.springframework.util.StringUtils;
 
 @ShellComponent
 public class ComponentCommands extends AbstractShellComponent {
 
 	@ShellMethod(key = "component string", value = "String input", group = "Components")
-	public String stringInput(boolean mask, boolean required) {
-		StringInput component = new StringInput(getTerminal(), "Enter value", null);
+	public String stringInput(boolean mask, boolean required, @ShellOption(defaultValue = ShellOption.NULL) String defaultValue) {
+		StringInput component = new StringInput(getTerminal(), "Enter value", defaultValue);
 		component.setResourceLoader(getResourceLoader());
 		component.setTemplateExecutor(getTemplateExecutor());
 		component.setRequired(required);