From 004d0c1cf93ac47317b490f83523c799f8651afd Mon Sep 17 00:00:00 2001
From: "mkanat%kerio.com" <>
Date: Fri, 8 Apr 2005 01:18:52 +0000
Subject: Bug 286686: ORDER BY column needs to be selected or grouped Patch By
 Tomas Kopal <Tomas.Kopal@altap.cz> r=mkanat, a=justdave

---
 Bugzilla/User.pm | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

(limited to 'Bugzilla/User.pm')

diff --git a/Bugzilla/User.pm b/Bugzilla/User.pm
index 0d1b815c0..2096db407 100644
--- a/Bugzilla/User.pm
+++ b/Bugzilla/User.pm
@@ -598,7 +598,8 @@ sub match {
 
         # Build the query.
         my $sqlstr = &::SqlQuote($wildstr);
-        my $query  = "SELECT DISTINCT userid, realname, login_name " .
+        my $query  = "SELECT DISTINCT userid, realname, login_name, " .
+                     "LENGTH(login_name) AS namelength " .
                      "FROM profiles ";
         if (&::Param('usevisibilitygroups')) {
             $query .= ", user_group_map ";
@@ -613,7 +614,7 @@ sub match {
                       "AND grant_type <> " . GRANT_DERIVED;
         }
         $query    .= " AND disabledtext = '' " if $exclude_disabled;
-        $query    .= "ORDER BY length(login_name) ";
+        $query    .= "ORDER BY namelength ";
         $query    .= $dbh->sql_limit($limit) if $limit;
 
         # Execute the query, retrieve the results, and make them into
@@ -648,7 +649,8 @@ sub match {
 
         my $sqlstr = &::SqlQuote(uc($str));
 
-        my $query   = "SELECT DISTINCT userid, realname, login_name " .
+        my $query   = "SELECT DISTINCT userid, realname, login_name, " .
+                      "LENGTH(login_name) AS namelength " .
                       "FROM  profiles";
         if (&::Param('usevisibilitygroups')) {
             $query .= ", user_group_map";
@@ -665,7 +667,7 @@ sub match {
                       " AND grant_type <> " . GRANT_DERIVED;
         }
         $query     .= " AND disabledtext = ''" if $exclude_disabled;
-        $query     .= " ORDER BY length(login_name)";
+        $query     .= " ORDER BY namelength";
         $query     .= " " . $dbh->sql_limit($limit) if $limit;
         &::PushGlobalSQLState();
         &::SendSQL($query);
-- 
cgit v1.2.3-24-g4f1b