[patch] fhem.pl - attr <device> disable toggle

[betateilchen]

Code Auswählen Erweitern


Index: fhem.pl
===================================================================
--- fhem.pl (revision 7785)
+++ fhem.pl (working copy)
@@ -2277,6 +2277,10 @@
       }
     }

+    if($attrName eq 'disable' and $a[2] eq 'toggle') {
+       $a[2] = IsDisabled($sdev) ? 0 : 1;
+    }
+
     if($attrName eq "userReadings") {

       my %userReadings;


Vorteil: man kann das disable bei einem device einfach "umschalten" - was die einfachste Möglichkeit bietet, dies auch per fhem-URL

Code Auswählen Erweitern

...?cmd=attr%20deviceName%20disable%20toggle
auszuführen.

Im entsprechenden Modul muss natürlich in der attrList das disable auf 0,1,toggle geändert werden, sofern der Modulautor das Umschalten anbieten möchte.

Grundsätzlich verspreche ich mir davon eine an vielen Stellen einfachere Handhabung des disable-Attributs insbesondere bei at- und notify-devices.

[rudolfkoenig]

Wenn du das auch noch dokumentierst, dann nehem ich den Patch unveraendert.

Btw. Erweiterung im Modul ist nicht notwendig: die Liste (0,1,toggle) ist nur fuers Frontend von Bedeutung, und da braucht man kein toggle.

[betateilchen]

Wenn du das auch noch dokumentierst,

wäre es so genehm?

(http://up.picr.de/20857109ms.png)

Ich würde das dann heute so für deutsch und englisch in den commandref-frames einchecken.

[rudolfkoenig]

Habs eingecheckt.

[betateilchen]

Ich auch, danke