Autor Thema: [patch]: fhem.pl - Erweiterung structChangeHist per Attribut  (Gelesen 20114 mal)

Offline Benni

  • Developer
  • Hero Member
  • ****
  • Beiträge: 2370
  • FHEMinist
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#
Gefällt mir Gefällt mir x 1 Liste anzeigen

Offline betateilchen

  • Developer
  • Hero Member
  • ****
  • Beiträge: 17601
  • s/fhem\.cfg/configDB/g
Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
« Antwort #1 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.
-----------------------
Unaufgeforderte Anfragen per email werden von mir nicht beantwortet. Dafür ist das Forum da.
-----------------------
Lesen gefährdet die Unwissenheit!

Offline Benni

  • Developer
  • Hero Member
  • ****
  • Beiträge: 2370
  • FHEMinist
Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
« Antwort #2 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."

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 24882
Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
« Antwort #3 am: 04 März 2019, 18:12:20 »
Falls das noch jemanden interessiert, dann baue ich es ein.

Offline Loredo

  • Developer
  • Hero Member
  • ****
  • Beiträge: 3753
  • ~ Challenging Innovation ~
Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
« Antwort #4 am: 04 März 2019, 18:16:32 »
Find ich hilfreich  8)
Hat meine Arbeit dir geholfen? ⟹ https://paypal.me/pools/c/8gDLrIWrG9

Maintainer:
FHEM-Docker Image, https://github.com/fhem, Astro(Co-Maintainer), ENIGMA2, GEOFANCY, GUEST, HP1000, Installer, LaMetric2, MSG, msgConfig, npmjs, PET, PHTV, Pushover, RESIDENTS, ROOMMATE, search, THINKINGCLEANER

Offline betateilchen

  • Developer
  • Hero Member
  • ****
  • Beiträge: 17601
  • s/fhem\.cfg/configDB/g
Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
« Antwort #5 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.
-----------------------
Unaufgeforderte Anfragen per email werden von mir nicht beantwortet. Dafür ist das Forum da.
-----------------------
Lesen gefährdet die Unwissenheit!

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 24882
Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
« Antwort #6 am: 05 März 2019, 16:03:41 »
Habs uebernommen (mit weniger Pruefungen) und eingecheckt.
Gefällt mir Gefällt mir x 1 Liste anzeigen

Offline Benni

  • Developer
  • Hero Member
  • ****
  • Beiträge: 2370
  • FHEMinist
Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
« Antwort #7 am: 05 März 2019, 17:04:17 »
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#


Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 24882
Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
« Antwort #8 am: 05 März 2019, 21:16:19 »
Zitat
2 Sachen sind mir aufgefallen:
Danke, habe beide gefixt.

Zitat
Dem Code nach geht aber nur "-?".
Eigentlich nur "save ?".
"save -?" erzeugt schwer zu loeschende Dateien:
Zitat
% rm -?
rm: illegal option -- ?
:)
Gefällt mir Gefällt mir x 1 Liste anzeigen

Offline Benni

  • Developer
  • Hero Member
  • ****
  • Beiträge: 2370
  • FHEMinist
Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
« Antwort #9 am: 06 März 2019, 09:40:04 »
"save -?" erzeugt schwer zu loeschende Dateien::)

mit

rm -- -?
sollte es klappen.  :)


Offline justme1968

  • Developer
  • Hero Member
  • ****
  • Beiträge: 20971
Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
« Antwort #10 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.
« Letzte Änderung: 06 März 2019, 20:28:22 von justme1968 »
FHEM5.4,DS1512+,2xCULv3,DS9490R,HMLAN,2xRasPi
CUL_HM:HM-LC-Bl1PBU-FM,HM-LC-Sw1PBU-FM,HM-SEC-MDIR,HM-SEC-RHS
HUEBridge,HUEDevice:LCT001,LLC001,LLC006,LWL001
OWDevice:DS1420,DS18B20,DS2406,DS2423
FS20:fs20as4,fs20bs,fs20di
AKCP:THS01,WS15
CUL_WS:S300TH
Zustimmung Zustimmung x 1 Liste anzeigen

Offline juemuc

  • Hero Member
  • *****
  • Beiträge: 1059
Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
« Antwort #11 am: 06 März 2019, 22:22:15 »
Hallo zusammen,

ich erhalte nun folgende Meldung:
Zitat
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
3x Sonos Play 1, 1x Sonos Arc + Sub, 1 Sonos-One, 1x Sonos Playbar
Fritzbox 7490 mit 4x Dect 200 und 3 Dect-ULE-Thermostate,  raspberry3, HM Funkmodul HM-MOD-RPI-PCB, HM Klingelsensor HM-Sen-DB-PCB, HM Fensterkontakte und  Amazon Echo Dot, raspberry3B+ mit RPI-RF-MOD und piVCCU, Raspbian ...

Offline Benni

  • Developer
  • Hero Member
  • ****
  • Beiträge: 2370
  • FHEMinist
[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
« Antwort #12 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
FHEM/98_structure.pm         rudolfkoenig         Automatisierung
gb#
« Letzte Änderung: 06 März 2019, 22:39:45 von Benni »

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 24882
Antw:[patch]: fhem.pl - Erweiterung structChangeHist per Attribut
« Antwort #13 am: 08 März 2019, 12:16:47 »
Zitat
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.

 

decade-submarginal