Autor Thema: Attribut global uniqueID => keine Auswirkung  (Gelesen 436 mal)

Offline hexenmeister

  • Developer
  • Hero Member
  • ****
  • Beiträge: 3879
    • tech_LogBuch
Attribut global uniqueID => keine Auswirkung
« am: 12 Mai 2018, 15:51:21 »
Moin!

Mir ist zufällig aufgefallen, dass im 'global'-Gerät definierte 'uniqueID'-Attribut keine Auswirkungen hat.
In 'fhem.pl' definierte Funktionen 'getKeyValue' und 'setKeyValue' verwendet festverdrahtete Werte.
Sollte das nicht repariert weden, oder werden Nebenwirkungen erwartet?

sub
getKeyValue($)
{
  my ($key) = @_;
  my $fName = $attr{global}{modpath}."/FHEM/FhemUtils/uniqueID";
  my ($err, @l) = FileRead($fName);
  return ($err, undef) if($err);
  for my $l (@l) {
    return (undef, $1) if($l =~ m/^$key:(.*)/);
  }
  return (undef, undef);
}

# Use an undefined value to delete the key
sub
setKeyValue($$)
{
  my ($key,$value) = @_;
  my $fName = $attr{global}{modpath}."/FHEM/FhemUtils/uniqueID";
  my ($err, @old) = FileRead($fName);
  my @new;
  if($err) {
    push(@new, "# This file is auto generated.",
               "# Please do not modify, move or delete it.",
               "");
    @old = ();
  }
Cubietruck, RPI3, HM, EnOcean, 1wire, Firmata, MySensors, ESP8266, ESPEasy, MQTT, NodeRED, Alexa, Telegram

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 18420
Antw:Attribut global uniqueID => keine Auswirkung
« Antwort #1 am: 13 Mai 2018, 15:09:57 »
uniqueID wird mW z.Zt. nur von fheminfo verwendet, um die freiwilligen Statistiken zu erstellen. uniqueID wird in der Datei FHEM/FhemUtils/uniqueID abgelegt, und kann vom Benutzer geaendert werden, nur eben nicht ganz einfach, was meiner Ansicht nach angesichts des "typischen" FHEM-Benutzers auch ok ist.
Ich bin z.Zt. eher der Ansicht, dass das global Attribut ueberfluessig ist, und ich wuerde es ausbauen.Andere Meinungen?

Offline hexenmeister

  • Developer
  • Hero Member
  • ****
  • Beiträge: 3879
    • tech_LogBuch
Antw:Attribut global uniqueID => keine Auswirkung
« Antwort #2 am: 13 Mai 2018, 22:14:09 »
Es gibt bereits eine Reihe weiterer Module, die diese Datei verwenden (vor Allem um Passwörter abzulegen). Das wären z.B. FRITZBOX, SYSMON, FB_CALLMONITOR,
FB_CALLLIST.

Ich würde es besser finden, wenn man das definieren kann.
Mein Anwendungsfall: ich habe mehrere FHEM-Instanzen parallel aus dem selben Verzeichniss (natürlich mit unterschiedlichen Config-Dateien) laufen. Es wäre meines Erachtens sinnvoll, wenn diese jeweils eine eigene ID hätten.

Cubietruck, RPI3, HM, EnOcean, 1wire, Firmata, MySensors, ESP8266, ESPEasy, MQTT, NodeRED, Alexa, Telegram

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 18420
Antw:Attribut global uniqueID => keine Auswirkung
« Antwort #3 am: 14 Mai 2018, 09:16:01 »
Wenn ich mich recht erinnere, was das uniqueID Attribut nicht fuers spezifizieren des Dateinamens, sondern fuer den Inhalt von uniqueID gedacht. Um deinen Wunsch zu entsprechen, muesste man ein neues Attribut uniqueIDFileName (oder besser "keyFileName") einfuehren.

Offline hexenmeister

  • Developer
  • Hero Member
  • ****
  • Beiträge: 3879
    • tech_LogBuch
Antw:Attribut global uniqueID => keine Auswirkung
« Antwort #4 am: 15 Mai 2018, 07:15:58 »
Ok, wird aus dem Commandref leider nicht klar. Ich wäre dann dafür, diesen Attribut zu entfernen und eine neuen "keyFileName" in "global" zu definieren. Dein Einverständnis vorausgesetzt, würde ich in den  nächten Tagen einen entsprechenden Patch vorbereiten.

Cubietruck, RPI3, HM, EnOcean, 1wire, Firmata, MySensors, ESP8266, ESPEasy, MQTT, NodeRED, Alexa, Telegram

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 18420
Antw:Attribut global uniqueID => keine Auswirkung
« Antwort #5 am: 15 Mai 2018, 09:41:40 »
Kein Grund fuer den Patch, steht schon auf meiner TodoListe.
Gefällt mir Gefällt mir x 1 Liste anzeigen

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 18420
Antw:Attribut global uniqueID => keine Auswirkung
« Antwort #6 am: 15 Mai 2018, 22:08:24 »
Habe keyFileName ein- und uniqueID ausgebaut.

Offline hexenmeister

  • Developer
  • Hero Member
  • ****
  • Beiträge: 3879
    • tech_LogBuch
Antw:Attribut global uniqueID => keine Auswirkung
« Antwort #7 am: 16 Mai 2018, 22:19:09 »
Funktioniert wie gewollt. Vielen Dank! :D
Cubietruck, RPI3, HM, EnOcean, 1wire, Firmata, MySensors, ESP8266, ESPEasy, MQTT, NodeRED, Alexa, Telegram

Offline betateilchen

  • Developer
  • Hero Member
  • ****
  • Beiträge: 14540
  • s/fhem\.cfg/configDB/g
Antw:Attribut global uniqueID => keine Auswirkung
« Antwort #8 am: 16 Mai 2018, 22:25:27 »
Vielleicht sollte man in diesem Zusammenhang irgendwann mal die 99_Utils.pm bereinigen,

Dort stehen nämlich getUniqueId(), setkeyValue() und getKeyValue() noch in der commandref, vermutlich noch aus der Zeit, als diese Funktionen noch nicht in der fhem.pl vorhanden waren.

-----------------------
Unaufgeforderte Anfragen per email werden von mir nicht beantwortet. Dafür ist das Forum da.
-----------------------
Nächster Hamburg-Stammtisch: 01.06.2018

Offline betateilchen

  • Developer
  • Hero Member
  • ****
  • Beiträge: 14540
  • s/fhem\.cfg/configDB/g
Antw:Attribut global uniqueID => keine Auswirkung
« Antwort #9 am: 16 Mai 2018, 22:27:17 »
Und könnte es vielleicht sein, dass dieser Thread in einem völlig falschen Forumbereich steht? Unter

CUL -> Entwicklung -> Fehlerberichte

finde ich das Thema irgendwie völlig merkwürdig.
-----------------------
Unaufgeforderte Anfragen per email werden von mir nicht beantwortet. Dafür ist das Forum da.
-----------------------
Nächster Hamburg-Stammtisch: 01.06.2018

Offline hexenmeister

  • Developer
  • Hero Member
  • ****
  • Beiträge: 3879
    • tech_LogBuch
Antw:Attribut global uniqueID => keine Auswirkung
« Antwort #10 am: 16 Mai 2018, 22:47:48 »
Unter

CUL -> Entwicklung -> Fehlerberichte

finde ich das Thema irgendwie völlig merkwürdig.


Ähem.. Du hast recht. Anfangs habe ich jedoch angenommen, dass es das Attribut 'uniqueID' nicht funktioniert hat (tat es ja auch nicht und wurde jetzt auch entfernt). Daher Fehlerberichte. Habe mich wohl aber auf 'CUL' verklickt. Ich verschiebe mal FHEM - Entwicklung -> FHEM Development. Danke für den Hinweis.


Cubietruck, RPI3, HM, EnOcean, 1wire, Firmata, MySensors, ESP8266, ESPEasy, MQTT, NodeRED, Alexa, Telegram

Offline betateilchen

  • Developer
  • Hero Member
  • ****
  • Beiträge: 14540
  • s/fhem\.cfg/configDB/g
Antw:Attribut global uniqueID => keine Auswirkung
« Antwort #11 am: 16 Mai 2018, 22:56:54 »
Man hätte den Ausbau des Attributes vielleicht auch ankündigen sollen. Es tauchen schon die ersten Fehler-Threads auf, in denen User nicht wissen, wie sie mit der Fehlermeldung beim FHEM Start umgehen sollen, wenn sie das Attribut bisher gesetzt hatten.

Mal wieder so ein typischer Schnellschuß, mit dem man die Anwender grundlos ärgert. Niemandem hätte das Attribut wehgetan, auch wenn es nicht benutzt wird. Es gibt so viele sinnlose Attribute in FHEM, da kommt es auf eines mehr oder weniger nicht an.

Das Attribut keyFileName hätte man ja trotzdem einführen können.

-----------------------
Unaufgeforderte Anfragen per email werden von mir nicht beantwortet. Dafür ist das Forum da.
-----------------------
Nächster Hamburg-Stammtisch: 01.06.2018

Offline hexenmeister

  • Developer
  • Hero Member
  • ****
  • Beiträge: 3879
    • tech_LogBuch
Antw:Attribut global uniqueID => keine Auswirkung
« Antwort #12 am: 16 Mai 2018, 23:18:49 »
Grundsätzlich könnte man natürlich ankündigen. Da das Attribut jedoch eh nie funktioniert hat, ist es sichern nicht so wild...
Cubietruck, RPI3, HM, EnOcean, 1wire, Firmata, MySensors, ESP8266, ESPEasy, MQTT, NodeRED, Alexa, Telegram

 

decade-submarginal