Neue Beta Test Runde für alle MAX Module

[adn77]

I believe there may be a mistake in line 2003 of the 10_MAX.pm file. I think that instead...
elsif (AttrNum($name, 'keepAuto', 0)  && (ReadingsVal($hash, 'mode', 'auto') eq 'auto')) {

should be
elsif (AttrNum($name, 'keepAuto', 0)  && (ReadingsVal($name, 'mode', 'auto') eq 'auto')) {

I guess you're right about the correction, but fixing that leads to some unwanted behavior. I eventually came up with this:

Code Auswählen Erweitern

--- FHEM/10_MAX.pm      2026-03-16 09:11:35.998161250 +0100
+++ FHEM/10_MAX.pm      2026-04-12 18:14:04.885823595 +0200
@@ -1986,25 +1986,27 @@

        $ctrlmode = 0; # auto
     } # auto
-
-    if ($args[0] eq 'boost') {
+    elsif ($args[0] eq 'boost') {
        return Log3Return($name, 'set_desiredTemp : too many parameters for boost') if (@args > 1);
        $temperature = 0;
        $ctrlmode = 3;
-       # TODO: auto mode with temperature is also possible
     }
-
-    if ($args[0] eq 'manual') {
+    elsif ($args[0] eq 'manual') {
        # User explicitly asked for manual mode
        $ctrlmode = 1; #manual, possibly overwriting keepAuto
        shift @args;
        return Log3Return($name, 'set_desiredTemp : not enough parameters after desiredTemperature manual') if (!@args);
     }
-    elsif (AttrNum($name, 'keepAuto', 0)  && (ReadingsVal($name, 'mode', 'auto') eq 'auto')) {
+    elsif (AttrNum($name, 'keepAuto', 0)) {
        # User did not ask for any mode explicitly, but has keepAuto
-       Log3($hash, 5, "$name, SetTemperature: keepAuto and mode auto = staying in auto mode");
+       Log3($name, 5, "$name, SetTemperature: keepAuto and mode auto = staying in auto mode");
        $ctrlmode = 0; # auto
     }
+    else {
+       if (ReadingsVal($name, 'mode', 'auto') eq 'auto') { $ctrlmode = 0; }
+       else { $ctrlmode = 1; }
+       Log3($name, 5, "$name, SetTemperature: no mode explcitly given = staying in current mode");
+    }

     $temperature = MAX_ReadingsVal($hash, 'ecoTemperature') if ($args[0] eq 'eco');



[adn77]

Das mit dem keepAuto klingt zwar komisch, ist aber richtig !

Hallo Wzut,

danke für die Info.
Habe es aber nicht so ganz verstanden.
Ist es "richtig", weil es aktuell im Code so hinterlegt ist und korrekt so ausgeführt wird?
Aber eigentlich ist durch das Code-Aufräumen nun dieses Verhalten so?

In diesem Fall, wirst Du es auf den Stand "boost funktioniert bei keepAuto=1" ändern?
Danke für eine kurze Rückmeldung.

Viele Grüße,
Andreas

Das "else" wird erst mit meinem Patch oben wieder repariert.

Soweit ich die alten Posts aus 2014 zu dem Thema richtig deute, wird die Boost Funktion außerdem mit einem Broadcast Flag gesendet, was nur funktioniert, wenn die GroupId <> 0 ist.
Sofern man also nie die originale MAX Lösung genutzt hat, muss man manuell unterschiedliche GroupIds für alle Geräte eines Raumes setzen (also innerhalb eines Raumes immer die gleiche ).

[Wzut]

a. das die Group ID wichtig ist habe ich schon mehrfach  geschrieben.

b. das ReadingsVal($hash ein Tippfehler ist und es eigentlich ReadingsVal($name sein müsste da hat asdfasdfasdfasdfasdf schon recht.

c. OK, das Problem von 2022 ist das der crtlmode überschrieben wird wenn keepAuto = 1 ist, habe ich soweit verstanden, THX fürs finden & posten

Den Patch werde ich in dieser Form so nicht einsetzen, da ich diese verschachtelten elsif/else Konstrukte wenn möglich meide wie der Teufel das Weihwasser. Ich lasse mir etwas einfallen das den gleichen Zweck erfüllt..