183388 - email not sent to new assignee or new qa contact when only "added/removed from capacity" selected

Status: RESOLVED FIXED

(Bugzilla :: Email Notifications, defect)

Description

[Mark Drew]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.2b) Gecko/20021016
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.2b) Gecko/20021016

If a user sets email prefs to ONLY send email when he is added/removed from
assignee, he will NOT get an e-mail if an existing bug is reassigned to him. 
(He will, however, get an e-mail if an existing bug for which he is the assignee
is reassigned to someone else.)  The same thing is true for the qa_contact
relationship to a bug.

Reproducible: Always

Steps to Reproduce:
1. Set your e-mail prefs to ONLY send e-mail when you are added/removed from
assignee (or qa contact)
2. Ensure that NO OTHER email prefs are selected.
3. Reassign an existing bug (not new entry) that is owned by someone else to
yourself.  (Ensure that your email prefs are such that you will receive e-mail
notifications of changes that you make.)
4. Notice that you do not receive an e-mail.

Actual Results:  
I did not receive an e-mail

Expected Results:  
It should have sent me an e-mail since I was added to the capacity of Owner (or
qa contact).

Comment 1

[Mark Drew]

Attachment: patch for processmail

I suppose this is actually two bugs (assignee and qa_contact).	I believe I
have fixed both in this patch.

The assignee problem is the result of the db field "AssignedTo" being referred
to incorrectly as "Owner".

The qa contact problem is the result of incorrect case.  The db field is
"QAContact", but the hash key is (should be) "QAcontact" (notice the case).

Comment 2

[Jacob Steenhagen]

Comment on attachment 108146 [details] [diff] [review]
patch for processmail

Looks good to me...
r=jake

Comment 3

[Jacob Steenhagen]

-> patch author

Comment 4

[Dave Miller [:justdave]]

Jake, can you check it in?

Comment 5

[Jacob Steenhagen]

Checking in processmail;
/cvsroot/mozilla/webtools/bugzilla/processmail,v  <--  processmail
new revision: 1.94; previous revision: 1.93
done