changing timezone for a city overrides the weather code

Bug #200911 reported by Steve Langasek
2
Affects Status Importance Assigned to Milestone
GNOME Panel
Fix Released
Medium
gnome-panel (Ubuntu)
Fix Released
Low
Ubuntu Desktop Bugs

Bug Description

Binary package hint: gnome-panel

In some cases (such as bug #199582), one may wish to pick a location for weather and specify a different timezone than the one autopopulated by the location tool. In that case, gnome-panel will override the location code used for weather with the one corresponding to the city of the *timezone*, not the location code matching the city originally selected!

Here is a result after choosing North America -> US -> Oregon -> Portland -> HIllsboro Airport from the chooser:

$ gconftool -g /apps/panel/applets/clock_screen0/prefs/cities
[<location name="Portland" timezone="America/Vancouver" latitude="45.548054" longitude="-122.954445" code="KHIO" current="true"/>]

And here is the setting after manually changing the timezone to the correct value through the UI by typing out the timezone name:

$ gconftool -g /apps/panel/applets/clock_screen0/prefs/cities
[<location name="Portland" timezone="America/Los_Angeles" latitude="45.548054" longitude="-122.954445" code="CYVR" current="true"/>]

And again, here is the value if I select the timezone from the pulldown list:

$ gconftool -g /apps/panel/applets/clock_screen0/prefs/cities
[<location name="Portland" timezone="America/Los_Angeles" latitude="45.548054" longitude="-122.954445" code="KCQT" current="true"/>]

The METAR code should only be updated when changing the location, not when changing the timezone.

Related branches

Revision history for this message
Steve Langasek (vorlon) wrote :

The attached patch fixes this issue for me; clicking 'edit' for a location now no longer corrupts the weather code value.

Changed in gnome-panel:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Pedro Villavicencio (pedro) wrote :

Thanks for your bug report. This bug has been reported to the developers of the software. You can track it and make comments here: http://bugzilla.gnome.org/show_bug.cgi?id=522007

Changed in gnome-panel:
assignee: nobody → desktop-bugs
status: Confirmed → Triaged
Changed in gnome-panel:
status: Unknown → New
Revision history for this message
Pedro Villavicencio (pedro) wrote :

the bug has been fixed upstream. thanks for reporting.

Changed in gnome-panel:
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gnome-panel - 1:2.22.1-0ubuntu1

---------------
gnome-panel (1:2.22.1-0ubuntu1) hardy; urgency=low

  * New upstream version:
    Panel
    - Convert username/login to UTF-8 in logout dialog
    - Fix applet processes not quitting when the applet is removed (lp: #195915)
    Misc
    - Remove old and useless files
    - Require gweather 2.22.1
    - Don't explicitly require libxml for the weather applet anymore
    Clock Applet
    - Fix crash when there are no locations available
    - Fix locations not appearing in the preferences dialog
    - Make find location window larger
    - Handle escape to close the dialog in the time/timezone dialogs
    - Fix various issues with adding/removing locations
    - Fix crash when PolicyKit helper can't be called (lp: #208791)
    - Fix crash when closing the calendar popup when a city is blinking
      on the map (lp: #200881, ##207954)
    - Improve the blinking on the map when clicking on various locations
    - Cleanups
    - Correctly redraw the map outline on expose events
    - Fix crash when the location doesn't have coordinates (lp: #201102)
    - Fix crash when updating locations while calendar popup is not shown
      (lp: #210470)
    - Fix initial visibility of the appointments to actually follow the
      state of the expander (lp: #206118)
    - Make sure we always use data from the right country (lp: #185190)
    - Show caldav appointments in the calendar
    - Ignore dbus errors when setting the time and there's no reply on the bus
      (lp: #207890)
    - Open evolution on the right day, in all timezones (lp: #66028)
    - Fix crash when an evolution-data-server dies (lp: #122590)
    - Correctly set time, even when DST is on (lp: #208598)
    - Handle multiple locations in the same timezone in a better way
    - Fix weather tooltip to show the real temperature and the apparent
      temperature, instead of only the apparent one (lp: #188183)
    - Do not lose the weather location information when changing the
      timezone of a location (lp: #200911)
    - Don't show weather information of the last location once it has been
      removed
    - Remove empty space on the left of the clock when there's no location
      (lp: #203691)
    - Potential fix for another crash in the calendar code
    - When adding a location, make it the current one if it makes sense
  * debian/control.in:
    - updated libgweather requirement
  * debian/patches/17_expose_wm_keybindings.patch:
    - dropped, the current version has some issues and need to be rework
      during next cycle (lp: #211112)
  * debian/patches/80_from_bugzilla_use_correct_current_timezone.patch:
    - dropped, fixed in the new version

 -- Sebastien Bacher <email address hidden> Tue, 08 Apr 2008 01:47:12 +0200

Changed in gnome-panel:
status: Fix Committed → Fix Released
Changed in gnome-panel:
status: New → Fix Released
Changed in gnome-panel:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.