FHEM Forum

FHEM - Anwendungen => Multimedia => Thema gestartet von: knopf_piano am 29 Dezember 2015, 11:50:35

Titel: Bugfix-Vorschlag 71_YAMAHA_AVR.pm - volumeStraight
Beitrag von: knopf_piano am 29 Dezember 2015, 11:50:35
Hi Markus,
Fehler im Modul in der regex für volumeStraight-Prüfung.
Version

# $Id: 71_YAMAHA_AVR.pm 10278 2015-12-28 09:51:20Z markusbloch $


volumeStraight hat Minus-Zeichen im Argument. Ich hab die Regex erweitert (Zeile 440 - [-])

elsif(defined($a[2]) and $a[2] =~ /^[-]\d+(?:\.\d)?$/)


Sonst kommt $usage zum vorschein.
Ist mir beim Lautstärkeabsenken beim Tel-Klingeln aufgefallen.

getestet, i.O. -- Modul-Code anbei zum diffen.

Grüße
Steffen
Titel: Antw:Bugfix-Vorschlag 71_YAMAHA_AVR.pm - volumeStraight
Beitrag von: Markus Bloch am 29 Dezember 2015, 14:04:43
Hallo Steffen,

stimmt, da hab ich nicht aufgepasst. Da mich heute noch niemand angerufen hat und der Fernseher noch nicht an war, habe ich das auch nicht bemerkt. :D Prinzipiell muss aber kein Minus-Zeichen davor sein (wenn man über 0 dB geht, die Nachbarn freuts jedenfalls), daher habe ich folgende Regexp gewählt:

/^-?\d+(?:\.\d)?$/

Die Änderungen habe ich soeben eingecheckt und stehen ab morgen zur Verfügung.

Viele Grüße

Markus
Titel: Antw:Bugfix-Vorschlag 71_YAMAHA_AVR.pm - volumeStraight
Beitrag von: knopf_piano am 29 Dezember 2015, 14:07:15
Zitat von: Markus Bloch am 29 Dezember 2015, 14:04:43
Hallo Steffen,

stimmt, da hab ich nicht aufgepasst. Da mich heute noch niemand angerufen hat und der Fernseher noch nicht an war, habe ich das auch nicht bemerkt. :D Prinzipiell muss aber kein Minus-Zeichen davor sein (wenn man über 0 dB geht, die Nachbarn freuts jedenfalls), daher habe ich folgende Regexp gewählt:

/^-?\d+(?:\.\d)?$/

Die Änderungen habe ich soeben eingecheckt und stehen ab morgen zur Verfügung.

Viele Grüße

Markus
Right