From dc4d421be78c471017fd8cf0d3b7986147844dab Mon Sep 17 00:00:00 2001
From: Dries Buytaert <dries@buytaert.net>
Date: Fri, 22 May 2009 11:33:18 +0000
Subject: - Patch #396284 by Berdir, chx, jcfiala, csevb10 et al: make sure to
 order by table headers first, before ordering by other fields.

---
 includes/tablesort.inc | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

(limited to 'includes')

diff --git a/includes/tablesort.inc b/includes/tablesort.inc
index 382014fa8..e6a7b785a 100644
--- a/includes/tablesort.inc
+++ b/includes/tablesort.inc
@@ -23,8 +23,13 @@ class TableSort extends SelectQueryExtender {
   protected $header = array();
 
   public function execute() {
+    return $this->query->execute();
+  }
+
+  public function orderByHeader(Array $header) {
+    $this->header = $header;
     $ts = $this->init();
-    if ($ts['sql']) {
+    if (!empty($ts['sql'])) {
       // Based on code from db_escape_table(), but this can also contain a dot.
       $field = preg_replace('/[^A-Za-z0-9_.]+/', '', $ts['sql']);
 
@@ -33,11 +38,6 @@ class TableSort extends SelectQueryExtender {
       $sort = in_array($sort, array('ASC', 'DESC')) ? $sort : '';
       $this->orderBy($field, $sort);
     }
-    return $this->query->execute();
-  }
-
-  public function setHeader(Array $header) {
-    $this->header = $header;
     return $this;
   }
 
-- 
cgit v1.2.3