processmail rescanall will not send multiple e-mails to the same address

RESOLVED FIXED in Bugzilla 2.16

Status

()

defect
RESOLVED FIXED
16 years ago
7 years ago

People

(Reporter: dkolath, Assigned: justdave)

Tracking

({dataloss})

unspecified
Bugzilla 2.16
Bug Flags:
approval +

Details

(Whiteboard: [fixed in 2.16.4] [does not affect trunk])

Attachments

(1 attachment)

User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.0)
Build Identifier: 

When running processmail rescanall %seen and @sentmail are being declared as 
global variables.

The rescanall loop runs though each issue that needs mail sent.  When a user 
received an email $sent{'person'} gets set to 1.

When moving on to the next issue %sent never gets re-initialized.  So 
processing the next issue, if the user received an e-mail for a previous issue 
they will not receive any more e-mails.

Reproducible: Always

Steps to Reproduce:
1.execute processmail recanall where the same user needs to receive mail on 
multiple issues.
2.
3.

Actual Results:  
Only a single e-mail was received.

Expected Results:  
It should sent out an e-mail for every issue that you are supposed to be 
notified on.
verified in 2.16.3

This does not affect the cvs tip, "rescanall" got a complete rewrite in 2.17.4
and is now done via sanitycheck.cgi and BugMail.pm, and it does properly
initialize %seen for each bug.
Assignee: preed → justdave
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: dataloss
OS: Windows 2000 → All
Hardware: PC → All
Whiteboard: [wanted for 2.16.4] [does not affect trunk]
Target Milestone: --- → Bugzilla 2.16
Comment on attachment 131627 [details] [diff] [review]
Patch against 2.16 branch

You didn't ask, but I saw the bugmail fly by...

r=preed
Attachment #131627 - Flags: review+
Comment on attachment 131627 [details] [diff] [review]
Patch against 2.16 branch

anyone have a 2.16.x system they can try this on?
Attachment #131627 - Flags: review?
Comment on attachment 131627 [details] [diff] [review]
Patch against 2.16 branch

er, that's what I get for leaving the window open too long. :)
Attachment #131627 - Flags: review?
Flags: approval+
Checking in processmail;
/cvsroot/mozilla/webtools/bugzilla/Attic/processmail,v  <--  processmail
new revision: 1.81.2.1; previous revision: 1.81
done
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Whiteboard: [wanted for 2.16.4] [does not affect trunk] → [fixed in 2.16.4] [does not affect trunk]
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.