From 958f1f6a9f1ee5b4f5057b089fef3b262046b5ce Mon Sep 17 00:00:00 2001
From: Steven Murray <Steven.Murray@cern.ch>
Date: Mon, 12 Jun 2017 16:46:06 +0200
Subject: [PATCH] Fixed a bug introduced with the previous commit concerning
 this issue.

---
 catalogue/RdbmsCatalogue.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/catalogue/RdbmsCatalogue.cpp b/catalogue/RdbmsCatalogue.cpp
index b863041eb4..72f2f23b51 100644
--- a/catalogue/RdbmsCatalogue.cpp
+++ b/catalogue/RdbmsCatalogue.cpp
@@ -1731,9 +1731,7 @@ std::list<common::dataStructures::Tape> RdbmsCatalogue::getTapes(rdbms::PooledCo
         "LAST_UPDATE_HOST_NAME AS LAST_UPDATE_HOST_NAME,"
         "LAST_UPDATE_TIME AS LAST_UPDATE_TIME "
       "FROM "
-        "TAPE "
-      "ORDER BY "
-        "VID";
+        "TAPE";
 
     if(searchCriteria.vid||
        searchCriteria.logicalLibrary||
@@ -1781,6 +1779,8 @@ std::list<common::dataStructures::Tape> RdbmsCatalogue::getTapes(rdbms::PooledCo
       sql += " LBP_IS_ON = :LBP_IS_ON";
     }
 
+    sql += " ORDER BY VID";
+
     auto stmt = conn.createStmt(sql, rdbms::Stmt::AutocommitMode::OFF);
 
     if(searchCriteria.vid) stmt->bindString(":VID", searchCriteria.vid.value());
-- 
GitLab