events während update

[herrmannj]

Aufgrund einer user Fehlermmeldung ist mir aufgefallen das update (produziert ja events) auch den Inhalt von "changed" als event verteilt ?

ist das absichtlich so ?
Wenn in changed jetzt zB ein Eintrag "initialized" lautet dann wird dadurch "global:initialized" getriggert, etc.

Patch zur Beseitigung erwünscht ?

vg
joerg

Code Auswählen Erweitern

2016.01.10 16:45:58 1: UPD docs/commandref_frame_DE.html
2016.01.10 16:45:58 1: PERL WARNING: Use of uninitialized value $e[0] in string eq at ./FHEM/32_TechemHKV.pm line 85.
2016.01.10 16:45:58 1: PERL WARNING: Use of uninitialized value $e[0] in string eq at ./FHEM/32_TechemHKV.pm line 87.
2016.01.10 16:45:58 1: PERL WARNING: Use of uninitialized value $e[0] in string eq at ./FHEM/32_TechemHKV.pm line 89.
2016.01.10 16:45:58 1: PERL WARNING: Use of uninitialized value $e[0] in string eq at ./FHEM/32_TechemWZ.pm line 122.
2016.01.10 16:45:58 1: PERL WARNING: Use of uninitialized value $e[0] in string eq at ./FHEM/32_TechemWZ.pm line 124.
2016.01.10 16:45:58 1: PERL WARNING: Use of uninitialized value $e[0] in string eq at ./FHEM/32_TechemWZ.pm line 126.
2016.01.10 16:45:58 1:
2016.01.10 16:45:58 1: New entries in the CHANGED file:
2016.01.10 16:45:58 1:   - change:  FB_CALLMONITOR: reading "missed_call_line" will not be triggered
2016.01.10 16:45:58 1:                              anymore. Use internal_number instead, which
2016.01.10 16:45:58 1:                              contains the same content
2016.01.10 16:45:58 1:   - feature: FB_CALLMONITOR: new attribute answMachine-is-missed-call to
2016.01.10 16:45:58 1:                              treat incoming calls, which were taken by
2016.01.10 16:45:58 1:                              an answering machine, as "missed call"
2016.01.10 16:45:59 1: Calling /usr/bin/perl ./contrib/commandref_join.pl, this may take a while

[Markus Bloch]

Dazu müssten in CHANGED diese entsprechenden Events auch in Großbuchstaben enthalten sein. Wenn man auf "global:INITIALIZED" ein notify hat, dann ist das kein Problem. Bei "global:.*INITIALIZED.*" durchaus, sofern eben "INITIALIZED" im CHANGED irgendwo auftaucht.

Eine sichere Möglichkeit in FHEM selber, dass zu unterbinden sehe ich nicht, außer alle global-Events (INITIALIZED, REREADCFG, ...) in CHANGED zu verbieten (z.B. über pre-commit Hook).

Gruß
Markus

[Loredo]

Übrigens fehlt im Eventmonitor seit einiger Zeit die abschließende Meldung nach generieren der Commandref, dass das Update abgeschlossen wurde.

[herrmannj]

Dazu müssten in CHANGED diese entsprechenden Events auch in Großbuchstaben enthalten sein. Wenn man auf "global:INITIALIZED" ein notify hat, dann ist das kein Problem. Bei "global:.*INITIALIZED.*" durchaus, sofern eben "INITIALIZED" im CHANGED irgendwo auftaucht.

Eine sichere Möglichkeit in FHEM selber, dass zu unterbinden sehe ich nicht, außer alle global-Events (INITIALIZED, REREADCFG, ...) in CHANGED zu verbieten (z.B. über pre-commit Hook).

Gruß
Markus

ich denke das geht.

Frage 1: muss der Inhalt von "changed" überhaupt per event verteilt werden ?

Wenn das überhaupt so sein soll dann könnte man ein "UPD changed:" oder so etwas VOR jede Zeile setzen. Dann wäre das auch beseitigt. Aufgefallen ist mir das im übrigen auch nur weil ich auf "global" nach "initialized" suche und die erste leere Zeile hat produziert ein "global" undef .

vg
joerg

[rudolfkoenig]

Die Daten per Event an das Frontend zu schicken ist ein grober Hack, ich meine wir haben auch andere Mechanismen fuer diese Aufgabe.

Ich nehme es auf meine TODO Liste.

[herrmannj]

drängt nicht. Wenn ich helfen kann bitte Info.

vg
joerg

[Markus Bloch]

Vielleicht kann man ja hier jetzt die "withLog" Option im Event-Monitor nutzen. Anstatt Events zu triggern gibt es dann nur noch Logmeldungen die dann in Rot erscheinen.

Gruß
Markus

[CoolTux]

Vielleicht kann man ja hier jetzt die "withLog" Option im Event-Monitor nutzen. Anstatt Events zu triggern gibt es dann nur noch Logmeldungen die dann in Rot erscheinen.

Gruß
Markus

Da wäre ich dafür, das sieht in meinen Augen am saubersten aus. Auch für die User.


Grüße

[betateilchen]

gibt es dann nur noch Logmeldungen die dann in Rot erscheinen.

rot sollten - wenn überhaupt - nur Meldungen erscheinen, die tatsächlich Fehler sind.

[CoolTux]

Auch war. Da halt gelb   

[Markus Bloch]

rot sollten - wenn überhaupt - nur Meldungen erscheinen, die tatsächlich Fehler sind.

Mag durchaus sein, so ist es aktuell im Event-Monitor implementiert um Logmeldungen von Events zu unterscheiden. Ich war das nicht, da musst du dich bei Rudi beschweren.

[herrmannj]

Rudi hat es doch auf todo genommen. Und sooo dringend ist es doch wirklich nicht...

Vg
Joerg

[betateilchen]

so ist es aktuell im Event-Monitor implementiert um Logmeldungen von Events zu unterscheiden

Ich beschwere mich ja nicht, ich habe die Logmeldungen im Eventmonitor noch nie gebraucht.
Für mich gehören die da auch nicht hin.

[rudolfkoenig]

Geaendert:
- default bei update ist jetzt "updateInBackground" (Voreinstellung geaendert)
- update in background erzeugt keine Events mehr, nur noch logs
- ActivateInformFn aktiviert das Beobachten des Logs (wird bisher nur vom Update verwendet).
- CommandInform von fhem.pl nach telnet.pm geschoben.
- log als Parameter zum inform Befehl hinzugefuegt
- Logmeldungen im EventMonitor sind nicht mehr rot, sondern gruen (im default) oder weiss (darkstyle)
- update bug gefixt (keine Abschlussmeldung in manchen Faellen bei mehreren Quellen)

[Loredo]

- update bug gefixt (keine Abschlussmeldung in manchen Faellen bei mehreren Quellen)

Danke, Rudi!


Mir ist noch etwas aufgefallen. Irgendwie scheint ein Update von einer Drittquelle hier nicht richtig verarbeitet zu werden, denn ich bekomme die Meldung jedes Mal bei einem erneuten Update:

Code Auswählen Erweitern



2016-01-17 12:21:49 Global global fhemtabletui
2016-01-17 12:21:50 Global global UPD www/tablet/js/widget_button.js
2016-01-17 12:21:50 Global global
2016-01-17 12:21:50 Global global New entries in the CHANGED file:
2016-01-17 12:21:50 Global global FHEM tablet UI last change:
2016-01-17 12:21:50 Global global 2016-01-16
2016-01-17 12:21:50 Global global  - Updated thermostat
2016-01-17 12:21:50 Global global
2016-01-17 12:21:50 Global global
2016-01-17 12:21:50 Global global widgets-for-fhem-tablet-ui
2016-01-17 12:21:50 Global global UPD www/tablet/js/widget_button.js