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: Unmaintained
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
Sentry Crash Report:

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
View All Add an attachment

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.