From owner-freebsd-www@FreeBSD.ORG  Tue Apr  8 08:48:49 2003
Return-Path: <owner-freebsd-www@FreeBSD.ORG>
Delivered-To: freebsd-www@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 4071037B401
	for <www@freebsd.org>; Tue,  8 Apr 2003 08:48:49 -0700 (PDT)
Received: from flood.ping.uio.no (flood.ping.uio.no [129.240.78.31])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 2EA0343F3F
	for <www@freebsd.org>; Tue,  8 Apr 2003 08:48:48 -0700 (PDT)
	(envelope-from des@ofug.org)
Received: by flood.ping.uio.no (Postfix, from userid 2602)
	id C262B5308; Tue,  8 Apr 2003 17:48:45 +0200 (CEST)
X-URL: http://www.ofug.org/~des/
X-Disclaimer: The views expressed in this message do not necessarily
  coincide with those of any organisation or company with
  which I am or have been affiliated.
To: www@freebsd.org
From: Dag-Erling Smorgrav <des@ofug.org>
Date: Tue, 08 Apr 2003 17:48:44 +0200
Message-ID: <xzpznn1m1lv.fsf@flood.ping.uio.no>
User-Agent: Gnus/5.090015 (Oort Gnus v0.15) Emacs/21.2
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="=-=-="
Subject: Change default mode for query-pr-summary
X-BeenThere: freebsd-www@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: FreeBSD Project Webmasters <freebsd-www.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-www>,
	<mailto:freebsd-www-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-www>
List-Post: <mailto:freebsd-www@freebsd.org>
List-Help: <mailto:freebsd-www-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-www>,
	<mailto:freebsd-www-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 08 Apr 2003 15:48:49 -0000

--=-=-=

The current default behaviour for query-pr-summary is to group PRs by
severity.  This is meaningless as originators often set an inflated
severity on their PRs, and handlers rarely change it.  The attached
patch changes the default to just show all PRs, without removing the
option to group them by severity.

DES
-- 
Dag-Erling Smorgrav - des@ofug.org


--=-=-=
Content-Type: text/x-patch
Content-Disposition: attachment; filename=query-pr-summary.diff

Index: en/cgi/query-pr-summary.cgi
===================================================================
RCS file: /home/dcvs/www/en/cgi/query-pr-summary.cgi,v
retrieving revision 1.39
diff -u -r1.39 query-pr-summary.cgi
--- en/cgi/query-pr-summary.cgi	6 Mar 2003 23:51:58 -0000	1.39
+++ en/cgi/query-pr-summary.cgi	8 Apr 2003 15:47:27 -0000
@@ -150,9 +150,9 @@
 # If someone does a multiple-variable query they will probably do weird things.
 
 $self_ref1 = $self_ref . '?';
-$self_ref1 .= 'sort=' . escape($input{'sort'}) if $input{'sort'};
-print '<P>You may view summaries by <A HREF="', $self_ref1, '">Severity</A>, ';
-$self_ref1 .= '&' if ($self_ref1 !~/\?$/);
+$self_ref1 .= 'sort=' . escape($input{'sort'}) . "&" if $input{'sort'};
+print '<P>You may view summaries by ';
+print '<A HREF="', $self_ref1, 'severity=summary">Severity</A>, ';
 print '<A HREF="', $self_ref1, 'state=summary">State</A>, ';
 print '<A HREF="', $self_ref1, 'category=summary">Category</A>, or ';
 print '<A HREF="', $self_ref1, 'responsible=summary">Responsible Party</A>.';
@@ -252,7 +252,7 @@
 } elsif ($input{'category'} eq 'summary') {
 	&cat_summary;
 
-} elsif ($input{'severity'} eq '') {
+} elsif ($input{'severity'} eq 'summary') {
 	&severity_summary;
 
 } else {

--=-=-=--