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 642087 - The order of favorite rooms listed in menu items of Room menu change automatically sometimes
The order of favorite rooms listed in menu items of Room menu change automati...
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: Multi User Chat
unspecified
Other Linux
: Normal normal
: ---
Assigned To: empathy-maint
Depends on:
Blocks:
 
 
Reported: 2011-02-11 05:31 UTC by Chandni Verma
Modified: 2011-05-09 14:21 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Chandni Verma 2011-02-11 05:31:12 UTC
This should not happen and they should remain stable when user has made no change to the list of added rooms (ie. by adding or removing any room)
Comment 1 Chandni Verma 2011-02-25 08:46:48 UTC
I think replacing the G_FILE_MONITOR_EVENT_CHANGES_DONE_HINT isn't justified here as any change either by writing, moving, renaming the file or unmounted filesystem should be checked upon. 
But this is happening even when I do not do any such thing!
Comment 2 Chandni Verma 2011-05-07 20:11:26 UTC
I think I have figured this out.
Presently we are updating chatrooms.xml every time a change is made in any of the properties in chat-rooms, even for those properties which the file doesn't and shouldn't care about, like "tp-chat" which causes all chat-rooms to be reloaded into the main window's Rooms menu(even changing their order sometimes) when a channel is obtained and set.

My branch to fix this annoying behaviour:
https://gitorious.org/glassrose-gnome/empathy/commits/save-chatrooms.xml-only-when-needed
Comment 3 Guillaume Desmottes 2011-05-09 10:09:59 UTC
I'd add a comment explaining that you only watch the properties exported in the saved file. Except that, look good!
Comment 4 Chandni Verma 2011-05-09 12:55:15 UTC
Done :)
Comment 5 Guillaume Desmottes 2011-05-09 14:21:49 UTC
Merged to master; thanks !

This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.