fix(sun-mgmt+salesforce): two coding issues that caused perl warnings (…

…#5369) Refs: CTOR-1197
centreon · Dec 27, 2024 · b8927e6 · b8927e6
1 parent f1ab4d9
commit b8927e6
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 25 deletions.
diff --git a/src/apps/salesforce/restapi/custom/api.pm b/src/apps/salesforce/restapi/custom/api.pm
@@ -112,7 +112,7 @@ sub request_api {
         $self->{output}->option_exit();
     }
     if ($self->{http}->get_code() != 200) {
-        $self->{output}->add_option_msg(short_msg => "Connection issue: " . $decoded->{message});
+        $self->{output}->add_option_msg(short_msg => "Connection issue: " . defined($decoded->{message}) ? $decoded->{message}  : '');
         $self->{output}->option_exit();
     }
 
@@ -125,27 +125,27 @@ __END__
 
 =head1 NAME
 
-SFDC API boilerplate
+Monitor SFDC API boilerplate
 
 =head1 SYNOPSIS
 
-Get informations from SFDC API
+Get information from SFDC API
 
 =head1 REST API OPTIONS
 
 =over 8
 
 =item B<--hostname>
 
-Set hostname to query (default: 'api.status.salesforce.com')
+Define the hostname to query (default: C<api.status.salesforce.com>)
 
 =item B<--timeout>
 
-Set HTTP timeout in seconds (default: '10').
+Define the HTTP timeout in seconds (default: '10').
 
 =item B<--api-version>
 
-API version (default: 'v1').
+Define the API version (default: 'v1').
 
 =back
 

diff --git a/src/hardware/server/sun/mgmt_cards/mode/showfaults.pm b/src/hardware/server/sun/mgmt_cards/mode/showfaults.pm
@@ -64,8 +64,8 @@ sub check_options {
         command => $self->{option_results}->{command_plink}
     );
 
-    $self->{option_results}->{command} = 'plink'
-        if (!defined($self->{option_results}->{command}) || $self->{option_results}->{command} eq '');
+    $self->{option_results}->{command_plink} = 'plink'
+        if (!defined($self->{option_results}->{command_plink}) || $self->{option_results}->{command_plink} eq '');
 }
 
 sub run {
@@ -132,25 +132,25 @@ __END__
 
 =head1 MODE
 
-Check Sun 'T1xxx', 'T2xxx' Hardware (through ALOM4v).
+Check Sun C<T1xxx>, C<T2xxx> management cards hardware (using C<ALOM4v>).
 
 =over 8
 
 =item B<--hostname>
 
-Hostname to query.
+Define the hostname to query.
 
 =item B<--username>
 
-ssh username.
+Define the ssh username.
 
 =item B<--password>
 
-ssh password.
+Define the ssh password.
 
 =item B<--command-plink>
 
-Plink command (default: plink). Use to set a path.
+Define the C<plink> command (default: C<plink>).
 
 =item B<--timeout>
 

diff --git a/src/hardware/server/sun/mgmt_cards/mode/showfaulty.pm b/src/hardware/server/sun/mgmt_cards/mode/showfaulty.pm
@@ -71,8 +71,8 @@ sub check_options {
         command => $self->{option_results}->{command_plink}
     );
 
-    $self->{option_results}->{command} = 'plink'
-        if (!defined($self->{option_results}->{command}) || $self->{option_results}->{command} eq '');
+    $self->{option_results}->{command_plink} = 'plink'
+        if (!defined($self->{option_results}->{command_plink}) || $self->{option_results}->{command_plink} eq '');
 }
 
 sub run {
@@ -185,29 +185,29 @@ __END__
 
 =head1 MODE
 
-Check Sun 'T3-x', 'T4-x' and 'T5xxx' Hardware (through ILOM).
+Check Sun C<T3-x>, C<T4-x> and C<T5xxx> management cards hardware (using C<ILOM>).
 
 =over 8
 
 =item B<--hostname>
 
-Hostname to query.
+Define the hostname to query.
 
 =item B<--username>
 
-ssh username.
+Define the ssh username.
 
 =item B<--password>
 
-ssh password.
+Define the ssh password.
 
-=item B<--memory>
+=item B<--command-plink>
 
-Returns new errors (retention file is used by the following option).
+Define the C<plink> command (default: C<plink>).
 
-=item B<--command-plink>
+=item B<--memory>
 
-Plink command (default: plink). Use to set a path.
+Returns new errors (retention file is used by the following option).
 
 =item B<--timeout>
 

diff --git a/tests/resources/spellcheck/stopwords.txt b/tests/resources/spellcheck/stopwords.txt
@@ -49,11 +49,11 @@ hashicorpvault
 HPE
 Huawei
 ifAlias
-includeAllDisks
 ifDesc
 ifName
 IMEI
 in-bcast
+includeAllDisks
 in-crc
 in-fcserror
 in-mcast
@@ -128,6 +128,7 @@ RRDCached
 Sansymphony
 SAS
 scenarii
+SFDC
 sfp.temperature
 SkyHigh
 SNMP
@@ -144,8 +145,8 @@ Teldat
 timeframe
 TiMOS
 tmnxSasAlarmInputDescription
-total-oper-up
 total-oper-down
+total-oper-up
 TrendMicro
 tsdb-version
 UCD