FHEM Forum

FHEM => Sonstiges => Thema gestartet von: betateilchen am 01 Februar 2015, 01:15:43

Titel: [patch] fhem.pl - attr <device> disable toggle
Beitrag von: betateilchen am 01 Februar 2015, 01:15:43

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
...?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.
Titel: Antw:[patch] fhem.pl - attr <device> disable toggle
Beitrag von: rudolfkoenig am 01 Februar 2015, 12:32:08
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.
Titel: Antw:[patch] fhem.pl - attr <device> disable toggle
Beitrag von: betateilchen am 01 Februar 2015, 13:22:45
Zitat von: rudolfkoenig am 01 Februar 2015, 12:32:08
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.
Titel: Antw:[patch] fhem.pl - attr <device> disable toggle
Beitrag von: rudolfkoenig am 01 Februar 2015, 14:20:07
Habs eingecheckt.
Titel: Antw:[patch] fhem.pl - attr <device> disable toggle
Beitrag von: betateilchen am 01 Februar 2015, 14:21:16
Ich auch, danke :)