Closed (fixed)
Project:
User Registration Notification
Version:
6.x-1.10
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
14 Jun 2009 at 05:22 UTC
Updated:
3 Jan 2014 at 00:29 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
iva2k commentedAttached is a patch that:
1. addresses that issue of "Send to both" giving error message to register user.
2. Removes duplicate email addresses from send to list, if any
3. Removes empty log message that carries no meaning
Comment #2
rmiddle commentedI need to do some testing. I don't see anything wrong with the patch but it will only catch duplicate addresses if you only enter 1 custom address. It will catch dupes but if you enter several emails all separated by comma's it wont work.
Comment #3
rmiddle commentedComment #4
rmiddle commentedComment #5
iva2k commentedOh, I see, you're right - it won't catch duplicates if there are multiple addresses in the "Send notifications to this custom email address:" field. I know how to fix that though. New patch attached. The trick is using preg_split():
Comment #6
iva2k commentedComment #7
rmiddle commentedCommitted to CVS. Will be in the 6.x-1.11.
Thanks
Robert
Comment #8
rmiddle commented