Bug 236309

Summary:	LCD Weather Station shows no values using wetter.com
Product:	[Unmaintained] plasma4	Reporter:	foobar22
Component:	widget-weather	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	andresbajotierra, aseigo, asraniel, ewoerner, KaiUweBroulik2, scp.stjohn, shawn.starr, thilo
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed In:	4.5.2
Sentry Crash Report:
Attachments:	Screenshot of both weather applets that use same city's information from wetter.com Screenshot of the layout problem XML data returned from the API call Patch

Description foobar22 2010-05-04 13:24:40 UTC

Version:            (using KDE 4.4.2)
OS:                Linux
Installed from:    Ubuntu Packages

Steps to reproduce:
Add the LCD Weather Station to your desktop
Go to the Weather Station Settings and change your location
Choose any place you'd like and choose wetter.com as your provider

What should happen:
LCD Weather Station should show the temperature, pressure, etc.

What actually happens:
The widget shows 0.0hPa, -273°C, no humidity and no wind

Regards

Comment 1 Eckhart Wörner 2010-05-08 21:14:08 UTC

This bug has also been reported as http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=580518

Comment 2 Hakan Bayindir 2010-06-20 22:21:14 UTC

Created attachment 48171 [details]
Screenshot of both weather applets that use same city's information from wetter.com

Comment 3 Pablo 2010-08-11 17:32:49 UTC

Created attachment 50018 [details]
Screenshot of the layout problem

Comment 4 Thilo-Alexander Ginkel 2010-08-11 21:07:18 UTC

Does this issue still occur with more recent KDE releases (such as 4.4.5)?

Comment 5 Kai Uwe Broulik 2010-09-08 11:59:47 UTC

It still states 0 K and no values in KDE 4.5.1 ..

Comment 6 Thilo-Alexander Ginkel 2010-09-08 21:16:10 UTC

Created attachment 51439 [details]
XML data returned from the API call

It seems that wetter.com returns a decimal fraction for the temperatures when requesting the forecast data for a Turkish city. I assume that this is not correctly handled by the xml.readElementText().toInt() call in the XML parser.

Right now, my KDE development environment is somewhat non-functional, so if someone can commit a quick fix that would be great. Otherwise, I'll have a look as soon as my dev environment is back. ;-)

Comment 7 Thilo-Alexander Ginkel 2010-09-10 10:24:44 UTC

Created attachment 51506 [details]
Patch

The attached patch should solve the issue. I will commit it to SVN once I have done some more testing.

Comment 8 Thilo-Alexander Ginkel 2010-09-11 00:18:40 UTC

SVN commit 1173938 by ginkel:

wetter.com Ion: fix forecast retrieval for some geographic regions

CCBUG: 236309


 M  +2 -2      ion_wettercom.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1173938

Comment 9 Thilo-Alexander Ginkel 2010-09-11 00:35:47 UTC

SVN commit 1173946 by ginkel:

wetter.com Ion: fix forecast retrieval for some geographic regions

BUG: 236309
FIXED-IN: 4.5.2


 M  +2 -2      ion_wettercom.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1173946

Comment 10 Dario Andres 2010-12-10 22:23:13 UTC

[Comment from a bug triager]
There are new cases described on bug 259471, even on KDE SC 4.6beta2 . Should this be reopened ?
Regards

Comment 11 SP 2012-12-01 21:27:53 UTC

bug exists in KDE 4.93.  should be reopened.