]
George Gastaldi closed FORGE-2296.
----------------------------------
Fix Version/s: 2.15.3.Final
Resolution: Done
Forge - Shell in CLI interactive mode, show * for required option and
? for optional
------------------------------------------------------------------------------------
Key: FORGE-2296
URL:
https://issues.jboss.org/browse/FORGE-2296
Project: Forge
Issue Type: Feature Request
Components: UI - Shell
Affects Versions: 2.15.2.Final
Reporter: Claus Ibsen
Assignee: George Gastaldi
Fix For: 2.15.3.Final
When using forge commands in interactive mode, the CLI shows options line by line. Its a
bit confusing which are mandatory ,and which are optional.
Today it shows ? for all options, eg
{code}
Press <ENTER> to confirm, or <CTRL>+C to cancel.
? componentName (Name of component type to add): [0-179] 165
? file (The XML file to use (either Spring or Blueprint)) [META-INF/spring/foo.xml]:
? timerName (The name of the timer):
***ERROR*** timerName must be specified.
{code}
Notice that both file and timerName has a starting ? on the CLI. But both options is
required. So its IMHO better to show a * instead to indicate required.
Here is an example where I am a bit further down in the CLI for the same command
{code}
Press <ENTER> to confirm, or <CTRL>+C to cancel.
? componentName (Name of component type to add): [0-179] 165
? file (The XML file to use (either Spring or Blueprint)) [META-INF/spring/foo.xml]:
? timerName (The name of the timer): foo
? time (A java.util.Date the first event should be generated. If using the URI the
pattern expected is: yyyy-MM-dd HH:mm:ss or yyyy-MM-dd'T'HH:mm:ss.):
? period (If greater than 0 generate periodic events every period milliseconds. The
default value is 1000.) [1000]: 2000
? delay (The number of milliseconds to wait before the first event is generated. Should
not be used in conjunction with the time option. The default value is 1000.) [1000]:
{code}
The time, period, and delay option is optional. But some of them have a default value
which is shown nicely.
But again this ticket is about changing the ? to a * for the required options.