Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: sdepassio <114986849+sdepassio@users.noreply.github.com>
  • Loading branch information
omercier and sdepassio committed Nov 8, 2024
1 parent 2148490 commit 1250b1e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/apps/vmware/vsphere8/custom/api.pm
Original file line number Diff line number Diff line change
Expand Up @@ -84,11 +84,11 @@ sub check_options {
$self->{output}->add_option_msg(short_msg => "Need to specify --hostname option.");
$self->{output}->option_exit();
}
if (!defined($self->{username}) || $self->{username} eq '') {
if ($self->{username} eq '') {
$self->{output}->add_option_msg(short_msg => "Need to specify --username option.");
$self->{output}->option_exit();
}
if (!defined($self->{password}) || $self->{password} eq '') {
if ($self->{password} eq '') {
$self->{output}->add_option_msg(short_msg => "Need to specify --password option.");
$self->{output}->option_exit();
}
Expand Down

0 comments on commit 1250b1e

Please sign in to comment.