← Back to team overview

vm team mailing list archive

[Bug 422576] Re: Allow empty email fields in mailto URL:s

 

** Changed in: vm
       Status: New => Fix Committed

** Changed in: vm
    Milestone: None => 8.1.0-beta

** Changed in: viewmail
       Status: New => Fix Committed

-- 
Allow empty email fields in mailto URL:s
https://bugs.launchpad.net/bugs/422576
You received this bug notification because you are a member of VM
development team, which is the registrant for VM (defunct).

Status in VM - Goto http://launchpad.net/vm for active development: Fix Committed
Status in VM (View Mail) for Emacs: Fix Committed

Bug description:
Some functions, like "send link" in Firefox, uses mailto URL:s where there is no email address, but other parts like an initial body is provided, like this:

   mailto:?body=http%3A%2F%2Fwww.nongnu.org%2Fviewmail%2F&subject=View%20Mail%20(aka%20VM)%20for%20Emacs

When handing that to vm-mail-to-mailto-url, the entire string up to the subject is used as an address, which obviously isn't what one wants.

In that function, vm-parse is called with the pattern

   "^mailto:\\([^?]+\\)\\??\\|\\([^&]+\\)&?"

If one changed that to allow an empty string as the first match

   "^mailto:\\([^?]*\\)\\??\\|\\([^&]+\\)&?"

it would work correctly.  At least when I tried it with my VM 8.0.7.522, and a quick look in the current 8.0.12 code looks the same.