Merge lp:~martin-borho/ubuntu-weather-app/iconfix into lp:ubuntu-weather-app/obsolete.trunk

Proposed by Martin Borho
Status: Merged
Approved by: Raúl Yeguas
Approved revision: 21
Merged at revision: 21
Proposed branch: lp:~martin-borho/ubuntu-weather-app/iconfix
Merge into: lp:ubuntu-weather-app/obsolete.trunk
Diff against target: 17 lines (+6/-1)
1 file modified
components/WeatherConditionIconComponent.qml (+6/-1)
To merge this branch: bzr merge lp:~martin-borho/ubuntu-weather-app/iconfix
Reviewer Review Type Date Requested Status
Raúl Yeguas Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+164586@code.launchpad.net

Commit message

hotfix for inaccurate api data (icon code incorrect)

Description of the change

Fix for broken value in OWM api response for condition icon. See http://bugs.openweathermap.org/issues/72

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Raúl Yeguas (neokore) wrote :

Thank you for being noticing these kind of details. Great work!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'components/WeatherConditionIconComponent.qml'
--- components/WeatherConditionIconComponent.qml 2013-04-21 18:05:01 +0000
+++ components/WeatherConditionIconComponent.qml 2013-05-18 11:01:26 +0000
@@ -18,7 +18,12 @@
18 property bool __snow: false18 property bool __snow: false
19 property bool __fog: false19 property bool __fog: false
2020
21 onConditionChanged: weatherCondition.state = condition21 onConditionChanged: {
22 // hotfix for inaccurate api data
23 // see http://bugs.openweathermap.org/issues/72
24 if(condition === "10") condition ="10d"
25 weatherCondition.state = condition
26 }
2227
23 Image {28 Image {
24 id: sun29 id: sun

Subscribers

People subscribed via source and target branches