Autor Thema: vorschlag: set, Regexp parts und Create SVG plot für fakelog verhindern  (Gelesen 622 mal)

Offline justme1968

  • Developer
  • Hero Member
  • ****
  • Beiträge: 19234
mit dem folgenden kleinen patch werden set, Regexp parts und Create SVG plot für fakelog nicht mehr angezeigt. meiner meinung nach ist nichts davon sinnvoll und verhindert so fehlbedienung.

ich würde gerne ein solches fakelog device dann auch für die anzeige der alexa-fhem logs nutzen statt den anzeige teil doppelt zu implementieren. aber mit den vielen knöpfen ist das eine mögliche fehlerquelle :)

--- 92_FileLog.pm (revision 18125)
+++ 92_FileLog.pm (working copy)
@@ -269,6 +269,8 @@
   my ($hash, @a) = @_;
   my $me = $hash->{NAME};

+  return undef if( $hash->{REGEXP} eq 'fakelog' );
+
   return "no set argument specified" if(int(@a) < 2);
   my %sets = (reopen=>0, clear=>0, absorb=>1, addRegexpPart=>2,
               removeRegexpPart=>1);
@@ -409,6 +411,7 @@
   }
   $ret .= "</table>";
   return $ret if($pageHash);
+  return $ret if( $defs{$d}{REGEXP} eq 'fakelog' );

   # DETAIL only from here on
   my $hash = $defs{$d};
« Letzte Änderung: 05 Januar 2019, 22:32:44 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
Gefällt mir Gefällt mir x 1 Liste anzeigen

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 20575
Eingecheckt

Offline justme1968

  • Developer
  • Hero Member
  • ****
  • Beiträge: 19234
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

Offline betateilchen

  • Developer
  • Hero Member
  • ****
  • Beiträge: 15956
  • s/fhem\.cfg/configDB/g
Könnte man dieses Abschalten bitte irgendwie konfigurierbar machen?

Oder zumindest dafür sorgen, dass ein "set ... clear" noch funktioniert, wenn man es über die Befehlszeile eingibt?

Diese Möglichkeit scheint nicht nur mir zu fehlen.

https://forum.fhem.de/index.php/topic,96099.0.html
« Letzte Änderung: 18 Januar 2019, 20:49:43 von betateilchen »
-----------------------
Unaufgeforderte Anfragen per email werden von mir nicht beantwortet. Dafür ist das Forum da.
-----------------------
Der Hamburg-Stammtisch am 14.06.2019 fällt aus!
Gefällt mir Gefällt mir x 1 Zustimmung Zustimmung x 2 Liste anzeigen

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 20575
Ich frage mich, wieso du mit der trivialen Loesung (Regexp von fakelog in FakeLog umbenennen) nicht klarkommst.

Offline betateilchen

  • Developer
  • Hero Member
  • ****
  • Beiträge: 15956
  • s/fhem\.cfg/configDB/g
Und ich frage mich, welchen triftigen Grund es gibt, ein „set clear“ für fakelog  plötzlich nicht mehr zuzulassen?

meiner meinung nach ist nichts davon sinnvoll

Zumindest das Löschen des Loginhaltea ist eine sinnvolle Anforderung



fakelog (kleingeschrieben) ist seit vielen Jahren Standard in FHEM. Im Lauf der Zeit kamen in 92_FileLog.pm die setList und die Bearbeiung von RegExParts im Frontend dazu und haben sich auch zum Standard in vielen Tausend vorhandenen FHEM Installationen entwickelt. Niemand hatte damit jemals ein Problem oder die Ursache für ein Problem bei einem fakelog device.

Dann kommt Andre um die Ecke und hat für EINEN bestimmten Anwendungsfall den Vorschlag, etwas zu ändern. Schwupps wird ohne Vorankündigung diese Änderung umgesetzt und das Standardverhalten in 100% aller bestehenden Installationen verändert.

Hätte man nicht einfach die Logik umdrehen können? Das Ausblenden findet nur statt, wenn die REGEXP in der Form FakeLog angegeben wird?

Damit würde man zwei Dinge erreichen:

  • Andre kann sein alexa-log ohne die gewünschten Elemente im Frontend anzeigen, indem er bei der Anlage seines Logfies FakeLog verwendet
  • in allen anderen Fällen und in bestehenden FHEM Installationen verändert sich nichts

« Letzte Änderung: 19 Januar 2019, 10:14:17 von betateilchen »
-----------------------
Unaufgeforderte Anfragen per email werden von mir nicht beantwortet. Dafür ist das Forum da.
-----------------------
Der Hamburg-Stammtisch am 14.06.2019 fällt aus!
Gefällt mir Gefällt mir x 1 Zustimmung Zustimmung x 2 Liste anzeigen

 

decade-submarginal