Add group-based lists to whining

RESOLVED FIXED in Bugzilla 2.20

Status

()

Bugzilla
Whining
P3
enhancement
RESOLVED FIXED
13 years ago
3 years ago

People

(Reporter: Joel Peshkin, Assigned: Erik Stambaugh)

Tracking

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

Details

Attachments

(1 attachment)

(Reporter)

Description

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 Search.pm) or the author (need a new
optional "author" param to Search.pm) has permission to reference the group pronoun.
(Reporter)

Updated

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

Updated

13 years ago
Blocks: 255562
(Assignee)

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
things.

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 Search.pm and added a
new file, Group.pm, 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)
(Assignee)

Updated

13 years ago
Status: NEW → ASSIGNED
Flags: blocking2.20?
(Reporter)

Updated

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
(Reporter)

Updated

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 checksetup.pl and Bugzilla/Constants.pm and just remove the conflict
markers (note that there's two sets of markers in Constants.pm).

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

Comment 7

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

Status: ASSIGNED → RESOLVED
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.