vm team mailing list archive
-
vm team
-
Mailing list archive
-
Message #00168
[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.