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 485700 - evolution: URLs with ampersand ("&") don't work
evolution: URLs with ampersand ("&") don't work
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.12.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Milan Crha
Evolution QA team
: 486852 489452 489984 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2007-10-11 11:18 UTC by Sebastien Bacher
Modified: 2008-07-17 10:51 UTC
See Also:
GNOME target: ---
GNOME version: 2.19/2.20


Attachments
proposed gtkhtml patch (7.85 KB, patch)
2007-10-18 11:05 UTC, Milan Crha
none Details | Review
proposed gtkhtml patch ][ (8.75 KB, patch)
2007-10-24 15:28 UTC, Milan Crha
committed Details | Review

Description Sebastien Bacher 2007-10-11 11:18:36 UTC
The bug has been opened on https://bugs.launchpad.net/bugs/149964

"...
I am using Gutsy Gibbon. Since a few weeks some URLs in e-mails get somehow corrupted when clicked on.

I can repro this as follows with the e-mails I regularily get from an eBay search subscription:

1. open evolution
2. open an arbitrary e-mail received from eBay
3. click on an arbitrary link to an eBay item
4. result: eBay prints "ungültiger Artikel" (in english: "invalid item"), since the URL is no longer correct.

This is a bug of evolution. I have tested exactly the same e-mail (from my central IMAP mailbox) using Thunderbird and it works perfectly. I also forwarded this e-mail to my Windows machine at work and also no problem.

Here is a sample URL (shown in the firefox window) when clicked on the link from Thunderbird:

http://cgi.ebay.de/ws/eBayISAPI.dll?ViewItem&item=150168307117&ssPageName=ADME:B:SS:DE:1

And here is the result I get when clicked on the link from evolution leading to the "invalid item" message in eBay:

http://cgi.ebay.de/ws/eBayISAPI.dll?ViewItem&item=150168307117&ssPageName=ADME:B:SS:DE:1

Do you see the difference ? The character "&" is changed to "&", i.e. the HTML equivalent of the ampersand sign, which obviously no longer works and leads to a malformed URL.
...
http://launchpadlibrarian.net/9858549/Meine%20eBay-Suche%3A%20Sinclair%20%28evolution%29.eml

attached is the mail exported by evolution - hope this is possible to import. Otherwise I can forward you this e-mail (using Thunderbird and/or evolution ?).

The preferred browser command is "use the preferences of the browser". The default browser is, of course, firefox.
..."
Comment 1 Jeffrey Stedfast 2007-10-11 14:57:19 UTC
this works fine in evo 2.10.1, so I guess the bug must have been introduced somewhere in 2.11... hope this helps tracking it down.
Comment 2 André Klapper 2007-10-15 23:21:06 UTC
probably already filed under gtkhtml
Comment 3 Sebastien Bacher 2007-10-16 08:10:16 UTC
I looked to the gtkhtml bugs without sending this one there was no obvious duplicate there
Comment 4 tjb 2007-10-16 12:46:49 UTC
I'm seeing this too on Fedora 8. The Fedora bug report is here:

https://bugzilla.redhat.com/show_bug.cgi?id=313061

It includes an mbox format ebay email showing the problem.
Comment 5 Milan Crha 2007-10-17 16:02:46 UTC
probably because of Bug 270487
Comment 6 Milan Crha 2007-10-18 11:05:36 UTC
Created attachment 97412 [details] [review]
proposed gtkhtml patch

for gtkhtml;

I added new files to gtkhtml, src/htmlutils.[h/c], to store helper functions.
When testing, please forget old mails, where are not properly escaped ampersands in url (href), there is not much what one can do, but when you will add new urls, then it will be internally properly escaped and when passing to ON_URL or LINK_CLICKED then it will be unescaped. (I hope it makes sense a bit.)
Comment 7 Milan Crha 2007-10-24 08:38:29 UTC
*** Bug 489452 has been marked as a duplicate of this bug. ***
Comment 8 Milan Crha 2007-10-24 15:28:07 UTC
Created attachment 97783 [details] [review]
proposed gtkhtml patch ][

for gtkhtml;

As Frederic pointed in bug 489452, patch here doesn't cover all what it should, so I'm attaching here improved patch to fix issue he mentioned. Thanks for help, Frederic.
Comment 9 Frederic Peters 2007-10-25 07:56:22 UTC
Seems to fix all issues, thanks Milan.
Comment 10 Srinivasa Ragavan 2007-10-26 06:58:07 UTC
Milan commit to stable/head.
Comment 11 Milan Crha 2007-10-26 07:55:46 UTC
Committed to trunk. Committed revision 8605.
Committed to stable. Committed revision 8606.
Comment 12 Milan Crha 2008-04-15 13:32:00 UTC
*** Bug 489984 has been marked as a duplicate of this bug. ***
Comment 13 Milan Crha 2008-07-17 10:51:37 UTC
*** Bug 486852 has been marked as a duplicate of this bug. ***