Add group-based lists to whining

RESOLVED FIXED in Bugzilla 2.20



13 years ago
3 years ago


(Reporter: Joel Peshkin, Assigned: Erik Stambaugh)


Bugzilla 2.20
Dependency tree / graph
Bug Flags:
approval +
blocking2.20 +



(1 attachment)



13 years ago
When defining scheduled whine events, add groups of users as targets of a whine.
Each will still get the report run under his own permissions, but a single event
schedule can cover a whole list of people.

In the same update, the use of group pronouns in whining should be permitted if
EITHER the recipient ("user" param to or the author (need a new
optional "author" param to has permission to reference the group pronoun.


13 years ago
Severity: normal → enhancement
Priority: -- → P3
Target Milestone: --- → Bugzilla 2.20
Component: Query/Bug List → Whining
Bugzilla 2.20 feature set is now frozen as of 15 Sept 2004.  Anything flagged
enhancement that hasn't already landed is being pushed out.  If this bug is
otherwise ready to land, we'll handle it on a case-by-case basis, please set the
blocking2.20 flag to '?' if you think it qualifies.
Target Milestone: Bugzilla 2.20 → Bugzilla 2.22


13 years ago
Blocks: 255562

Comment 2

13 years ago
Created attachment 170285 [details] [diff] [review]
whine at groups

Okay, here it is!

This patch adds group targets for whining, but I snuck in a couple of other

The first thing I did was fix the problem that was reported on bug 185090 after
it was closed, where changes to schedules were not working unless the day/time
of the schedule was altered.  It's a trivial enough change that I figured I
could be a schmuck and roll it in.  I can break it out and put it into a 3-line
patch somewhere else if I need to.

I cleaned up a small amount of the existing code to make it more readable. 
Some of those '$_->[0]'s were getting on my nerves.

Last, I broke Joel's ValidateGroupName function out of and added a
new file,, for group manipulation functions.  I only put the one
function in for now, but I'm sure later on it'll have more.

Dave: I'm going to request blocking2.20 on this because it fixes a fairly major
bug, and I believe any differences in whining between 2.19.1 and 2.20 won't
really hurt anyone, but I'm willing to be argued with on that.
Attachment #170285 - Flags: review?(bugreport)


13 years ago
Flags: blocking2.20?


13 years ago
Attachment #170285 - Flags: review?(justdave)
Attachment #170285 - Flags: review?(bugreport)
Attachment #170285 - Flags: review+
yeah, why not.  It's big enough I want to hold it, but on the other hand, it is
one of our advertised major new features, so we might as well have it working
completely and correctly from the get-go.
Flags: blocking2.20? → blocking2.20+
Target Milestone: Bugzilla 2.22 → Bugzilla 2.20


13 years ago
Flags: approval?
/me drools

I need the second review on here before I approve it, but leaving it in the
queue as it's more likely I'll see it that way :)  I'll come back to this
tomorrow when I have time to play.
Comment on attachment 170285 [details] [diff] [review]
whine at groups

OK, this fails to apply cleanly, but the two conflicts are places where things
were added to the same place.

Procedure to unbitrot this:

cvs -q up -dP -D2005-01-04
patch -p0 < whineatgroups.patch
cvs -q up -AdP

then edit and Bugzilla/ and just remove the conflict
markers (note that there's two sets of markers in

You might want to update the date in the comment in on checkin as
Attachment #170285 - Flags: review?(justdave) → review+
Note to travis: let Erik do the checkin on this please. :)
Flags: approval? → approval+

Comment 7

13 years ago
Checking in;
/cvsroot/mozilla/webtools/bugzilla/,v  <--
new revision: 1.347; previous revision: 1.346
Checking in editgroups.cgi;
/cvsroot/mozilla/webtools/bugzilla/editgroups.cgi,v  <--  editgroups.cgi
new revision: 1.49; previous revision: 1.48
Checking in editwhines.cgi;
/cvsroot/mozilla/webtools/bugzilla/editwhines.cgi,v  <--  editwhines.cgi
new revision: 1.3; previous revision: 1.2
Checking in;
/cvsroot/mozilla/webtools/bugzilla/,v  <--
new revision: 1.7; previous revision: 1.6
Checking in Bugzilla/;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/,v  <--
new revision: 1.18; previous revision: 1.17
Checking in Bugzilla/;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/,v  <--
new revision: 1.78; previous revision: 1.77
Checking in template/en/default/whine/schedule.html.tmpl;
 <--  schedule.html.tmpl
new revision: 1.2; previous revision: 1.1
RCS file: /cvsroot/mozilla/webtools/bugzilla/Bugzilla/,v
Checking in Bugzilla/;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/,v  <--
initial revision: 1.1

Last Resolved: 13 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.