Was choking on order='Importance'.

diff --git a/buglist.cgi b/buglist.cgi
index 75549e730df745c16898bede5286a26db42efc92..8cb09d427cb10ac4bb0caee547ff2c105e9edacc 100755 (executable)
--- a/buglist.cgi
+++ b/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'};
 }