From e53fba47d731c4894ed65213556cfc4a3bd6c192 Mon Sep 17 00:00:00 2001 From: "terry%mozilla.org" Date: Tue, 7 Mar 2000 21:04:35 +0000 Subject: [PATCH] Was choking on order='Importance'. git-svn-id: svn://10.0.0.236/trunk@62326 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/webtools/bugzilla/buglist.cgi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mozilla/webtools/bugzilla/buglist.cgi b/mozilla/webtools/bugzilla/buglist.cgi index 75549e730df..8cb09d427cb 100755 --- a/mozilla/webtools/bugzilla/buglist.cgi +++ b/mozilla/webtools/bugzilla/buglist.cgi @@ -888,8 +888,6 @@ if (defined $::FORM{'order'} && $::FORM{'order'} ne "") { $::FORM{'order'} =~ s/assign\.login_name/map_assigned_to.login_name/g; # Another backwards compatability hack. - die "Invalid order: $::FORM{'order'}" unless - $::FORM{'order'} =~ /^([a-zA-Z0-9_., ]+)$/; ORDER: for ($::FORM{'order'}) { /\./ && do { # This (hopefully) already has fieldnames in it, so we're done. @@ -910,6 +908,8 @@ if (defined $::FORM{'order'} && $::FORM{'order'} ne "") { # DEFAULT $::FORM{'order'} = "bugs.bug_status, bugs.priority, map_assigned_to.login_name, bugs.bug_id"; } + die "Invalid order: $::FORM{'order'}" unless + $::FORM{'order'} =~ /^([a-zA-Z0-9_., ]+)$/; $query .= $::FORM{'order'}; }