This indicator evals json data (which is potentially dangerous) and the 
evaluated data has ['ResultSet']['Error'] as "0" (string value) while the code 
checks for 0, a number.
>>> '0' == 0
False

So the quick and dirty fix in this case is to modify the condition at line 367 
in indicator-weather to read:
if (yahoo_woeid_result['ResultSet']['Error'] != '0') and  
(yahoo_woeid_result['ResultSet']['Results'] != None):

But the proper fix is to rewrite the evals using json module.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1093769

Title:
  indicator-weather Cannot add location

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/indicator-weather/+bug/1093769/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to