[Rt-devel] patch: Optimize recipients for Notifications Mails

Jesse Vincent jesse at bestpractical.com
Fri Jun 15 08:46:53 EDT 2007


On Jun 15, 2007, at 4:20 AM, Ruslan Zakirov wrote:

> good patch, vote for applying

Ok. Let's aim for 3.6.5?



>
> On 6/15/07, Dirk Pape <pape-rt at inf.fu-berlin.de> wrote:
>> Hello,
>>
>> we found, that if we have ScripActions that notify more than one  
>> watcher
>> group (e.g. "Notify AdminCC and Cc as Comment"), people get more  
>> than one
>> message for the same notification, if they have more than one role  
>> for the
>> ticket (e.g. beeing AdminCc and Cc).
>>
>> This is not necessary at all but only annoying, so I propose  
>> following
>> general patch for Action/SendEmail.pm which removes unnecessary  
>> recipient
>> entries: It removes every Cc already in To and every Bcc if  
>> already in To
>> or Cc.
>>
>> The patch is tested here and behaves like expected. Will this make  
>> it into
>> the core?
>>
>> <http://page.mi.fu-berlin.de/~pape/rt3/patches/rt/ 
>> optimize_recipients.patch>
>>
>> Regards, Dirk.
>> --
>> Dr. Dirk Pape (eAS - Projektleitung Campus Management)
>> Freie Universitaet Berlin
>> Grunewaldstr. 34a, 12165 Berlin
>> Tel. +49 (0)30 838 75143, Fax. +49 (0)30 838 54654
>> _______________________________________________
>> List info: http://lists.bestpractical.com/cgi-bin/mailman/listinfo/ 
>> rt-devel
>>
>
>
> -- 
> Best regards, Ruslan.
> _______________________________________________
> List info: http://lists.bestpractical.com/cgi-bin/mailman/listinfo/ 
> rt-devel
>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 186 bytes
Desc: This is a digitally signed message part
Url : http://lists.bestpractical.com/pipermail/rt-devel/attachments/20070615/2df933fc/PGP.pgp


More information about the Rt-devel mailing list