widget_weather: Patchvorschlag

Begonnen von CQuadrat, 14 August 2018, 09:46:18

Vorheriges Thema - Nächstes Thema

CQuadrat

Hallo,

da PROPLANTA um eine 3h-Auflösung erweitert wurde (siehe: https://forum.fhem.de/index.php/topic,89499.0.html), gibt es jetzt z.B. folgendes Reading:
fc0_weather03Icon

Daher ist es meiner Meinung nach sinnvoll, das Weather-Widget entsprechend zu erweitern.

Mein Vorschlag:
Zeile
if (par.match(/^fc\d+_weather(Day|Evening|Morning|Night)(?:Icon)?$/)) {
                            device_type = 'PROPLANTA';

durch Zeile
if (par.match(/^fc\d+_weather(Day|Evening|Morning|Night|\d\d)(?:Icon)?$/)) {
                            device_type = 'PROPLANTA';

ersetzen.

Wie sieht das der Maintainer?


Danke und Gruß

Christoph
FHEM auf Mini-ITX-Server mit Intel Quad-Core J1900:
+ HM: HM-LAN, HM-USB, HM-MOD-UART mit div. HM-Komponenten
+ RFXtrx: Funkwetterstation Bresser mit ext. Thermometer, Regenmesser und Windmesser
+ TUL (KNX-Anbindung), KM271 (per ser2net), SONOS (div. Gimmicks), OneWire, Hue

setstate

Vielen Dank, habe die Änderung so eingecheckt.

CQuadrat

FHEM auf Mini-ITX-Server mit Intel Quad-Core J1900:
+ HM: HM-LAN, HM-USB, HM-MOD-UART mit div. HM-Komponenten
+ RFXtrx: Funkwetterstation Bresser mit ext. Thermometer, Regenmesser und Windmesser
+ TUL (KNX-Anbindung), KM271 (per ser2net), SONOS (div. Gimmicks), OneWire, Hue