[patch] 10_MYSENSORS_DEVICE.pm - Fix für falsche Bootloader-Abfrage beim Update

[pldemon]

Hallo,

danke für das Modul! Top! Die Anfrage des eingesetzten Bootloaders beim Reboot-Kommando wird bei MYSBootloader aber so nicht auslösen, da der Hash (zumindest bei mir) leer ist. Bei Optiboot greift die zweite Kondition, weshalb das Konstrukt funktioniert.

Habe das Modul für meine Bedürfnisse folgendermaßen angepasst:

Code: [Auswählen]

--- 10_MYSENSORS_DEVICE.orig    2020-11-28 11:54:56.261163746 +0100
+++ 10_MYSENSORS_DEVICE.pm      2020-11-28 11:58:06.755827232 +0100
@@ -282,14 +282,18 @@
       
       if ($command eq "reboot") {
         my $blVersion = ReadingsVal($name, "BL_VERSION", "");
-        defined($hash->{OTA_BL_Type}) or $blVersion eq "3.0"
-          ? return sendClientMessage($hash,
+        my $blType = AttrVal($name, "OTA_BL_Type", "");
+
+        return sendClientMessage($hash,
                                      childId => 255,
                                      cmd => C_INTERNAL,
                                      ack => 0,
                                      subType => I_REBOOT
-                                     )
-            : return;
+                                     )
+           if (($blType eq "MYSBootloader") ||
+               ($blType eq "Optiboot" and $blVersion eq "3.0"));
+
+       return;
       }
       
       if ($command eq "clear") {

Gruß,
demon

[Beta-User]

Danke für die Rückmeldung und den Hinweis!

Hab's leicht verändert eingecheckt, wäre nett, wenn du die aktuelle Version aus dem svn holen könntest und kurz checken, ob alles so funktioniert wie gewünscht . (oder eben ab morgen früh via update; ich habe derzeit keine Test-Nodes, insbesondere keine mit allen möglichen Bootloadern, es müßte aber eigentlich funktionieren...)

[pldemon]

Hi,

so wird es nicht funktionieren. Die Anfrage benötigt Klammern 
https://svn.fhem.de/trac/changeset/23254/trunk/fhem/FHEM/10_MYSENSORS_DEVICE.pm

So wird's...

Code: [Auswählen]

(AttrVal($name, "OTA_BL_Type", 0) or ReadingsVal($name, "BL_VERSION", 0))

Hab im Übrigen gestern auch FOTA getestet und es funktionierte tadellos. Saubere Arbeit!

Gruß,
demon

[pldemon]

Mit dem aktuellen Stand aus dem SVN kommt folgende Fehlermeldung beim reboot:

Code: [Auswählen]

reboot not defined: no mapping for reading reboot

Gruß,
demon

[Beta-User]

Thx für's testen, update mit der Klammer ist im svn...

(Der OTA-Basiscode stammt btw. - wie der überwiegende Teil des gesamten Codes - von anderen...)