Roundcube composer should filter out 'mailto:' in addresses (for use as protocol handler)

[Tobbi]

It would be great if Roundcube could support external mailto calls. At the moment, when opening a mail address like mailto:test@… with Roundcube, the mailto: protocol still stays together with the mail address. We should filter that out to make it support that.

See ​https://bugzilla.mozilla.org/show_bug.cgi?id=509086 for the Bugzilla bug.

[thomasb]

I don't quite understand what you're asking for. Could give an example?

[Tobbi]

Replying to thomasb:

I don't quite understand what you're asking for. Could give an example?

Firefox is able to support external Webmail clients as mailto handlers ​http://support.mozilla.com/en-US/kb/Changing+the+e-mail+program+used+by+Firefox#Using_webmail_services

There's a way to add webmail services, like GMail, Yahoo! (AND Roundcube), to this list. However, when registering Roundcube as external handler for mailto: links (cf. ​http://www.roundcubeforum.net/roundcube-discussion/1231-mailto-link-opening-roundcube.html), Firefox passes the whole mailto string to roundcube, which is (mailto:mailaddress@…).

Is there a way to filter the mailto: per default so that it doesn't appear in the 'TO' bar at all?

Did you understand now, what I'm pointing to?

[trisk]

The problem is that if the composer is invoked as "?_task=mail&_action=compose&to=mailto:user@…"

The Recipient field has "mailto:user@…" instead of "user@host".

[trisk]

Here's a solution for the problem: Extract the address from the string after "mailto:", and reassign the 'to' parameter. As before, we parse additional parameters in the URI.

Index: program/steps/mail/compose.inc
===================================================================
--- program/steps/mail/compose.inc      (revision 3214)
+++ program/steps/mail/compose.inc      (working copy)
@@ -44,9 +44,11 @@
   
   // process values like "mailto:foo@bar.com?subject=new+message&cc=another"
   if ($_SESSION['compose']['param']['to']) {
-    $mailto = explode('?', $_SESSION['compose']['param']['to']);
+    if (preg_match('/^mailto:(.*)/i', $_SESSION['compose']['param']['to'], $mailto)) {
+      $mailto = explode('?', $mailto[1]);
+      $_SESSION['compose']['param']['to'] = $mailto[0];
+    }
     if (count($mailto) > 1) {
-      $_SESSION['compose']['param']['to'] = $mailto[0];
       parse_str($mailto[1], $query);
       foreach ($query as $f => $val)
         $_SESSION['compose']['param'][$f] = $val;

[alec]

Fixed in [3e8898ef].