vorschlag: set, Regexp parts und Create SVG plot für fakelog verhindern

Begonnen von justme1968, 05 Januar 2019, 13:40:50

Vorheriges Thema - Nächstes Thema

justme1968

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};
hue, tradfri, alexa-fhem, homebridge-fhem, LightScene, readingsGroup, ...

https://github.com/sponsors/justme-1968


justme1968

hue, tradfri, alexa-fhem, homebridge-fhem, LightScene, readingsGroup, ...

https://github.com/sponsors/justme-1968

betateilchen

#3
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
-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!

rudolfkoenig

Ich frage mich, wieso du mit der trivialen Loesung (Regexp von fakelog in FakeLog umbenennen) nicht klarkommst.

betateilchen

#5
Und ich frage mich, welchen triftigen Grund es gibt, ein ,,set clear" für fakelog  plötzlich nicht mehr zuzulassen?

Zitat von: justme1968 am 05 Januar 2019, 13:40:50
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

-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!