! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
weather plugin measurement unit not persistent
Status:
RESOLVED: FIXED
Product:
Xfce4-weather-plugin
Component:
General

Comments

Description Ken Dreyer 2006-06-07 00:49:50 CEST
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.2) Gecko/20060419 CentOS/1.5.0.2-3.c4.centos Firefox/1.5.0.2
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.2) Gecko/20060419 CentOS/1.5.0.2-3.c4.centos Firefox/1.5.0.2

All other values for the weather plugin's configuration (location code, etc.) are saved and reloaded when I restart Xfce except for the Measurement Unit value.

I am using I am using the weather plugin 0.5, Xfce 4.4Beta, GTK+ 2.6.7, CentOS 4.3, kernel 2.6.9-34.EL.

Reproducible: Always

Steps to Reproduce:
1. Change the Measurement Value from Metric to Imperial
2. Restart Xfce
3. The Measurement Value is back to Metric




Someone mentioned this problem in #XFCE earlier today, and another user pointed out that the cause of the problem is a misspelling of "celcius" on line 315 of plugin.c. Changing "celsius" to "celcius" fixes the problem.
Comment 1 Ken Dreyer 2006-06-07 00:51:26 CEST
Created attachment 598 
Patch to fix Measurement Unit bug

Attached a patch to plugin.c.
Comment 2 Ralf Meyer 2006-06-07 05:35:45 CEST
According to 
http://en.wikipedia.org/wiki/Celsius

celsius is written with "S". There must be another place in the source where it falsely is written with "C" in the middle.
Comment 3 Nick Schermer editbugs 2006-06-07 07:00:16 CEST
This is already fixed in trunk
Comment 4 Nick Schermer editbugs 2006-06-07 11:17:14 CEST
Change to fixed, will be in the new package.
Comment 5 Jannis Pohlmann editbugs 2009-06-12 13:21:36 CEST
Re-assigning to Colin Leroy who is the new maintainer of the plugin.

Bug #1906

Reported by:
Ken Dreyer
Reported on: 2006-06-07
Last modified on: 2014-12-07

People

Assignee:
Colin Leroy
CC List:
2 users

Version

Version:
0.7.3 or older

Attachments

Patch to fix Measurement Unit bug (403 bytes, patch)
2006-06-07 00:51 CEST , Ken Dreyer
no flags

Additional information