"I'm added to or removed from this capacity" for "CCed" doesn't work correctly if several users are added or removed at once from the CC list

RESOLVED FIXED in Bugzilla 3.0

Status

()

Bugzilla
Email Notifications
--
major
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: Gavin, Assigned: Frédéric Buclin)

Tracking

2.20
Bugzilla 3.0
Bug Flags:
approval +
approval3.0 +
blocking3.0.3 +

Details

Attachments

(3 attachments)

Created attachment 279505 [details]
Gavin's Bugzilla Email Prefs

Based on some ad-hoc testing, it looks as though "The CC field changes" preference on the current instance of b.m.o overriddes "I'm added or removed from this capacity", but only for some people (it seems to work for me, but not work for Phil). This means that you can't disable getting mail for other people's CC changes, but still get mail when you're added or removed from the list.

When Ryan CCed me to bug 234982 (a bug that I had no previous relationship to), I got mail for the CC addition that said "You are receiving this mail because: You are on the CC list for the bug." (note that I don't have "CCed x The CC field changes" checked). When I CCed Phil on bug 394791, he didn't get mail. He then checked "CCed x The CC field changes", and then I CCed him on bug 394775. He got bugmail for that change. He had "CCed x I'm added or removed from this capacity" checked in both cases.

Here's a screenshot of my bugzilla email prefs.
I just CCed mkanat and philor (and made no other changes) - mkanat got mail, philor didn't.
This is potentially related to bug 366181.
Created attachment 279510 [details]
Phil's prefs

No difference that should be significant, the only difference that shouldn't be, but might, being that I don't have anything at all in the "The CC field changes" row checked. Neither of Gavin's checks applied when he did get it, but maybe it's the condition of nothing, rather than the condition of matching, that matters?
I didn't get any mail when I was added to bug 396830's CC list (I only got mail when Jesse posted a comment). I think this is caused by people adding more than one person to the CC list at a time (Jesse and I were added at the same time). Is the check for whether "I'm being added or removed from this capacity" exact, such that adding me+someone else doesn't trigger it?

This is a fairly major issue, because myself and others are missing bugmail that we should be getting.
Flags: blocking3.0.3?

Comment 5

10 years ago
Yeah, this is important enough to be a blocker.
Flags: blocking3.0.3? → blocking3.0.3+
(Assignee)

Updated

10 years ago
Target Milestone: --- → Bugzilla 3.0
(Assignee)

Comment 6

10 years ago
Created attachment 282015 [details] [diff] [review]
patch, v1

Gavin was right. The problem only occurs if you add/remove several users at once to/from the CC list, due to an incorrect regexp. This looks like a regression due to bug 73665, meaning that Bugzilla 2.20 and 2.22 are also affected by this bug.
Assignee: email-notifications → LpSolit
Status: NEW → ASSIGNED
Attachment #282015 - Flags: review?(bugzilla-mozilla)
(Assignee)

Updated

10 years ago
Summary: "I'm added to or removed from this capacity " for "CCed" is overridden by "The CC field changes" → "I'm added to or removed from this capacity" for "CCed" doesn't work correctly if several users are added or removed at once from the CC list
Version: 3.0.1 → 2.20
Comment on attachment 282015 [details] [diff] [review]
patch, v1

r=Wurblzap provided you tested (I didn't). Nit: don't we want to allow whitespace before the following comma, too? I think so. (Fix this on check-in, if you want.)
Attachment #282015 - Flags: review?(bugzilla-mozilla) → review+
(Assignee)

Comment 8

10 years ago
(In reply to comment #7)
> (From update of attachment 282015 [details] [diff] [review])
> r=Wurblzap provided you tested (I didn't).

Yes, I tested it, on both 3.1.2 and 3.0.2.


> Nit: don't we want to allow
> whitespace before the following comma, too?

No, because process_bug.cgi (3.0.x) and Bug.pm (3.1.x) both use join(", ", @list), and so it's a bug if a whitespace appears *after* the email address.
Flags: approval3.0+
Flags: approval+
(Assignee)

Comment 9

10 years ago
tip:

Checking in Bugzilla/User.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/User.pm,v  <--  User.pm
new revision: 1.160; previous revision: 1.159
done

3.0.2:

Checking in Bugzilla/User.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/User.pm,v  <--  User.pm
new revision: 1.148.2.4; previous revision: 1.148.2.3
done
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
(Assignee)

Updated

10 years ago
Keywords: relnote

Comment 10

9 years ago
Removing relnote keyword as this was fixed and probably relnoted in a 3.0.x release long ago.
Keywords: relnote
You need to log in before you can comment on or make changes to this bug.