Loglevel bei fhem() in fhem.pl

[Dr. Boris Neubert]

Hallo,

fhem() in fhem.pl loggt die Meldungen mit Loglevel 3, es sei denn, man gibt als zweiten Parameter z.B. 1 an.

Loglevel 3 als Default scheint mir hoch. Nach meinem Gefühl sollte verbose=3 keine "normalen" Aktivitäten loggen. Bei mir hat das zur Folge, daß der von RSS alle 10 Sekunden aufgerufene { fhem("get Calendar text all") } das Log vollmüllt.

Möchte zur Diskussion stellen, den Loglevel auf 4 zu setzen oder gleich fhem() wie folgt zu ändern:

Code Auswählen Erweitern

fhem($@)
{
  my ($param, $loglevel) = @_;
  $loglevel= 4 unless($loglevel);
  my $ret = AnalyzeCommandChain(undef, $param);
  Log $loglevel, "$param : $ret" if($ret);
  return $ret;
}


Grüße
Boris

[rudolfkoenig]

Nach der Definition (http://fhem.de/commandref.html#verbose: 3 - commands sent out will be logged.) finde ich 3 richtig. In deinem Beispiel solte man fhem() mit dem Parameter $silent=1 aufrufen, oder den "get Calendar text all" etwas "leiser" stellen.

Ich wuerde dein Patch (bis auf dem 4) uebernehmen, falls es mit der alten Semantik ($param, $silent) kompatibel waere.

[betateilchen]

Ich verwende in solchen Fällen, wo fhem() für mich überflüssiges Logging produziert, auch das $silent.

@Boris: was spricht denn dagegen, in Deinem RSS einfach den fhem() Aufruf um ein ", 1" zu ergänzen?

[Dr. Boris Neubert]

Hallo Rudi,

Danke für die Rückmeldung!

Nach der Definition (http://fhem.de/commandref.html#verbose: 3 - commands sent out will be logged.) finde ich 3 richtig.

Diese Definition ist irgendwie an mir vorbeigegangen...

Fazit: ich will verbose=2.

Baustelle: ich muß alle meine Module auf Konformität mit dieser Beschreibung prüfen und ggf. anpassen.

Viele Grüße
Boris

[Dr. Boris Neubert]

@Boris: was spricht denn dagegen, in Deinem RSS einfach den fhem() Aufruf um ein ", 1" zu ergänzen?

Nachdem ich jetzt verstanden habe, daß ich verbose=2 will, brauche ich das Flag nicht zu setzen. Es spricht auch nichts dagegen, es zu verwenden. Ich bin von falschen Voraussetzungen ausgegangen.

Grüße
Boris

[betateilchen]

Baustelle: ich muß alle meine Module auf Konformität mit dieser Beschreibung prüfen und ggf. anpassen.

Viel Spaß... (aber eigentlich gäbe es da mMn gerade wichtigere Baustellen)