| Summary | Protect against notifying email recipients twice or more |
| Queue | Whups |
| Type | Enhancement |
| State | Resolved |
| Priority | 1. Low |
| Owners | chuck (at) horde (dot) org |
| Requester | thomas.jarosch (at) intra2net (dot) com |
| Created | 06/06/2008 (6360 days ago) |
| Due | |
| Updated | 06/07/2008 (6359 days ago) |
| Assigned | |
| Resolved | 06/07/2008 (6359 days ago) |
| Milestone | |
| Patch | No |
Assigned to Chuck Hagenbuch
State ⇒ Resolved
http://cvs.horde.org/diff.php/whups/lib/Driver.php?r1=1.189&r2=1.190&ty=u
Priority ⇒ 1. Low
New Attachment: whups-notify-only-once.patch
Patch ⇒ No
Milestone ⇒
Queue ⇒ Whups
Summary ⇒ Protect against notifying email recipients twice or more
Type ⇒ Enhancement
State ⇒ New
I've been receiving email notifies for
#6837twice for every comment.After some investigation it turned out I sometimes used my bugtracker
account and sometimes my plain email address to reply to comments.
Attached patch protects against notifying the same email recipient twice.
I also fixed $full_name handling, which looked broken to me:
$full_name gets updated -after- it has been used and used the
"mailbox" instead of "personal" field of MIME::parseAddressList(). If
we are unable to determine a full name, we fallback to the email
address.
The code is untested as I don't have a whups/SQL database running over here,
so please be kind to syntax errors, if any :-)