GNOME Bugzilla – Bug 411125
bad name sorting in "Contacts" for vowel with accents
Last modified: 2007-09-21 06:58:53 UTC
What were you doing when the application crashed? Referente a ordenação dos nomes nos "Contatos", quando coloco uma vogal com ascento, ela vai para o final da lista, tendo como referência, o ascento e não a vogal. Distribution: Ubuntu 6.10 (edgy) Gnome Release: 2.16.1 2006-10-02 (Ubuntu) BugBuddy Version: 2.16.0
sorry, can you please translate this to english? thanks in advance. (gnome 2.18's bug-buddy will have a hint to please report in english language, if possible, so the users will hopefully save some time)
Translation: Regarding name sorting in "Contacts", when I use a vowel with an accent (accented character) the contact goes to the end of the list using the accent as a sorting reference instead of the vowel.
Ubuntu bug on https://bugs.launchpad.net/ubuntu/+source/evolution/+bug/106160 "Binary package hint: evolution Contacts viewed by Phone List view doesn't sort fields correctly. Instead it allways discriminates characters containing diacritic like czech characters ěščřž. ... http://librarian.launchpad.net/7333245/snapshot5.png Screenshot of Evolution Contact List Ubuntu is of version Edgy, evolution is of version 2.8.1-0ubuntu4.1. Screenshot included showing the end of the contact list."
Created attachment 86913 [details] sort contacts Problem confirmed. Look at the name "Éric" out of order.
Created attachment 95119 [details] [review] proposed evo patch for evolution; I looked on all calls for e_str_compare and it's used only in widgets/table, so I think this change is safe.
Instead of chaning e_str_compare(), call e_collate_compare() instead.
s/chaning/changing/
Hm, if you need to fix it in this way, then it should be better to change all .etspec files and use 'compare="collate"' instead of 'compare="string"', but I don't know where should be better to use strcmp than g_utf8_collate, especially with ETable, where it is prepared for utf8, I think. I meant, the code change is much shorter than the proper definition change.
Created attachment 95852 [details] [review] proposed evo patch for evolution's *.etspec files No code change at all, I just changed *.etspec files, it's using compare="collate" instead of compare="string" now.
Patch looks good to me but I'll defer to Srini for approval.
Looks good to me also. I dont see any problem straight away. So commit to head/stable.
Committed to trunk. Committed revision 34289. Committed to gnome-2-20 branch. Committed revision 34290.