From 9fca638d5e730e101b3cae93ee4cd72719b70a83 Mon Sep 17 00:00:00 2001
From: Antoine Millet <antoine.millet@smartjog.com>
Date: Fri, 18 Mar 2011 16:07:12 +0100
Subject: [PATCH] [bug#4070] Avoid duplicate tags in order list.

---
 ccserver/handlers.py | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/ccserver/handlers.py b/ccserver/handlers.py
index 8a737be..f82470f 100644
--- a/ccserver/handlers.py
+++ b/ccserver/handlers.py
@@ -7,6 +7,7 @@ import inspect
 import logging
 from sjrpc.utils import RpcHandler
 from sjrpc.core import RpcError
+from ccserver.orderedset import OrderedSet
 from ccserver.conf import CCConf
 from ccserver.exceptions import (AlreadyRegistered, AuthenticationError,
                                  RightError, ReservedTagError, BadObjectError,
@@ -196,10 +197,10 @@ class CliHandler(OnlineCCHandler):
         self._check(conn, 'list', query)
         logging.debug('Executed list function with query %s', query)
         objects, tags = self._server.list(query, return_toshow=True)
-        order = ['id']
+        order = OrderedSet(['id'])
         if tags is not None:
-            order += tags
-        return {'objects': objects, 'order': order}
+            order |= OrderedSet(tags)
+        return {'objects': objects, 'order': list(order)}
 
     def _vm_action(self, query, method, *args, **kwargs):
         vms = self._server.list(query, show=set(('r', 'h')))
-- 
GitLab