[Bug?] FHEMWEB erzeugt room oder group mit dem Namen 1

[betateilchen]

Hallo Rudi,

wenn man über die Weboberfläche ein Attribut group (oder room) setzt, ohne im Feld für den Wert etwas auszuwählen, wird eine group (oder room) mit dem Namen "1" angelegt.

Meines Erachtens sollte in dem Fall eine Fehlermeldung ausgegeben werden, dass der Wert für das Attribut fehlt.

[betateilchen]

Ergänzung: der Effekt ist nicht auf room und group beschränkt, sondern tritt (vermutlich) bei allen Attributen ohne Wert auf.

[rudolfkoenig]

Und das ist seit 13 Jahren so, und in manchen Faellen, wie "attr global stacktrace" ganz praktisch.

[betateilchen]

Dass das seit 13 Jahren so ist, mag ja sein. Damals gab es aber auch noch sehr viel weniger Attribute als heute.

Heutzutage finde ich dieses Verhalten weder logisch noch für Otto Normaluser verständlich. Aufgekommen ist das Thema heute wieder in einem Thread, in dem ein User ziemlich verzweifelt darüber war, dass er die 1 nicht wegbekommt, die er selbst nie (bewusst) angelegt hat.

Vielleicht sollte man dieses Verhalten von FHEM nun doch irgendwann mal überdenken

[rudolfkoenig]

Aufgekommen ist das Thema heute wieder in einem Thread, in dem ein User ziemlich verzweifelt darüber war, dass er die 1 nicht wegbekommt, die er selbst nie (bewusst) angelegt hat.

Wenn ein Benutzer das 1 nicht wegbekommt, dann sehe ich fuers Verstehen der Fehlermeldung schwarz.
Sind auch noch Andere der Ansicht, dass diese Aenderung die Welt besser macht?

[Christoph Morrison]

Wenn ein Benutzer das 1 nicht wegbekommt, dann sehe ich fuers Verstehen der Fehlermeldung schwarz.
Sind auch noch Andere der Ansicht, dass diese Aenderung die Welt besser macht?

Man lernt damit umzugehen, aber wirklich sinnig ist das Verhalten eine 1 zu setzen wenn man (versehentlich) gar nichts setzt, nicht, oder?
Ich würde eine Fehlermeldung erwarten, wenn ein leerer String / undef übergeben wird. Auch in Hinblick eines automatisierten Aufruf aus einem Programm heraus.

[CoolTux]

Ich wäre auch für eine Fehlermeldung wenn nichts übergeben wird. Macht irgendwie Sinn.

[amenomade]

in manchen Faellen, wie "attr global stacktrace" ganz praktisch.

In manchen anderen Fällen, eher kontraproduktiv. Ich glaube, jeder Helfer im Forum hat schon mal ein notify, oder ein komplexes DOIF untersucht, bis er feststellen musste, dass im (eh nicht-) triggernden Device folgendes stand:

Code Auswählen Erweitern

attr <device> event-on-change-reading 1

[Wernieman]

Wenn gewünscht wäre ein besserer Wert als "1" zu empfehlen .. wie z.B. "Dummy".

[Christoph Morrison]

Wenn gewünscht wäre ein besserer Wert als "1" zu empfehlen .. wie z.B. "Dummy".

Aber warum? Dann hätte der User ja gleich "Dummy" nehmen können. Es ist halt ein Fehler, wenn man room/group/... ohne Parameter aufruft und dann sollte es imho eine Fehlermeldung geben.

[rudolfkoenig]

Ab sofort gibt es eine Fehlermeldung, wenn man ein Attribut ohne Wert setzen will.
Achtung: das setzen von 1 beim Fehlen des Arguments ist _NICHT_ FHEMWEB spezifisch gewesen, sondern eine Eigenschaft des attr Befehls.

Ich gehe davon aus, dass wegen dieser Aenderung hoechstens Probleme bei fhem.cfg Editierer geben wird, die nie save verwenden, aber man weiss es ja nie. Und natuerlich bei denen, die meine Ratschlaege aus dem Forum wegen "attr global stacktrace", "attr WEB HTTPS", usw. befolgen wollen. Und nur weil FHEM massentauglich werden muss.

[amenomade]

Und nur weil FHEM massentauglich werden muss.

Sei doch nicht so traurig! Du kennst ja die Alternative: "Die Programmierer (als Gauchos) muessen die Benutzer (als Rindviecher) unermuedlich vor sich hertreiben, damit sie richtig gut werden?"

[betateilchen]

Ab sofort gibt es eine Fehlermeldung, wenn man ein Attribut ohne Wert setzen will.

Super, danke!

[frank]

jetzt ist mir endlich klar, warum so viele leute "attr event-on-change-reading 1" gesetzt haben.

[nils_]

jetzt ist mir endlich klar, warum so viele leute "attr event-on-change-reading 1" gesetzt haben.

genau das hat mich auch immer gewundert - und otto hatte mich drauf hingewiesen wie einfach es ist dieses "Problem" zu erzeugen.

danke rudi für die änderung