Evolution mailing list does not render accents correctly

Bug #229187 reported by sebek on 2008-05-11
24
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Evolution
Fix Released
Low
evolution (Ubuntu)
Low
Unassigned
Lucid
Low
Didier Roche

Bug Description

Uploaded to lucid-proposed:
evolution (2.28.3-0ubuntu10.3) lucid-proposed

Test case:
1. install the version from -proposed
2. in the same or similar layout than described below, try to answer to a mail address with an accent
3. report here that the accent is correctly displayed in the mail view

-----------------

Under Ubuntu Gutsy, evolution 2.12.1, env var LANG=fr_FR.UTF-8

When creating 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, it is not rendered correctly in the contact selector not in the mailing list view.

Instead of displaying :
"Sébastien ABEILLE <email address hidden>"
it is displaying :
"=?ISO-8859-1?Q?S=E9bastien?= ABEILLE <email address hidden>"

Screenshot attached.

There are no problems to render a accentuated name in a contact entry, problem is only with mailing lists.

I had a look at #146184 which is also a rendering problem in mailing list display, but with html special characters, fixed in 2.21.90, which I can not test currently.

Would be nice if someone could try to reproduce it with Hardy see if it is still there.

sebek (sebeeek) wrote :
Sebastien Bacher (seb128) wrote :

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

Changed in evolution:
assignee: nobody → desktop-bugs
importance: Undecided → Low
status: New → Invalid
sebek (sebeeek) wrote :

Hi

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

David Ayers (ayers) wrote :

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.

David Ayers (ayers) wrote :

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).

David Ayers (ayers) wrote :

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.

Changed in evolution (Ubuntu):
status: Invalid → Confirmed
David Ayers (ayers) wrote :

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.

David Ayers (ayers) wrote :

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.

sebek (sebeeek) wrote :

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" ?

David Ayers (ayers) wrote :

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.

David Ayers (ayers) wrote :

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.

Changed in evolution:
status: Unknown → New
Changed in evolution:
status: New → Incomplete
Changed in evolution:
status: Incomplete → New
Changed in evolution:
status: New → Fix Released
David Ayers (ayers) wrote :

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?

Sebastien Bacher (seb128) wrote :

the bug has been fixed in git now

affects: evolution (Ubuntu) → evolution-data-server (Ubuntu)
Changed in evolution-data-server (Ubuntu):
status: Confirmed → Fix Committed
Sebastien Bacher (seb128) wrote :

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

Changed in evolution-data-server (Ubuntu Lucid):
assignee: nobody → Didier Roche (didrocks)
importance: Undecided → Low
status: New → Triaged
Didier Roche (didrocks) wrote :

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

Didier Roche (didrocks) wrote :

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!

Changed in evolution-data-server (Ubuntu Lucid):
status: Triaged → Fix Committed
tags: added: verification-needed
David Ayers (ayers) wrote :

Hello Martin,

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

Thank you Martin, Didier and Sebastien.
David

Martin Pitt (pitti) on 2010-06-10
tags: added: verification-done
removed: verification-needed
Florian Neyret (fneyret) wrote :

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

Thanks.

Launchpad Janitor (janitor) wrote :

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

Changed in evolution-data-server (Ubuntu Lucid):
status: Fix Committed → Fix Released
Sebastien Bacher (seb128) wrote :

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

Didier Roche (didrocks) wrote :

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

Changed in evolution-data-server (Ubuntu Lucid):
status: Fix Released → Fix Committed
Launchpad Janitor (janitor) wrote :
Download full text (3.9 KiB)

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 ...

Read more...

Changed in evolution-data-server (Ubuntu):
status: Fix Committed → Fix Released
Didier Roche (didrocks) wrote :

revert in maverick as well as per issues ^

Changed in evolution-data-server (Ubuntu):
status: Fix Released → Triaged
Sebastien Bacher (seb128) wrote :
affects: evolution-data-server (Ubuntu) → evolution (Ubuntu)
Changed in evolution (Ubuntu):
assignee: Ubuntu Desktop Bugs (desktop-bugs) → nobody
status: Triaged → Fix Committed
Changed in evolution (Ubuntu Lucid):
status: Fix Committed → Triaged
Changed in evolution:
importance: Unknown → Low
Didier Roche (didrocks) wrote :

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

Changed in evolution (Ubuntu):
status: Fix Committed → Fix Released
Didier Roche (didrocks) on 2010-11-03
description: updated
David Ayers (ayers) wrote :

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...

Didier Roche (didrocks) wrote :

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

Martin Pitt (pitti) wrote :

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!

Changed in evolution (Ubuntu Lucid):
status: Triaged → Fix Committed
tags: removed: verification-done
tags: added: verification-needed
David Ayers (ayers) wrote :

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.

rakesh (rakeshsoans) wrote :

Error while Fetching Mail.

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

Marc Deslauriers (mdeslaur) wrote :

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

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

Martin Pitt (pitti) wrote :

Got overwritten by another SRU, and failed verification.

tags: added: verification-failed
removed: verification-needed
Changed in evolution (Ubuntu Lucid):
status: Fix Committed → Triaged
David Ayers (ayers) wrote :

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.

Sebastien Bacher (seb128) wrote :

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

David Ayers (ayers) wrote :

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!

Changed in evolution:
status: Fix Released → Confirmed
Changed in evolution:
status: Confirmed → Fix Released
Didier Roche (didrocks) wrote :

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 :)

Changed in evolution (Ubuntu Lucid):
assignee: Didier Roche (didrocks) → nobody
Changed in evolution (Ubuntu):
status: Fix Released → Triaged
status: Triaged → Fix Released
Didier Roche (didrocks) wrote :

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

Changed in evolution (Ubuntu Lucid):
status: Triaged → Fix Committed
assignee: nobody → Didier Roche (didrocks)
Didier Roche (didrocks) on 2011-06-24
description: updated
Chris Halse Rogers (raof) wrote :

SRU team ack. Please accept into lucid-proposed.

Martin Pitt (pitti) wrote :

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!

tags: removed: verification-failed
tags: added: verification-needed
David Ayers (ayers) wrote :

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

Martin Pitt (pitti) on 2011-07-04
tags: added: verification-done
removed: verification-needed
Launchpad Janitor (janitor) wrote :

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

Changed in evolution (Ubuntu Lucid):
status: Fix Committed → Fix Released
David Ayers (ayers) wrote :

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)

Sebastien Bacher (seb128) wrote :

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

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.