Review Request 119221: Don't rewrite mailto: URLs als mailto://

[Felix Geyer]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119221/
-----------------------------------------------------------

Review request for kdelibs.


Repository: kdelibs


Description
-------

KUrl::prettyUrl() rewrites mailto: URLs als mailto:// if the "path" is empty.
MUAs don't handle mailto:// URLs correctly.

mailto:test at example.com
and
mailto:?to=test at example.com
are both valid and semantically equivalent.

KUrl::prettyUrl() converts the latter one to mailto://?to=test@example.com

This works fine:
kde-open mailto:test at example.com

but this fails:
kde-open mailto:?to=test at example.com

For example with Thunderbird it creates a mail with:
To: //
To: test at example.com

The proposed change special cases mailto so it always uses mailto: instead of mailto://.


Diffs
-----

  kdecore/io/kurl.cpp 8c491f7 
  kdecore/tests/kurltest.cpp 1fd204f 

Diff: https://git.reviewboard.kde.org/r/119221/diff/


Testing
-------

Tested with kdelibs 4.13.2 and Thunderbird, Evolution and KMail.


Thanks,

Felix Geyer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140710/b3d7709e/attachment.htm>