Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-10-15 06:15:43 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix for bug 229800: whineatnews.pl doesn't whine about REOPENED bugs.

Browse Source 
Change whine script to look to REOPENED bugs too, and modified text in
relevant param to account for this. Patch by joseph@picsel.com (Joseph
Heenan). r=kiko, a=myk.
This commit is contained in:
kiko%async.com.br 2004-01-14 13:59:16 +00:00
parent 386f40c13a
commit 7215f26933
2 changed files with 23 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
webtools/bugzilla/defparams.pl
View File

@ -24,6 +24,7 @@
# Jacob Steenhagen <jake@bugzilla.org>
# J. Paul Reed <preed@sigkill.com>
# Bradley Baetz <bbaetz@student.usyd.edu.au>
# Joseph Heenan <joseph@heenan.me.uk>
#
# This file defines all the parameters that we have a GUI to edit within
@ -557,16 +558,16 @@ Configure bugmail: %urlbase%userprefs.cgi?tab=email
{
name => 'whinemail',
desc => 'The email that gets sent to anyone who has a NEW bug that '.
'hasn\'t been touched for more than <b>whinedays</b>. Within ' .
'this text, %email% gets replaced by the offender\'s email ' .
'address. %userid% gets replaced by the offender\'s bugzilla ' .
'login (which, in most installations, is the same as the email ' .
' address.) %<i>anythingelse</i>% gets replaced by the definition ' .
'of that parameter (as defined on this page).<p> It is a good ' .
'idea to make sure this message has a valid From: address, so ' .
'that if the mail bounces, a real person can know that there are ' .
'bugs assigned to an invalid address.',
desc => 'The email that gets sent to anyone who has a NEW or REOPENED ' .
'bug that hasn\'t been touched for more than <b>whinedays</b>. ' .
'Within this text, %email% gets replaced by the offender\'s ' .
'email address. %userid% gets replaced by the offender\'s ' .
'bugzilla login (which, in most installations, is the same as ' .
'the email address.) %<i>anythingelse</i>% gets replaced by the ' .
'definition of that parameter (as defined on this page).<p> It ' .
'is a good idea to make sure this message has a valid From: ' .
'address, so that if the mail bounces, a real person can know '.
'that there are bugs assigned to an invalid address.',
type => 'l',
default => 'From: %maintainer%
To: %email%
@ -578,9 +579,9 @@ You have one or more bugs assigned to you in the Bugzilla
bugsystem (%urlbase%) that require
attention.
All of these bugs are in the NEW state, and have not been touched
in %whinedays% days or more. You need to take a look at them, and
decide on an initial action.
All of these bugs are in the NEW or REOPENED state, and have not
been touched in %whinedays% days or more. You need to take a look
at them, and decide on an initial action.
Generally, this means one of three things:
@ -593,9 +594,10 @@ Generally, this means one of three things:
(3) You decide the bug belongs to you, but you can\'t solve it this moment.
Just use the "Accept bug" command.
To get a list of all NEW bugs, you can use this URL (bookmark it if you like!):
To get a list of all NEW/REOPENED bugs, you can use this URL (bookmark
it if you like!):
%urlbase%buglist.cgi?bug_status=NEW&assigned_to=%userid%
%urlbase%buglist.cgi?bug_status=NEW&bug_status=REOPENED&assigned_to=%userid%
Or, you can use the general query page, at
%urlbase%query.cgi.

10
webtools/bugzilla/whineatnews.pl
View File

@ -19,12 +19,13 @@
# Rights Reserved.
#
# Contributor(s): Terry Weissman <terry@mozilla.org>
# Joseph Heenan <joseph@heenan.me.uk>
# This is a script suitable for running once a day from a cron job. It
# looks at all the bugs, and sends whiny mail to anyone who has a bug
# assigned to them that has status NEW that has not been touched for
# more than 7 days.
# assigned to them that has status NEW or REOPENED that has not been
# touched for more than 7 days.
use strict;
@ -33,8 +34,9 @@ require "globals.pl";
ConnectToDatabase();
SendSQL("select bug_id,short_desc,login_name from bugs,profiles where " .
"bug_status = 'NEW' and to_days(now()) - to_days(delta_ts) > " .
Param('whinedays') . " and userid=assigned_to order by bug_id");
"(bug_status = 'NEW' or bug_status = 'REOPENED') and " .
"to_days(now()) - to_days(delta_ts) > " . Param('whinedays') .
" and userid=assigned_to order by bug_id");
my %bugs;
my %desc;