----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119221/#review62106 -----------------------------------------------------------
This change makes kdelibs consistent with RFC 2368 (which defined the mailto URL scheme). - Scott Kitterman On July 10, 2014, 8:40 p.m., Felix Geyer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119221/ > ----------------------------------------------------------- > > (Updated July 10, 2014, 8:40 p.m.) > > > 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:[email protected] > and > mailto:[email protected] > are both valid and semantically equivalent. > > KUrl::prettyUrl() converts the latter one to mailto://[email protected] > > This works fine: > kde-open mailto:[email protected] > > but this fails: > kde-open mailto:[email protected] > > For example with Thunderbird it creates a mail with: > To: // > To: [email protected] > > 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 > >
