[PATCH] encode mail address when passing via KApplication::invokeMailer

Ingo Klöcker kloecker at kde.org
Sat Jan 8 15:02:43 GMT 2005


On Saturday 08 January 2005 15:53, Oswald Buddenhagen wrote:
> On Sat, Jan 08, 2005 at 03:45:06PM +0100, Ingo Klöcker wrote:
> > This does already break for the simple email address
> >   "User, Joe" <joe.user at kde.org>
> > So please NEVER try to split a list of email addresses with a
> > QStringList::split( ',', foo ).
>
> this is relatively theoretical, as we are mostly talking about
> mailto: urls and hard-coded links which hardly ever include a real
> name.

This is absolutely not theoretical. If you change the formatted name of 
a contact to "Lastname, Firstname" (which is pretty usual in corporate 
environments) then KAddressBook will call invokeMailer() with an 
address in the above form.

> > While improving Tobias patch I also fixed the code which builds the
> > mailto: url. Splitting the comma separated lists of email addresses
> > isn't necessary (according to RFC 2368 - The mailto URL scheme),
> > neither for the path (where the To addresses are put) nor for the
> > queries. Moreover, I thought it might be useful to actually specify
> > the protocol of the URL. Or does KURL automagically guess the right
> > protocol? Even if the path is empty?
>
> i don't care about the details, but if you break %u for mutt, i'll
> make you a head shorter. ;)

Can you then please test this with Mutt? Attached is a tiny HTML-test 
file.

Regards,
Ingo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20050108/f3ca349c/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20050108/f3ca349c/attachment.sig>


More information about the kde-core-devel mailing list