[patch]: fhem.pl - Erweiterung structChangeHist per Attribut

[Benni]

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#

[betateilchen]

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.

[Benni]

Ich finde aber gerade die Auflistung der einzelnen Änderungen meist ganz hilfreich, zumindest hilfreicher als nur ein Eintrag "Device Soundso wurde geändert."

[rudolfkoenig]

Falls das noch jemanden interessiert, dann baue ich es ein.

[Loredo]

Find ich hilfreich 

[betateilchen]

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.

[rudolfkoenig]

Habs uebernommen (mit weniger Pruefungen) und eingecheckt.

[Benni]

Habs uebernommen (mit weniger Pruefungen) und eingecheckt.

Danke! 

2 Sachen sind mir aufgefallen:

Code Auswählen Erweitern

   
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#

[rudolfkoenig]

2 Sachen sind mir aufgefallen:

Danke, habe beide gefixt.

Dem Code nach geht aber nur "-?".

Eigentlich nur "save ?".
"save -?" erzeugt schwer zu loeschende Dateien:

% rm -?
rm: illegal option -- ?

[Benni]

"save -?" erzeugt schwer zu loeschende Dateien:

mit

Code Auswählen Erweitern

rm -- -?

sollte es klappen. 

[justme1968]

Code Auswählen Erweitern

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.

[juemuc]

Hallo zusammen,

ich erhalte nun folgende Meldung:

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

[Benni]

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

Code Auswählen Erweitern

FHEM/98_structure.pm         rudolfkoenig         Automatisierung

gb#

[rudolfkoenig]

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