Comment 5 for bug 1093769

Revision history for this message
Roman Yepishev (rye) wrote :

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.