From 93fcd47f4069e2e3a60f0fd44c5a96b1e8712948 Mon Sep 17 00:00:00 2001
From: sidey79 <7968127+sidey79@users.noreply.github.com>
Date: Sat, 14 Aug 2021 10:14:35 +0200
Subject: [PATCH 1/2] SIGNALduino_calcRSSI check input parameter (#1007)

* 02_SIGNALduino_calcRSSI.t:

- Test extended
- Bug reproduced

* 00_SIGNALduino.pm

- SIGNALduino_calcRSSI Check input parameters
---
 FHEM/00_SIGNALduino.pm                          |  6 +++---
 controls_signalduino.txt                        |  2 +-
 t/FHEM/00_SIGNALduino/02_SIGNALduino_calcRSSI.t | 12 +++++++++++-
 3 files changed, 15 insertions(+), 5 deletions(-)

diff --git a/FHEM/00_SIGNALduino.pm b/FHEM/00_SIGNALduino.pm
index 573f74e2a..60c4562c9 100644
--- a/FHEM/00_SIGNALduino.pm
+++ b/FHEM/00_SIGNALduino.pm
@@ -1,4 +1,4 @@
-# $Id: 00_SIGNALduino.pm v3.5.2 2021-08-08 16:56:40Z HomeAutoUser $
+# $Id: 00_SIGNALduino.pm v3.5.2 2021-08-13 20:19:26Z sidey79 $
 #
 # v3.5.2 - https://github.com/RFD-FHEM/RFFHEM/tree/master
 # The module is inspired by the FHEMduino project and modified in serval ways for processing the incoming messages
@@ -39,7 +39,7 @@ use List::Util qw(first);
 
 
 use constant {
-  SDUINO_VERSION                  => '3.5.2+20210808',  # Datum wird automatisch bei jedem pull request aktualisiert
+  SDUINO_VERSION                  => '3.5.2+20210813',  # Datum wird automatisch bei jedem pull request aktualisiert
   SDUINO_INIT_WAIT_XQ             => 1.5,     # wait disable device
   SDUINO_INIT_WAIT                => 2,
   SDUINO_INIT_MAXRETRY            => 3,
@@ -2292,7 +2292,7 @@ sub SIGNALduino_moduleMatch {
 ############################# package main, test exists
 # calculated RSSI and RSSI value and RSSI string (-77,'RSSI = -77')
 sub SIGNALduino_calcRSSI {
-  my $rssi = shift;
+  my $rssi = shift // return ;
   my $rssiStr = '';
   $rssi = ($rssi>=128 ? (($rssi-256)/2-74) : ($rssi/2-74));
   $rssiStr = "RSSI = $rssi";
diff --git a/controls_signalduino.txt b/controls_signalduino.txt
index a80be7da8..d5c93dc2b 100644
--- a/controls_signalduino.txt
+++ b/controls_signalduino.txt
@@ -1,4 +1,4 @@
-UPD 2021-08-08_19:14:43 228763 FHEM/00_SIGNALduino.pm
+UPD 2021-08-13_22:06:50 228769 FHEM/00_SIGNALduino.pm
 UPD 2020-06-15_17:41:39 17876  FHEM/10_FS10.pm
 UPD 2020-05-26_11:51:12 20465  FHEM/10_SD_GT.pm
 UPD 2021-08-11_21:38:30 10096  FHEM/14_BresserTemeo.pm
diff --git a/t/FHEM/00_SIGNALduino/02_SIGNALduino_calcRSSI.t b/t/FHEM/00_SIGNALduino/02_SIGNALduino_calcRSSI.t
index 716055302..45ed1936b 100644
--- a/t/FHEM/00_SIGNALduino/02_SIGNALduino_calcRSSI.t
+++ b/t/FHEM/00_SIGNALduino/02_SIGNALduino_calcRSSI.t
@@ -6,7 +6,7 @@ use Test2::Tools::Compare qw{is};
 our %defs;
 
 InternalTimer(time()+1, sub {
-	plan(2);
+	plan(3);
     my $targetHash = $defs{dummyDuino};
 
 	subtest 'Test SIGNALduino_calcRSSI with fixed values' => sub {
@@ -48,6 +48,16 @@ InternalTimer(time()+1, sub {
 		is($rssi,-36.5,'check return value -36.5 for input '.$signal_parts{rssi});
 		is($rssiStr,'RSSI = -36.5','check return string RSSI = -36.5 for input '.$signal_parts{rssi});
 	};
+
+
+	subtest 'Test SIGNALduino_calcRSSI with undef' => sub {
+		plan(2);
+
+		my ($rssi,$rssiStr)=SIGNALduino_calcRSSI(undef);
+		is($rssi,undef,'check if return is undef');
+		is($rssiStr,undef,'check if return is undef');
+	};
+
 	exit(0);
 
 }, 0);

From 32f966ae1cd3c7bf513ffc1e217c4f6c3721553c Mon Sep 17 00:00:00 2001
From: GitHub Action <action@github.com>
Date: Sat, 14 Aug 2021 08:15:03 +0000
Subject: [PATCH 2/2] Automatic updated controls and CHANGED

---
 CHANGED                  | 10 ++++++++++
 controls_signalduino.txt |  2 +-
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/CHANGED b/CHANGED
index 48a59613f..18584745a 100644
--- a/CHANGED
+++ b/CHANGED
@@ -1,3 +1,13 @@
+2021-08-14 - SIGNALduino_calcRSSI check input parameter (#1007)
+
+* 02_SIGNALduino_calcRSSI.t:
+
+- Test extended
+- Bug reproduced
+
+* 00_SIGNALduino.pm
+
+- SIGNALduino_calcRSSI Check input parameters
 2021-08-13 - Fixes perlcritic return undef in 14_SD_WS (#1006)
 2021-08-11 - Fixes perlcritic return undef; (#979)
 
diff --git a/controls_signalduino.txt b/controls_signalduino.txt
index d5c93dc2b..ce3d5c3ea 100644
--- a/controls_signalduino.txt
+++ b/controls_signalduino.txt
@@ -1,4 +1,4 @@
-UPD 2021-08-13_22:06:50 228769 FHEM/00_SIGNALduino.pm
+UPD 2021-08-14_10:14:35 228769 FHEM/00_SIGNALduino.pm
 UPD 2020-06-15_17:41:39 17876  FHEM/10_FS10.pm
 UPD 2020-05-26_11:51:12 20465  FHEM/10_SD_GT.pm
 UPD 2021-08-11_21:38:30 10096  FHEM/14_BresserTemeo.pm