Download & Extend

fix: incorrect parsing of 'Send notifications to this custom email address'

Project:User registration notification
Version:7.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review

Issue Summary

A list of emails entered into the user_register_notify_mailto field should be separated by a comma, and zero or more spaces.

But when I enter emails like that, they just dont get delivered.

FIX: So in the .module, replace the two lines like:
$emails = preg_split('/,[\w]+/', variable_get('user_register_notify_mailto', $from));
with:
$emails = preg_split('/,[\s]*/', variable_get('user_register_notify_mailto', $from));

Comments

#1

Status:active» needs review

#2

Here's a patch with boran's changes. I can confirm that they work.

AttachmentSize
user_register_notify-email-address-parsing-1304218.patch 916 bytes
nobody click here