59_Weather.pm: Patch für Januar-Bug

[Baumi]

Hallo!

Ich hab' einen kleinen Bug im Weather-Modul gefunden: Im Moment stolpert es über den Januar. Bei allen Daten in diesem Monat wird im Reading pubDateComment die Meldung "could not parse pubDate" angezeigt.

Version:

Code Auswählen Erweitern

59_Weather.pm         9950 2015-11-21 05:57:31Z betateilchen

Grund sind diese Zeilen (Zeile 178 und 179) in der Funktion Weather_ParseDateTime:

Code Auswählen Erweitern


my $m= $monthindex{$mon};
return undef unless $m;

Da $m im Januar 0 ist, trifft die Bedingung nicht zu und es wird undef zurückgegeben.

Bugfix dafür: Man ändere die zweite Zeile in:

Code Auswählen Erweitern

return undef unless $m ne "";

Dann wird nur noch undef zurückgegeben, wenn der Monat nicht gefunden werden konnte.

Ich weiß nicht genau, wie man Bugfixes einreicht, darum habe ich es einfach mal hier ins Forum gepackt. Falls man das eigentlich nicht tun sollte, bitte gerne anmeckern.

[Dr. Boris Neubert]

Herzlichen Dank, Baumi, für den Hinweis.

Hab's gefixt und eingecheckt.

Und ja, Du warst hier genau richtig.

Viele Grüße
Boris

[Baumi]

Danke! Und der Fix, der im Code gelandet ist, ist natürlich nochmal ne Ecke "sauberer" als mein Hack.