E-mail notification of new attachments

[Sean McAfee]

Gervase Markham <gerv at mozilla.org> wrote:
>Sean McAfee wrote:
>> if ($difftext eq "" && $newcomments eq "") { return; }
>> 
>> How this condition gets met, though, is proving difficult to determine.  So,
>> I'm punting to the list and hoping someone more familiar with the BugMail
>> guts can explain what's happening more quickly.

>Having just rewritten them in bug 73665, I'm fairly familiar with them 
>:-) $difftext is the text to go in mails containing the differences in 
>the bug. Attachment fields aren't included. $newcomments are the new 
>comments. So both of these will be empty for your change.
>
>This line is there as a sanity check to stop mail getting sent when no 
>actual changes have been made. I don't know what the side effects would 
>be if you removed it - try it and see if anyone gets blank "change" 
>emails from Bugzilla.

I will, but, uh... If mail can never be sent when an attachment is attached,
why does BugMail.pm get invoked?


--Sean