FHEM Forum

FHEM => Frontends => Thema gestartet von: betateilchen am 29 April 2014, 10:14:22

Titel: [patch] 91_eventTypes.pm - neuer Befehl "set eT clear"
Beitrag von: betateilchen am 29 April 2014, 10:14:22
Warum auch immer 91_eventTypes.pm laut Maintainer.txt zu Frontend gehört (ich fände Automatisierung logischer) ...

Bezugnehmend auf diesen Thread http://forum.fhem.de/index.php/topic,23020.0.html schlage ich folgende Änderung vor


###################################
sub
eventTypes_Set($@)
{
  my ($hash, @a) = @_;
  return eventTypes_Shutdown($hash, $hash->{NAME}) if($a[1] eq "flush");
  return $modules{eventTypes}{ldata} = undef if($a[1] eq "clear");
  return "Unknown argument $a[1], choose one of flush:noArg clear:noArg";
}


um die im Speicher gesammelten Daten für die eventTypes löschen zu können.

Da fällt mir grade auf, dass eventTypes eigentlich auch nach configDB schreiben können sollte. Aber das bauen wir besser erst ein, wenn wir beide die generischen Funktionen zum Lesen und Schreiben umgesetzt haben.
Titel: Antw:[patch] 91_eventTypes.pm - neuer Befehl "set eT clear"
Beitrag von: rudolfkoenig am 29 April 2014, 14:32:29
Kurz getestet und eingecheckt.
Titel: Antw:[patch] 91_eventTypes.pm - neuer Befehl "set eT clear"
Beitrag von: betateilchen am 30 April 2014, 08:38:46
Hallo Rudi,

ich habe hier noch die commandref ergänzt, flush war auch noch nicht dokumentiert.
Könntest Du beim Einchecken bitte auch das $Id Keyword setzen, damit die Versionsinfo aktualisiert wird?

Viele Grüße
Udo



Index: FHEM/91_eventTypes.pm
===================================================================
--- FHEM/91_eventTypes.pm (revision 5698)
+++ FHEM/91_eventTypes.pm (working copy)
@@ -184,7 +184,17 @@
   <br>

   <a name="eventTypesset"></a>
-  <b>Set</b> <ul>N/A</ul><br>
+  <b>Set</b>
+  <ul>
+      <li>flush<br>
+        used to write all collected event types into datafile.
+      </li>
+      <br/>
+      <li>clear<br>
+        used to clear the internal table containing all collected event types.
+      </li>
+  </ul>
+  <br>

   <a name="eventTypesget"></a>
   <b>Get</b>

Titel: Antw:[patch] 91_eventTypes.pm - neuer Befehl "set eT clear"
Beitrag von: rudolfkoenig am 30 April 2014, 11:20:47
Danke, eingecheckt.