[patch] 91_eventTypes.pm - neuer Befehl "set eT clear"

[betateilchen]

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

Code Auswählen Erweitern


###################################
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.

[rudolfkoenig]

Kurz getestet und eingecheckt.

[betateilchen]

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

Code Auswählen Erweitern


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>


[rudolfkoenig]

Danke, eingecheckt.