Weather Indicator

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 on 2013-10-01

476

This bug affects 92 people

Affects		Status	Importance	Assigned to	Milestone
	Weather Indicator	Fix Released	High	Joshua Tasker	Weather Indicator 13.8.0 "Rainy 8"

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

Tags:

Revision history for this message

Nicola Jelmorini (jelmorini) wrote on 2013-10-01:

Dependencies.txt Edit (4.5 KiB, text/plain; charset="utf-8")
IndicatorLog.txt Edit (20.2 KiB, text/plain; charset="utf-8")
ProcMaps.txt Edit (54.7 KiB, text/plain; charset="utf-8")
ProcStatus.txt Edit (823 bytes, text/plain; charset="utf-8")
Settings.txt Edit (2.2 KiB, text/plain; charset="utf-8")
Traceback.txt Edit (399 bytes, text/plain; charset="utf-8")

information type:

Private → Public

Revision history for this message

Bob Davidson (bobdavid) wrote on 2013-10-02:

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

Joshua Tasker (jtasker) on 2013-10-02

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 on 2013-10-02:

Fix for Forecast crash when missing weather.com ID Edit (727 bytes, text/plain)

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 on 2013-10-02:

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 on 2013-10-02:

After restart all seems ok now

Joshua Tasker (jtasker) on 2013-10-04

Changed in weather-indicator:
status:	Triaged → Fix Committed

adriel watt (adriel-watt) on 2013-10-05

Changed in weather-indicator:
status:	Fix Committed → Fix Released

Revision history for this message

Joshua Tasker (jtasker) wrote on 2013-10-09:

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 on 2013-10-22:

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 on 2013-11-02:

I applied the provided patch but still have the issue.

Revision history for this message

philinux (philcb) wrote on 2013-11-20:

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 on 2013-12-15:

#10

Like philcb, it crashed today with indicator-weather 13.7.1

Frank Steinert (franksteinert) on 2013-12-19

Changed in weather-indicator:
status:	Fix Committed → Fix Released

Revision history for this message

Lluïsa Nuñez (lluisanunez) wrote on 2014-02-01:

#11

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) on 2014-02-05

Changed in weather-indicator:
status:	Fix Released → Fix Committed

Joshua Tasker (jtasker) on 2014-02-06

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 on 2014-02-06: 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')

#12

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
>