Autor Thema: vorschlag: alias statt name bei 'Probably associated with' in fhemweb  (Gelesen 1071 mal)

Offline justme1968

  • Developer
  • Hero Member
  • ****
  • Beiträge: 20311
ich glaube es wäre schöner wenn in der 'Probably associated with' tabelle der alias angezeigt wird statt dem device namen. das ist zum einen konsistent mit dem rest von fhemweb und zum anderen ist es übersichtlicher wenn man devices nicht umbenennt sondern die autocereate namen beibehält.

Index: 01_FHEMWEB.pm
===================================================================
--- 01_FHEMWEB.pm (revision 21351)
+++ 01_FHEMWEB.pm (working copy)
@@ -1559,7 +1559,7 @@
     FW_pO "<table class=\"block wide $class\">";
     foreach (sort @obj) {
       FW_pF "<tr class=\"%s\"><td>", (($row++)&1)?"odd":"even";
-      FW_pH "detail=$_", $_;
+      FW_pH "detail=$_", AttrVal($_, 'alias', $_);
       FW_pO "</td><td>";
       FW_pO $defs{$_}{STATE} if(defined($defs{$_}{STATE}));
       FW_pO "</td><td>";
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: 22538
Habs uebernommen.

Offline PatrickR

  • Developer
  • Sr. Member
  • ****
  • Beiträge: 776
vorschlag: alias statt name bei 'Probably associated with' in fhemweb
« Antwort #2 am: 04 März 2020, 17:01:12 »
Ich hoffe, ich habe das nicht falsch verstanden, würde mich aber gerne dafür aussprechen, dass es bei der alten Variante bleibt. Der Name ist im Gegensatz zum Alias global eindeutig.


Von unterwegs gesendet.
lepresenced - Tracking von Bluetooth-LE-Tags (Gigaset G-Tag) mittels PRESENCE

"Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the universe trying to produce bigger and better idiots. So far, the universe is winning." - Rich Cook
Zustimmung Zustimmung x 2 Liste anzeigen

Offline Ellert

  • Developer
  • Hero Member
  • ****
  • Beiträge: 3638
Ich bin auch dafür, dass der Name weiterhin angezeigt wird und nicht der Alias, zum Einen wegen der Eindeutigkeit und zum Anderen müsste all jene, die die Gerätenamen kennen jetzt umlernen.
 
In einem notify oder DOIF werden typischer Weise die Namen oder Regex auf den Namen verwendet. Wenn bei der Fehlersuche in der Definition der Name verwendet wird und man sich diese Gerät ansehen will, findet man es unter den verlinkten Aliasnamen nicht, wenn man nicht beides kennt.

Wenn überhaupt sollte die Verwendung des Alias per Attribut eingeschaltet werden.
Zustimmung Zustimmung x 1 Liste anzeigen

Offline justme1968

  • Developer
  • Hero Member
  • ****
  • Beiträge: 20311
der alias ist dazu da um aus den nicht sprechenden namen die z.b. bei autocreate erzeugt werden einen für menschen einfach zu verstehende namen zu machen unter dem man sich etwas vorstellen kann.

in fhemweb wird überall der alias angezeigt. dafür ist der alias da!

also z.b. aus HUEDevice65545 ein 'Rollo Terasse' oder ähnliches zu machen.

ich kann gerade nicht nachvollziehen unter welchen umständen der erste screenshot besser verständlich ist als der zweite.

vielleicht würde ein beispiel helfen.
« Letzte Änderung: 05 März 2020, 13:24:36 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

Offline marvin78

  • Developer
  • Hero Member
  • ****
  • Beiträge: 5665
Du pickst aber auch das denkbar schlechteste Szenario raus. Meine Devices haben sprechende Namen (nicht alias) und ich erkenne sofort am Namen, worum es sich handelt. Das ist tatsächlich beim alias nicht so, da dieser sich oft auf Szenarios (wie bspw. einen Raum) bezieht. Ich halte es für gut, dass gerade an dieser Stelle kein alias verwendet wird, weil man hier doch sehr häufig Rückschlüsse auf Nebeneffekte ziehen kann.  Die Fehlersuche ist hier ein gutes Stichwort.

Man kann also sagen, dass das "Problem" hinter der Anzeige der Namen, eher individuell ist und man deine Argumentation leicht rumdrehen kann.
« Letzte Änderung: 05 März 2020, 13:29:03 von marvin78 »
Zustimmung Zustimmung x 2 Liste anzeigen

Online CoolTux

  • Developer
  • Hero Member
  • ****
  • Beiträge: 25697
Wie ist das wenn ich in 2 Räumen 2 Vitrinen habe? Beide heißen Vitrine weil es unterschiedliche Räume sind? In Deiner Übersicht hätte ich 2mal Vitrine oder wegen meiner schlimmer einmal, weiß aber nicht welche. Muss dann erst den Link anklicken sofern vorhanden.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://paypal.me/pools/c/8gULisr9BT
My FHEM Git: https://git.cooltux.net/FHEM/
Mein Dokuwiki:
https://www.cooltux.net

Offline justme1968

  • Developer
  • Hero Member
  • ****
  • Beiträge: 20311
das ist nicht das denkbar schlechteste szenario, sondern normal wenn man autocreate verwendet. und es ist durchaus sinnvoll diese automatisch erzeugten namen nicht zu ändern weil mancher zusammenhang nur so erkannt werden kann.

nein. man muss nicht anklicken sondern nur die maus darüber bewegen und sieht den link mit dem device namen.

aber da es müssig ist darüber zu streiten schlage ich vor das name und alias angezeigt wird:--- 01_FHEMWEB.pm (revision 21351)
+++ 01_FHEMWEB.pm (working copy)
@@ -1559,7 +1559,11 @@
     FW_pO "<table class=\"block wide $class\">";
     foreach (sort @obj) {
       FW_pF "<tr class=\"%s\"><td>", (($row++)&1)?"odd":"even";
-      FW_pH "detail=$_", $_;
+      if( defined($attr{$_}) && defined($attr{$_}{alias}) ) {
+        FW_pH "detail=$_", "$_ ($attr{$_}{alias})";
+      } else {
+        FW_pH "detail=$_", $_;
+      }
       FW_pO "</td><td>";
       FW_pO $defs{$_}{STATE} if(defined($defs{$_}{STATE}));
       FW_pO "</td><td>";


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

Online CoolTux

  • Developer
  • Hero Member
  • ****
  • Beiträge: 25697
das ist nicht das denkbar schlechteste szenario, sondern normal wenn man autocreate verwendet. und es ist durchaus sinnvoll diese automatisch erzeugten namen nicht zu ändern weil mancher zusammenhang nur so erkannt werden kann.

nein. man muss nicht anklicken sondern nur die maus darüber bewegen und sieht den link mit dem device namen.

aber da es müssig ist darüber zu streiten schlage ich vor das name und alias angezeigt wird:--- 01_FHEMWEB.pm (revision 21351)
+++ 01_FHEMWEB.pm (working copy)
@@ -1559,7 +1559,11 @@
     FW_pO "<table class=\"block wide $class\">";
     foreach (sort @obj) {
       FW_pF "<tr class=\"%s\"><td>", (($row++)&1)?"odd":"even";
-      FW_pH "detail=$_", $_;
+      if( defined($attr{$_}) && defined($attr{$_}{alias}) ) {
+        FW_pH "detail=$_", "$_ ($attr{$_}{alias})";
+      } else {
+        FW_pH "detail=$_", $_;
+      }
       FW_pO "</td><td>";
       FW_pO $defs{$_}{STATE} if(defined($defs{$_}{STATE}));
       FW_pO "</td><td>";

Also ich streite nicht. Ich hatte nur ne Nachfrage/Diskussion.  ;D
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://paypal.me/pools/c/8gULisr9BT
My FHEM Git: https://git.cooltux.net/FHEM/
Mein Dokuwiki:
https://www.cooltux.net

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 22538
Ich finde die Argumentation von justme1968 schluessiger bzw. seinen Fall wahrscheinlicher.

Auf der anderen Seite habe ich gerade wiederentdeckt, dass alias Raum- oder Sonstwas-abhaengig (siehe nameDisplay) definiert sein kann, insofern ist die aktuelle Implementation auch nicht korrekt.

Seufz.

Offline marvin78

  • Developer
  • Hero Member
  • ****
  • Beiträge: 5665
das ist nicht das denkbar schlechteste szenario, sondern normal wenn man autocreate verwendet. und es ist durchaus sinnvoll diese automatisch erzeugten namen nicht zu ändern weil mancher zusammenhang nur so erkannt werden kann.

nein. man muss nicht anklicken sondern nur die maus darüber bewegen und sieht den link mit dem device namen.

aber da es müssig ist darüber zu streiten schlage ich vor das name und alias angezeigt wird:--- 01_FHEMWEB.pm (revision 21351)
+++ 01_FHEMWEB.pm (working copy)
@@ -1559,7 +1559,11 @@
     FW_pO "<table class=\"block wide $class\">";
     foreach (sort @obj) {
       FW_pF "<tr class=\"%s\"><td>", (($row++)&1)?"odd":"even";
-      FW_pH "detail=$_", $_;
+      if( defined($attr{$_}) && defined($attr{$_}{alias}) ) {
+        FW_pH "detail=$_", "$_ ($attr{$_}{alias})";
+      } else {
+        FW_pH "detail=$_", $_;
+      }
       FW_pO "</td><td>";
       FW_pO $defs{$_}{STATE} if(defined($defs{$_}{STATE}));
       FW_pO "</td><td>";

Es gibt keinen Streit, es gibt bloß unterschiedliche herangehensweisen. Die Notwendigkeit die Namen aus autocreate (nutze ich ebenfalls) beizubehalten, halte ich persönlich für gering bis nicht vorhanden, deshalb ist es (für mich) das denkbar schlechteste Szenario. Das Argument mit der Erkennen von Zuammenhängen kann man wieder herumdrehen, wie oben schon beschrieben.
Zustimmung Zustimmung x 1 Liste anzeigen

Offline marvin78

  • Developer
  • Hero Member
  • ****
  • Beiträge: 5665
Ich finde die Argumentation von justme1968 schluessiger bzw. seinen Fall wahrscheinlicher.



Auf welcher Grundlage?

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 22538
Ein alias vergibt man dann, wenn man NAME nicht sehen will, keiner wird zum alias gezwungen.
Sowas wie "ja, schon, aber hier nicht" ist fuer mich ein Sonderfall.

Offline marvin78

  • Developer
  • Hero Member
  • ****
  • Beiträge: 5665
Einen alias vergebe ICH für das Frontend => Anzeige für andere, mich selbst, wenn ich keine Verwaltung brauche. Im Backend will ICH die Namen sehen. 'Probably associated with' gehört für mich zum "Backend". Natürlich kann man so für viele weitere Stellen argumentieren. Damit lasse ich es gut sein. Es ist eine tote Diskussion, ich bleibe aber dabei, dass man es sehr individuell betrachten kann und es ggf. 50/50 ausgeht, wenn man die Leute befragt.
« Letzte Änderung: 05 März 2020, 14:09:04 von marvin78 »

Offline Ellert

  • Developer
  • Hero Member
  • ****
  • Beiträge: 3638
Einen alias vergebe ICH für das Frontend => Anzeige für andere, mich selbst, wenn ich keine Verwaltung brauche. Im Backend will ICH die Namen sehen. 'Probably associated with' gehört für mich zum "Backend". Natürlich kann man so für viele weitere Stellen argumentieren. Damit lasse ich es gut sein. Es ist eine tote Diskussion, ich bleibe aber dabei, dass man es sehr individuell betrachten kann und es ggf. 50/50 ausgeht, wenn man die Leute befragt.
Deshalb denke ich, der Gerätebezeichner in 'Probably associated with' sollte per Attribut wählbar sein, das würde die Zahl der Unzufriedenen reduzieren.