Crashes when opening Forecast if current location is missing weather.com id (indicator-weather crashed with KeyError in finish_initializing(): 'weather-com id')

Bug #1233778 reported by Nicola Jelmorini
476
This bug affects 92 people
Affects Status Importance Assigned to Milestone
Weather Indicator
High
Joshua Tasker

Bug Description

When I open the weather forecast window the indicator crashes and the window doesn't appear.

ProblemType: Crash
DistroRelease: Ubuntu 12.04
Package: indicator-weather 13.7.1 [origin: LP-PPA-weather-indicator-team]
ProcVersionSignature: Ubuntu 3.2.0-54.82-generic 3.2.50
Uname: Linux 3.2.0-54-generic x86_64
ApportVersion: 2.0.1-0ubuntu17.5
Architecture: amd64
CrashCounter: 1
CrashDB: indicator_weather
Date: Tue Oct 1 19:22:55 2013
ExecutablePath: /usr/bin/indicator-weather
InstallationMedia: Ubuntu 12.04.1 LTS "Precise Pangolin" - Release amd64 (20120823.1)
InterpreterPath: /usr/bin/python2.7
MarkForUpload: True
PackageArchitecture: all
ProcCmdline: /usr/bin/python /usr/bin/indicator-weather
ProcEnviron:
 LANGUAGE=it_CH:it
 PATH=(custom, no user)
 LANG=it_CH.UTF-8
 SHELL=/bin/bash
PythonArgs: ['/usr/bin/indicator-weather']
SourcePackage: indicator-weather
ThirdParty: True
Title: indicator-weather crashed with KeyError in finish_initializing(): 'weather-com id'
UpgradeStatus: No upgrade log present (probably fresh install)
UserGroups: adm cdrom dip lpadmin plugdev sambashare sudo vboxusers

Revision history for this message
Nicola Jelmorini (jelmorini) wrote :
information type: Private → Public
Revision history for this message
Bob Davidson (bobdavid) wrote :

I tried to open the forecast window and the app crashed.

Joshua Tasker (jtasker)
Changed in weather-indicator:
status: New → Triaged
importance: Undecided → High
summary: - indicator-weather crashed with KeyError in finish_initializing():
- 'weather-com id'
+ Crashes when opening Forecast if current location is missing weather.com
+ id (indicator-weather crashed with KeyError in finish_initializing():
+ 'weather-com id')
Revision history for this message
Joshua Tasker (jtasker) wrote :

This occurs because for some reason your chosen location has no Weather.com ID saved (probably the location was added to the list a long time ago with a very old version of Weather Indicator).

Ideally when this occurs Weather Indicator would try to lookup the Weather.com ID, and save it with the location data, but that will take some time to implement, so for right now I have attached a patch that should just avoid the crash.

Changed in weather-indicator:
assignee: nobody → Joshua Tasker (jtasker)
milestone: none → 13.7.2
Revision history for this message
Giorgio (gioloi72) wrote :

Same here, but I'm using Yahoo (not weather.com) as source (but the error reports weather.com...)
I have deleted the old location, and set it again (In the add location preference page it finds in the database the location I enter, so it should set the weather.com ID), but the error still occours when I try to open the forecast window.
This happens after this morning update to 13.7.1

Revision history for this message
Giorgio (gioloi72) wrote :

After restart all seems ok now

Joshua Tasker (jtasker)
Changed in weather-indicator:
status: Triaged → Fix Committed
Changed in weather-indicator:
status: Fix Committed → Fix Released
Revision history for this message
Joshua Tasker (jtasker) wrote :

Please do not change the bug status to 'Fix Released' unless the fix is included in the latest released version, not just as a patch or an unstable build. Thanks.

Changed in weather-indicator:
status: Fix Released → Fix Committed
Revision history for this message
Blue Boelter (blueaz) wrote :

I don't see how to get the patch, and it still crashes. A couple updates ago, it actually worked for awhile. Is there any way to get back to that version?

Revision history for this message
DK (dkretz) wrote :

I applied the provided patch but still have the issue.

Revision history for this message
philinux (philcb) wrote :

13.7.1 here but forecast still causes error although the indicator continues in the panel albeit prefs are inaccessible.

Revision history for this message
Id2ndR (id2ndr) wrote :

Like philcb, it crashed today with indicator-weather 13.7.1

Changed in weather-indicator:
status: Fix Committed → Fix Released
Revision history for this message
Lluïsa Nuñez (lluisanunez) wrote :

Not fixed still in ubuntu 13.10
My location DOES have an id at weather.com but it still crashes when asking for weather forecast

Joshua Tasker (jtasker)
Changed in weather-indicator:
status: Fix Released → Fix Committed
Joshua Tasker (jtasker)
Changed in weather-indicator:
milestone: 13.7.2 → 13.8.0
status: Fix Committed → Fix Released
Revision history for this message
Gene Bray (genebray) wrote : Re: [Bug 1233778] Re: Crashes when opening Forecast if current location is missing weather.com id (indicator-weather crashed with KeyError in finish_initializing(): 'weather-com id')

Installed 13.8.0 today. Seems to have fixed the display problem. I can
now see everything I should. Forecasts display. Refresh works.
Reported as bug but cannot find my e-mail thread on subject--sorry.

gene bray

On 02/06/2014 08:57 AM, Joshua Tasker wrote:
> ** Changed in: weather-indicator
> Milestone: 13.7.2 => 13.8.0
>
> ** Changed in: weather-indicator
> Status: Fix Committed => Fix Released
>

Revision history for this message
Diego Simak (diego-simak) wrote :

Installed 13.8.0 in Ubuntu 12.04
python-pywapi is 0.3.7

Forecast is working now.

Thanks

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers