Skip to content

Commit

Permalink
"zhmc ldap delete": fails with TypeError #460
Browse files Browse the repository at this point in the history
Changes made in file ../zhmccli/_cmd_ldap_server__definations.py
_ldadef_delete() function  was updated
parameters name LDAPSD updated instead of ldapdef

Signed-off-by: Samir Gorai <samirgorai17@gmail.com>
  • Loading branch information
samirgorai authored and andy-maier committed Oct 13, 2023
1 parent 6f13c1c commit 221f523
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions zhmccli/_cmd_ldap_server_definition.py
Expand Up @@ -68,7 +68,7 @@ def ldapdef_list(cmd_ctx, **options):
@ldapdef_group.command('show', options_metavar=COMMAND_OPTIONS_METAVAR)
@click.argument('LDAPSD', type=str, metavar='LDAP_SERVER_DEFINITION')
@click.pass_obj
def ldapdef_show(cmd_ctx, ldapdef):
def ldapdef_show(cmd_ctx, ldapsd):
"""
Show the details of an LDAP server definition.
Expand All @@ -81,7 +81,7 @@ def ldapdef_show(cmd_ctx, ldapdef):
general options (see 'zhmc --help') can also be specified right after the
'zhmc' command name.
"""
cmd_ctx.execute_cmd(lambda: cmd_ldapdef_show(cmd_ctx, ldapdef))
cmd_ctx.execute_cmd(lambda: cmd_ldapdef_show(cmd_ctx, ldapsd))


@ldapdef_group.command('create', options_metavar=COMMAND_OPTIONS_METAVAR)
Expand Down Expand Up @@ -189,7 +189,7 @@ def ldapdef_create(cmd_ctx, **options):
'user\'s directory entry in a subtree. Only for location method '
'"subtree".')
@click.pass_obj
def ldapdef_update(cmd_ctx, ldapdef, **options):
def ldapdef_update(cmd_ctx, ldapsd, **options):
"""
Update the properties of an LDAP server definition.
Expand All @@ -202,7 +202,7 @@ def ldapdef_update(cmd_ctx, ldapdef, **options):
general options (see 'zhmc --help') can also be specified right after the
'zhmc' command name.
"""
cmd_ctx.execute_cmd(lambda: cmd_ldapdef_update(cmd_ctx, ldapdef, options))
cmd_ctx.execute_cmd(lambda: cmd_ldapdef_update(cmd_ctx, ldapsd, options))


@ldapdef_group.command('delete', options_metavar=COMMAND_OPTIONS_METAVAR)
Expand All @@ -214,15 +214,15 @@ def ldapdef_update(cmd_ctx, ldapdef, **options):
prompt='Are you sure you want to delete this LDAP server '
'definition ?')
@click.pass_obj
def ldapdef_delete(cmd_ctx, ldapdef):
def ldapdef_delete(cmd_ctx, ldapsd):
"""
Delete a user-defined LDAP server definition.
In addition to the command-specific options shown in this help text, the
general options (see 'zhmc --help') can also be specified right after the
'zhmc' command name.
"""
cmd_ctx.execute_cmd(lambda: cmd_ldapdef_delete(cmd_ctx, ldapdef))
cmd_ctx.execute_cmd(lambda: cmd_ldapdef_delete(cmd_ctx, ldapsd))


def cmd_ldapdef_list(cmd_ctx, options):
Expand Down

0 comments on commit 221f523

Please sign in to comment.