indicator-weather crashed with KeyError in finish_initializing(): 'yahoo id'

[Mattijs Riekerk]

I checked the weather forecast in my town, all ok.
After that I selected a city in my list and checked the forecast.
At that moment the program crashed.

ProblemType: Crash
DistroRelease: Ubuntu 12.04
Package: indicator-weather 12.07.30-0ubuntu1.1-ppa2.3~precise1 [origin: LP-PPA-jtasker-weather-indicator]
ProcVersionSignature: Ubuntu 3.2.0-41.66-generic-pae 3.2.42
Uname: Linux 3.2.0-41-generic-pae i686
ApportVersion: 2.0.1-0ubuntu17.2
Architecture: i386
CrashCounter: 1
CrashDB: indicator_weather
Date: Fri May 3 21:56:01 2013
ExecutablePath: /usr/bin/indicator-weather
InstallationMedia: Ubuntu 12.04 LTS "Precise Pangolin" - Release i386 (20120423)
InterpreterPath: /usr/bin/python2.7
MarkForUpload: True
PackageArchitecture: all
ProcCmdline: /usr/bin/python /usr/bin/indicator-weather
PythonArgs: ['/usr/bin/indicator-weather']
SourcePackage: indicator-weather
ThirdParty: True
Title: indicator-weather crashed with KeyError in finish_initializing(): 'yahoo id'
UpgradeStatus: No upgrade log present (probably fresh install)
UserGroups: adm cdrom dip lpadmin plugdev sambashare sudo

[Joshua Tasker (jtasker)]

It crashed because the id for the city (Buenos Aires) was set to None instead of the correct value of '20067592'.

I can't reproduce this on my system, I don't know how it got set to None.

What happens when you go to Preferences... > Locations, then Delete and re-Add the city to your locations?

[Joshua Tasker (jtasker)]

So, I still don't know how it got set to None, but if you change line 1552 in indicator-weather from

if len(place)>1:

to

if len(place)>1 and place[0] is not None:

It should solve the crash issue. You will have to re-add your city.

I posted a new package to my ppa with that change (and a fix for an unrelated crash) for those interested.