Evolution mailing list does not render accents correctly

Thanks for the bug report. This particular bug has already been reported, but feel free to report any other bugs you find. That might be fixed in hardy

Hi

What is the duplicate you are talking about? The bug #146184 I am refering to is regarding html escape characters (> becomes > etc ...) not accents.

This bug indeed is a separate issue from the one mentioned above. It still exists in Karmic.
I will report back once I've upgraded to Lucid after the release.

I was just able to test on Lucid Beta2 current incl. upgrades and this issue persists. I've attached a screen shot of the Contact view.
It seems that the names are rendered correctly in some instances and incorrectly in others:

Correct:
- Contact Index
- Contact View

Incorrect:
- Contact list: Members view
- Expanded Contact list in To: field of Mail (note there is another issue that expanding a list inserts the expanded contents but does not remove the list, making the entries redundant, but I'll check for a different bug report for that).

Confirming that this bug still exists in Lucid RC.
Surprisingly I haven't found a corresponding bug upstream.
I'll try to reproduce this in other distributions.

I have verified that this bug exists in
Fedora 13 beta and https://launchpad.net/~francesco-marella
which are both current 2.30.1.* versions of evolution.

I've poked around a bit and tried to fix this in a VM and came up with this patch that works for me:

Return formatted address in e_destination_get_address

The documentation of this method claims to return the formatted name
yet it previously encoded them in quotable strings. This patch fixes
this which results in strings containing non ASCII chracters to be
rendered correctly in the UI. This also changes the vCard output but
vCard 3.0 specifies that quotable-printable inline ecnodings have been
eliminated. In any case this quoting should be handled in
e_destination_export_to_vcard_attribute instead.

I've sent a mail upstream mailing list also... I haven't file a bug report though... and I'm not sure about the formal procedures.

Hi David,

That's great news. Can you please add the link to the upstream bug report (is it on gnome.org ?) with link "Also affects project" ?

Hello sebek,

actually, I haven't filed a report yet. I've only posted the patch to the list. As soon as it's in the archives I'll link to it.

I find the bugs reported against evolution overwhelming already:
https://bugzilla.gnome.org/buglist.cgi?query_format=specific&order=relevance+desc&bug_status=__open__&product=Evolution&content=
3863 bugs found.

The need some real triaging support here an I fear adding a bug report for something that has a patch may add unnecessary burden. In case I don't get a reply in a few days (or they ask me to) I'll open one a link to it here.

It seems that my mail hasn't hit the archive yet, probably because I'm not subscribed and a moderator has to approve it first.

I have now submitted a bug report upstream and included the patch. I also hope I got all the references correct now.

So now that this fix is in upstream, what's the correct procedure to get this into Ubuntu (& Debian) distributions?

Should a Debian bug report be created?

the bug has been fixed in git now

Didier do you think you could have a go at backporting this easy change to lucid?

here is the cherry-pick from upstream, with package in -proposed.

Pushed in lucid-proposed:

 evolution-data-server (2.28.3.1-0ubuntu3) lucid-proposed; urgency=low
 .
   * 92_git_return_formatted_address.patch:
     - upstream git change to render correctly non ASCII characters in the UI
       in the address field. (LP: #229187)

Test case:
1. Install the -proposed package
2. Create a mailing list (shift+ctrl+L from the address book) and you select a accentuated name from your address book to be included in the mailing list.
3. Check that it's rendered correctly in the contact selector (not in the mailing list view).

Accepted evolution-data-server into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Hello Martin,

I can confirm that the issue is fixed with the packages in -proposed.

Thank you Martin, Didier and Sebastien.
David

Hi
I confirm the bug fix, but it does not affect the old lists, only those created after the update.

Thanks.

This bug was fixed in the package evolution-data-server - 2.28.3.1-0ubuntu3

---------------
evolution-data-server (2.28.3.1-0ubuntu3) lucid-proposed; urgency=low

  * 92_git_return_formatted_address.patch:
    - upstream git change to render correctly non ASCII characters in the UI
      in the address field. (LP: #229187)
 -- Didier Roche <email address hidden> Mon, 07 Jun 2010 21:53:35 +0200

it's likely that the update created bug #596531 or bug #595867

reopening the lucid task as the update in -proposed revert the change due to bugs #596531 and #595867

This bug was fixed in the package evolution-data-server - 2.30.2-0ubuntu1

---------------
evolution-data-server (2.30.2-0ubuntu1) maverick; urgency=low

  * New upstream release:
    - Mail inline parser doesn't always work (Milan Crha) (LP: #219104)
    - Corruption of mailbox and can't expunge trash (Milan Crha)
      (LP: #286497, #325157)
    - WebDAV addressbook is too strict about content-type (Petr Tomasek)
    - Handle server's resource rename on WebDAV PUT response (Milan Crha)
    - Use NSS SQLite database, if available (Craig Ringer)
    - Fix NULL return without exception on krb5-auth-dialog
      failure (David Woodhouse)
    - Categories not translated in menus (Matthew Barnes) (LP: #443215)
    - Fix translation domain for default categories (Gert Michael Kulyk)
    - Mishandling of '@' in WebDAV contacts username (Milan Crha)
      (LP: #497146)
    - Use CRLF in signed & encrypted S/MIME messages
    - Remove superfluous space in flags in APPEND command (David Woodhouse)
    - Crash in gnome_keyring_item_info_free from e-passwords.c (Milan Crha)
    - Fix GSSAPI (and other SASL) authentication for imapx (David Woodhouse)
    - Teach camel_sexp_to_sql_sexp about no-arg (match-all)
      (Matt McCutchen 2)
    - Contacts are not displayed in GW Addressbook (Bharath Acharya)
    - Don't use an unitialized key in folder_changed_remove_uid
      (Claudio Saavedra)
    - Evolution didn't quit due to deadlock (Milan Crha)
    - Crash of e-addressbook-factory on second start of evolution (Milan Crha)
    - Fix translation of birthday and anniversary events (Gert Michael Kulyk)
    - Return formatted address in e_destination_get_address (David Ayers)
      (LP: #229187)
    - Add imap-over-custom-command support for imapx (David Woodhouse)
    - Fix fetch of large mails from broken servers like Exchange
      (David Woodhouse)
    - Fix double command ->complete() in imapx_command_start_next()
      (David Woodhouse)
    - Fix double-free of is->literal in cancel_all_jobs() (David Woodhouse)
    - use 'UID FETCH' instead of 'FETCH' for scan_changes (David Woodhouse)
    - Remove GroupWise debugging messages. (Matthew Barnes)
    - Bump libedataserver soname twice. (Matthew Barnes)
    - Check EBook DBus error names without underscores too (Milan Crha)
    - Create a section in the name selector dialog in its own function
      (Federico Mena Quintero)
    - Create the configdir instead of bailing out if it does not exist
      (Fridrich Štrba)
    - Don't set a default vertical size for the name selector window
      (Federico Mena Quintero)
    - Fix locking issues in cancel_all_jobs() (David Woodhouse)
    - Fix pointlessly narrow default width in the name selection dialog
    - Name the pointless label at the top, so it can be disabled for MeeGo
      (Michael Meeks)
    - Fix tagprefix for multiple accounts. (David Woodhouse)
    - Fix typo in patch for bug on WebDAV (Milan Crha)
    - Put the name selector's section labels in the same row as the
      treeviews (Federico Mena Quintero)
    - Redo the container hierarchy in the name selector dialog
      (Federico Mena Quintero)
    - Remove an unused label in the name selector ...

revert in maverick as well as per issues ^

The evolution commit to fix the issue http://git.gnome.org/browse/evolution/commit/?id=9a8b5041b60bc5136dd21efb2e07b1cb172bce12

Fixed in maverick, will backport it this week on lucid.

I'd love to validate this for 2.28.3-0ubuntu10.1 supposedly on lucid-proposed but I believe the was some issue with the upload or it hasn't propagated yet. I've activated proposed in the software sources (and ran aptitude update) and also attempted to install via aptitude install evolution/lucid-proposed.

ayers@vmlucid:~$ LANG=C apt-cache policy evolution
evolution:
  Installed: 2.28.3-0ubuntu10
  Candidate: 2.28.3-0ubuntu10
  Version table:
 *** 2.28.3-0ubuntu10 0
        500 http://at.archive.ubuntu.com/ubuntu/ lucid-updates/main Packages
        100 /var/lib/dpkg/status
     2.28.3-0ubuntu9 0
        500 http://at.archive.ubuntu.com/ubuntu/ lucid/main Packages

I'll keep checking...

You will get an update here when the package is accepted in -proposed.

Accepted evolution into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

I now have the following versions installed:
evolution:
  Installed: 2.28.3-0ubuntu10.1
  Candidate: 2.28.3-0ubuntu10.1
  Version table:
 *** 2.28.3-0ubuntu10.1 0
        500 http://at.archive.ubuntu.com/ubuntu/ lucid-proposed/main Packages
        100 /var/lib/dpkg/status
     2.28.3-0ubuntu10 0
        500 http://at.archive.ubuntu.com/ubuntu/ lucid-updates/main Packages
     2.28.3-0ubuntu9 0
        500 http://at.archive.ubuntu.com/ubuntu/ lucid/main Packages

ayers@vmlucid:~$ LANG=C apt-cache policy evolution-data-server
evolution-data-server:
  Installed: 2.28.3.1-0ubuntu5
  Candidate: 2.28.3.1-0ubuntu5
  Version table:
 *** 2.28.3.1-0ubuntu5 0
        500 http://at.archive.ubuntu.com/ubuntu/ lucid-updates/main Packages
        100 /var/lib/dpkg/status
     2.28.3.1-0ubuntu2 0
        500 http://at.archive.ubuntu.com/ubuntu/ lucid/main Packages

When I first associate a contact with accents with a mailing list, the contact is rendered correctly. (This is different than the evolution version from lucid-updates (2.28.3-0ubuntu10)) But if I close the list and open the Contact List / Members View again, the contact is again rendered as quoted-printables.

Expanding the contact list in an e-mail header will also expand the name as quoted-printables.

Error while Fetching Mail.

Cannot append message to mbox file: /home/rakesh/.evolution/mail/local/Inbox: File too large

NACK on 2.28.3-0ubuntu10.1 in -proposed.

Mailing lists and expanded contact list in email header still show the issue.

Got overwritten by another SRU, and failed verification.

I'd like to note that I've verified that this issue continues to exist on:
Lucid 2.28.3-0ubuntu10.2
Maverick 2.30.3-1ubuntu7.3
Natty 2.32.2-0ubuntu4
(I also verified that it exists in Fedora 15 Alpha with evolution 2.91.6.2)

I've also reopened the upstream bug report.

David you said that you reopened the upstream bug report, which one? the only bug watch there point to a closed bug

Thanks Sebastien... I'm completely confused. I still remember parts of the comment's I added when I meant to reopen the upstream case and I was sure I did, but obviously I didn't... well I reopened it now.

Thanks for the heads up!

I don't think I'll have the time to investigate that one. It was in oneiric with evolution for GNOME 3. However, the changes are quite big between the version in natty and evolution 3. So I'm not sure it will be easy to backport the fix.

If someone is interested, looking if http://bugzilla-attachments.gnome.org/attachment.cgi?id=187755 can be backported and then subscribe ubuntu-sponsor to sponsor the fix in lucid will be nice :)

Ok, seemed that fixing it directly would take less time finally, hoping that it's fixing the issue in lucid this time.

SRU team ack. Please accept into lucid-proposed.

Hello sebek, or anyone else affected,

Accepted evolution into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Hello Martin,

I can confirm that this fixes the main issue! Thanks!

Yet it is possibly incomplete:
If you add the list containing escaped characters to a recipient field and the expand it explicitly, it is still expanded to the escaped name.

I can't seem to identify the upstream fix but it should be in libedataserverui/e-name-selector-entry.c popup_activate_inline_expand in evolution data server.

Cheers,
David

This bug was fixed in the package evolution - 2.28.3-0ubuntu10.3

---------------
evolution (2.28.3-0ubuntu10.3) lucid-proposed; urgency=low

  * debian/patches/95_git_fix_mailing_list_accents.patch:
    - fix evolution mailing list does not render accents correctly
      (LP: #229187)
 -- Didier Roche <email address hidden> Fri, 24 Jun 2011 18:36:28 +0200

Is there any chance that this issue will be fixed in Maverick and Natty?
(I haven't been able to install Oneiric on any VM/via test drive yet due to bugs in the installer so I haven't tested it there yet)

those are non LTS version and the issue is a small one and not a regression, we can probably sponsor an update if somebody works on it but it's not likely to be high enough on the priority list to get worked by the Desktop Team directly before Oneiric