From de58b8b37efbfb1d387db6d061e65cb2db0579c0 Mon Sep 17 00:00:00 2001
From: Vincent Pelletier <vincent@nexedi.com>
Date: Tue, 7 Aug 2007 17:52:36 +0000
Subject: [PATCH] Reuse existing list instead of generating it again.

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@15542 20353a03-c40f-0410-a6d1-a30d3c3de9de
---
 product/ERP5Type/patches/WorkflowTool.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/product/ERP5Type/patches/WorkflowTool.py b/product/ERP5Type/patches/WorkflowTool.py
index 0b307d7002..0d6ef6e437 100644
--- a/product/ERP5Type/patches/WorkflowTool.py
+++ b/product/ERP5Type/patches/WorkflowTool.py
@@ -245,7 +245,7 @@ def getWorklistListQuery(grouped_worklist_dict, securityQueryHook):
   if SECURITY_PARAMETER_ID not in total_criterion_id_list:
     # This request has no defined local_roles, so we must use default security query
     query = ComplexQuery(query, securityQueryHook(), operator='AND')
-  group_by_expression = ', '.join([x for x in total_criterion_id_dict.keys() if x != SECURITY_PARAMETER_ID])
+  group_by_expression = ', '.join([x for x in total_criterion_id_list if x != SECURITY_PARAMETER_ID])
   assert COUNT_COLUMN_TITLE not in total_criterion_id_dict
   select_expression = 'count(*) as %s, %s' % (COUNT_COLUMN_TITLE, group_by_expression)
   return (select_expression, group_by_expression, query)
-- 
2.30.9