FHEM Forum

FHEM => Sonstiges => Thema gestartet von: Benni am 04 März 2019, 10:13:46

Titel: [patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: Benni am 04 März 2019, 10:13:46
Hallo Rudi,

weil ich neulich selbst mal Bedarf hatte:Hier ein Patch (inkl. Doku), der eine Erweiterung der Change-History (structChangeHist) auf mehr als 10 Einträge per Attribut am global-Device ermöglicht.

gb#
Titel: Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: betateilchen am 04 März 2019, 10:20:02
Eigentlich fände ich es hilfreicher, wenn jedes device, an dem etwas geändert wurde, in der Liste nur einmal auftauchen würde - egal, wieviele und welche Änderungen es an dem device gab. Dann würde auch die bisherige Limitierung auf 10 Einträge vermutlich ausreichen.
Titel: Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: Benni am 04 März 2019, 10:25:51
Ich finde aber gerade die Auflistung der einzelnen Änderungen meist ganz hilfreich, zumindest hilfreicher als nur ein Eintrag "Device Soundso wurde geändert."
Titel: Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: rudolfkoenig am 04 März 2019, 18:12:20
Falls das noch jemanden interessiert, dann baue ich es ein.
Titel: Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: Loredo am 04 März 2019, 18:16:32
Find ich hilfreich  8)
Titel: Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: betateilchen am 04 März 2019, 18:45:59
hilfreich wobei?

Mir erschließt sich der Sinn noch nicht. Aber wenn das Standardverhalten bleibt und Sonderwünsche explizit per Attribut gesetzt werden müssen, kann es mir auch egal sein.
Titel: Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: rudolfkoenig am 05 März 2019, 16:03:41
Habs uebernommen (mit weniger Pruefungen) und eingecheckt.
Titel: Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: Benni am 05 März 2019, 17:04:17
Zitat von: rudolfkoenig am 05 März 2019, 16:03:41
Habs uebernommen (mit weniger Pruefungen) und eingecheckt.

Danke!  :)

2 Sachen sind mir aufgefallen:

   
return "Last saved structural changes:\n  ".join("\n  ", @structChangeHist);

sollte es hier nicht eher unsaved heißen? Die Änderungen wurden ja noch nicht in der config gespeichert.
Mann könnte "saved" aber auch einfach weg lassen.

Außerdem gibst du in der deutschen commandref zum Attribut "save -r" als Anzeigemöglichkeit an.
Dem Code nach geht aber nur "-?".

gb#

Titel: Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: rudolfkoenig am 05 März 2019, 21:16:19
Zitat2 Sachen sind mir aufgefallen:
Danke, habe beide gefixt.

ZitatDem Code nach geht aber nur "-?".
Eigentlich nur "save ?".
"save -?" erzeugt schwer zu loeschende Dateien:
Zitat% rm -?
rm: illegal option -- ?
:)
Titel: Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: Benni am 06 März 2019, 09:40:04
Zitat von: rudolfkoenig am 05 März 2019, 21:16:19
"save -?" erzeugt schwer zu loeschende Dateien::)

mit

rm -- -?

sollte es klappen.  :)

Titel: Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: justme1968 am 06 März 2019, 09:49:31
rm ./-?

ist weniger zu tippen :)

aber: man sollte das ? maskieren oder in anführungszeichen packen. sonst könnte es auf alle möglichen zeichen matchen.
Titel: Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: juemuc am 06 März 2019, 22:22:15
Hallo zusammen,

ich erhalte nun folgende Meldung:
Zitat2019.03.06 22:18:15 1: PERL WARNING: Use of uninitialized value $sret in substitution (s///) at ./FHEM/98_structure.pm line 479.
2019.03.06 22:18:15 1: PERL WARNING: Use of uninitialized value $sret in split at ./FHEM/98_structure.pm line 480.

Viele Grüße
Jürgen
Titel: [patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: Benni am 06 März 2019, 22:35:28
Sorry, aber das ist der falsche Thread und der falsche Forenbereich dafür!

Hier geht es um fhem.pl und um ungespeicherte structural changes der config.

Das hat nichts mit dem STRUCTURE-Modul zu tun.

Das gehört in den Forenbereich Automatisierung.

Auszug aus der MAINTAINER.txt (https://fhem.de/MAINTAINER.txt)
FHEM/98_structure.pm         rudolfkoenig         Automatisierung

gb#
Titel: Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
Beitrag von: rudolfkoenig am 08 März 2019, 12:16:47
Zitat2019.03.06 22:18:15 1: PERL WARNING: Use of uninitialized value $sret in substitution (s///) at ./FHEM/98_structure.pm line 479.
Einer der Mitglieder im structure liefert auf "set X ?" undefined zurueck.
Fuer eine weitere Analyse bitte alle structure Mitgliedstypen angeben, aber wie Benni das geschrieben hat, dafuer ein neues Thema anlegen.