Merge pull request #1748 from caberos/issue1747

Update Help message for commands that take in multiple arguments
softlayer · Sep 8, 2022 · 2c86f68 · 2c86f68
2 parents 735112e + b356690
commit 2c86f68
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 9 deletions.
diff --git a/SoftLayer/CLI/block/access/authorize.py b/SoftLayer/CLI/block/access/authorize.py
@@ -6,17 +6,19 @@
 from SoftLayer.CLI import environment
 from SoftLayer.CLI import exceptions
 
+MULTIPLE = '(Multiple allowed)'
+
 
 @click.command(cls=SoftLayer.CLI.command.SLCommand, )
 @click.argument('volume_id')
 @click.option('--hardware-id', '-h', multiple=True,
-              help='The id of one SoftLayer_Hardware to authorize')
+              help='The id of one SoftLayer_Hardware to authorize ' + MULTIPLE)
 @click.option('--virtual-id', '-v', multiple=True,
-              help='The id of one SoftLayer_Virtual_Guest to authorize')
+              help='The id of one SoftLayer_Virtual_Guest to authorize ' + MULTIPLE)
 @click.option('--ip-address-id', '-i', multiple=True,
-              help='The id of one SoftLayer_Network_Subnet_IpAddress to authorize')
+              help='The id of one SoftLayer_Network_Subnet_IpAddress to authorize ' + MULTIPLE)
 @click.option('--ip-address', multiple=True,
-              help='An IP address to authorize')
+              help='An IP address to authorize ' + MULTIPLE)
 @environment.pass_env
 def cli(env, volume_id, hardware_id, virtual_id, ip_address_id, ip_address):
     """Authorizes hosts to access a given volume"""

diff --git a/SoftLayer/CLI/file/access/authorize.py b/SoftLayer/CLI/file/access/authorize.py
@@ -6,20 +6,22 @@
 from SoftLayer.CLI import environment
 from SoftLayer.CLI import exceptions
 
+MULTIPLE = '(Multiple allowed)'
+
 
 @click.command(cls=SoftLayer.CLI.command.SLCommand, )
 @click.argument('volume_id')
 @click.option('--hardware-id', '-h', multiple=True,
-              help='The id of one SoftLayer_Hardware to authorize')
+              help='The id of one SoftLayer_Hardware to authorize ' + MULTIPLE)
 @click.option('--virtual-id', '-v', multiple=True,
-              help='The id of one SoftLayer_Virtual_Guest to authorize')
+              help='The id of one SoftLayer_Virtual_Guest to authorize ' + MULTIPLE)
 @click.option('--ip-address-id', '-i', multiple=True,
               help='The id of one SoftLayer_Network_Subnet_IpAddress'
-              ' to authorize')
+                   ' to authorize ' + MULTIPLE)
 @click.option('--ip-address', multiple=True,
-              help='An IP address to authorize')
+              help='An IP address to authorize ' + MULTIPLE)
 @click.option('--subnet-id', '-s', multiple=True,
-              help='The id of one SoftLayer_Network_Subnet to authorize')
+              help='The id of one SoftLayer_Network_Subnet to authorize ' + MULTIPLE)
 @environment.pass_env
 def cli(env, volume_id, hardware_id, virtual_id, ip_address_id,
         ip_address, subnet_id):