Bug 236309 - LCD Weather Station shows no values using wetter.com
Summary: LCD Weather Station shows no values using wetter.com
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: widget-weather (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-04 13:24 UTC by foobar22
Modified: 2012-12-01 21:27 UTC (History)
8 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.5.2


Attachments
Screenshot of both weather applets that use same city's information from wetter.com (98.06 KB, image/png)
2010-06-20 22:21 UTC, Hakan Bayindir
Details
Screenshot of the layout problem (599.46 KB, image/png)
2010-08-11 17:32 UTC, Pablo
Details
XML data returned from the API call (4.46 KB, text/xml)
2010-09-08 21:16 UTC, Thilo-Alexander Ginkel
Details
Patch (1.04 KB, patch)
2010-09-10 10:24 UTC, Thilo-Alexander Ginkel
Details

Note You need to log in before you can comment on or make changes to this bug.
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.