Skip to content

Issue with Weather Kludget #13

@GoogleCodeExporter

Description

@GoogleCodeExporter
The configuration script for the Weather Kludget is incorrectly written.

...not sure if the source for this is a seperate repository. However, the zip 
is named com.kludgets.weather.zip, and this is included in the default Kludget 
installer package

On line 79 in weather.js the following is written:

"userUnit = Kludget.Settings.read('kludget/weather/useMetric', 0) ? 'm' : 's';"

However, this always evaluates to 0, regardless of the configuration file value 
in the widget preferences.xml.

Suggest changing this to: 
"userUnit = Kludget.Settings.read('kludget/weather/useMetric', 0)==1 ? 'm' : 
's';

Which correctly interprets a 1 in the preferences file as 'm' and a 0 as 's'.

Original issue reported on code.google.com by smaudet2@gmail.com on 29 Aug 2010 at 7:57

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions