Falsche Reference auf Readings in Weather.pm

[knorxi]

In der Datei lib/FHEM/Core/Weather.pm werden Abfragen auf Readings in Kleinschreibung gemacht:

Code Auswählen Erweitern

$h->{readings}oder auch mit

Code Auswählen Erweitern

$hash->{readings}Müssten nicht alle Abfragen in Großschreibung READINGS also mit

Code Auswählen Erweitern

$h->{READINGS}oder

Code Auswählen Erweitern

$hash->{READINGS}gemacht werden?
Zumindest funktioniert bei mir der Get-Befehle nicht ohne diese Änderung.

Code Auswählen Erweitern

$h->{readings}existiert zwar, ist aber leer.

[CoolTux]

Muss ich mir anschauen.

[CoolTux]

Also auf dem ersten Blick gebe ich Dir Recht. Krass ist das daß ganze schon seit 3 Jahren so falsch im Code steht. Ich Versuche das die Tage mal zu richten.

[CoolTux]

Ok ich habe das nun gefixt

https://git.cooltux.net/FHEM/mod-Weather/src/branch/main/

Leider komme ich erst kommenden Freitag dazu es ins SVN ein zu checken.

[knorxi]

Danke dir!

Krass ist das daß ganze schon seit 3 Jahren so falsch im Code steht.

Das habe ich mir auch gedacht und war deshalb verunsichert, ob ich irgendwas übersehe.
Ich denke, wenn es 3 Jahre niemand bemerkt hat, dann kann man es verkraften, dass es erst nächsten Freitag eingecheckt wird