After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 86631 - GWeather applet doesnÃ,´t work at all
GWeather applet doesnÃ,´t work at all
Status: RESOLVED FIXED
Product: gnome-applets
Classification: Other
Component: gweather
unspecified
Other other
: High critical
: ---
Assigned To: gnome-applets Maintainers
gnome-applets Maintainers
Depends on:
Blocks:
 
 
Reported: 2002-06-27 12:49 UTC by m.r.martens
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (4.13 KB, patch)
2002-07-04 14:25 UTC, Deepa Chacko Pillai
none Details | Review

Description m.r.martens 2002-06-27 17:35:27 UTC
Package: gnome-applets
Severity: critical
Version: 2.00
Synopsis: GWeather applet doesn´t work at all
Bugzilla-Product: gnome-applets
Bugzilla-Component: gweather

Description:
Description of Problem: It says 'Retrieval failed'. Previous version
(with Gnome 1.4) worked fine though. None of the locations works, and I
don´t have a proxy server or something like that. Other internet
related applications are working fine.


Steps to reproduce the problem:
1. Add the weather report applet to a panel

Actual Results: ´Retrieval failed' (? -)

Expected Results: A nice icon showing the current weather.

How often does this happen? Always


Additional Information:




------- Bug moved to this database by unknown@bugzilla.gnome.org 2002-06-27 13:35 -------

Reassigning to the default owner of the component, gnome-applets-maint@bugzilla.gnome.org.

Comment 1 Andrew Sobala 2002-06-27 19:06:54 UTC
Confirming
Comment 2 Luis Villa 2002-06-27 19:16:03 UTC
Hrm. Works just fine here. Sure there is no proxy? And you've
configured the location correctly? 

Andrew: it is really helpful if you add the GNOME2 keyword to
unconfirmed bugs that relate to GNOME2 when you find them. Makes it
much easier for me to find them and adjust accordingly.
Comment 3 Andrew Sobala 2002-06-27 19:30:35 UTC
Luis: I know, just forgot on this one.

Anyway, reproducing sequence:

1. Add applet ;-)
2. Preferences -> Select location -> Europe -> United Kingdom -> Leeds
and Bradford
4. Update applet a lot

It _sometimes_ doesn't work, but after the first failure it keeps failing.
Comment 4 Kevin Vandersloot 2002-07-01 23:51:06 UTC
Okay, I just committed a changed to cvs that may fix this. If someone
could test this, that would be great.

Andrew: Many, Many thanks for providing repeatable steps to reproduce
this :)

Here's the relavant ChangeLog entry:
2002-07-01  Kevin Vandersloot <kfv101@psu.edu>

        * weather.c: set gweather_applet->gweather_info to the newly
created info structure. Hopefully fixes bug #86631 and maybe some
others. Prevents multiple http requests when the location has been
changed and updated
Comment 5 Spider (D.m.D. Ljungmark) 2002-07-02 00:46:58 UTC
okay, on a similar note here, I had this but with another procedure:

A) add applet
B) select Europe- > sweden -> norrköping
C) go to prefrences, set metric units
D) Close
-
retrieval will fail, when opening the location again it will be
unselected.
it works if one doubleclicks the location and then closes.

--

Since I'm unable to use CVS I can't help you with further information
here. 
System:
Gentoo, gcc 3.1 -O2 -march=athlon-tbird + debugging , No proxy. NAT
Comment 6 Andrew Sobala 2002-07-02 17:25:44 UTC
Kevin: I'm rebuilding gnome from CVS tonight, and I'll test this later.
Comment 7 Andrew Sobala 2002-07-02 20:58:36 UTC
Still broken with my both my steps and spider's steps to reproduce :-(
Comment 8 Kevin Vandersloot 2002-07-03 14:01:44 UTC
yah, and I need to revert the change since it leads to even more
breakage :( Really a lot of the code needs rewriting.
Comment 9 Deepa Chacko Pillai 2002-07-04 14:25:17 UTC
Created attachment 9634 [details] [review]
Proposed patch
Comment 10 Bharat Tewari 2002-07-09 11:49:57 UTC
does the above patch fixes the problem andrew?
Comment 11 Andrew Sobala 2002-07-09 14:31:02 UTC
Fixed for me
Comment 12 Andrew Sobala 2002-07-09 14:32:15 UTC
Woops, reopened since this isn't in CVS yet :-)
Comment 13 Ben FrantzDale 2002-07-12 02:43:46 UTC
kfv101@psu.edu pointed out that this bug is related to bug 79787. I
tend to agree. Perhaps that means that the patch posted above will fix
that bug?
Comment 14 Kevin Vandersloot 2002-07-12 21:00:56 UTC
Ben: can you try out the patch also?
Comment 15 Kevin Vandersloot 2002-07-12 21:04:02 UTC
Deepa: I don't think the applet should update when the section changes
in the prefs dialog. That means that keyboard users will get an update
every time they navigate with the arrow buttons. 

Otherwise I'm going to test the patch locally for a bit, before I commit. 
Comment 16 Kevin Vandersloot 2002-07-17 02:02:13 UTC
Okay, I just commited to cvs a fix based on Deepas patch. If you guys
could test this, that would be great. I'm going to leave this open for
now.
Comment 17 Andrew Sobala 2002-07-17 15:03:18 UTC
Definitely fixed.