[Erp5-report] r15475 - /erp5/trunk/products/ERP5Type/patches/WorkflowTool.py

[nobody at svn.erp5.org]

Author: vincent
Date: Fri Aug  3 18:46:53 2007
New Revision: 15475

URL: http://svn.erp5.org?rev=15475&view=rev
Log:
.reverse() and .pop(0) can be simplified into just .pop()

Modified:
    erp5/trunk/products/ERP5Type/patches/WorkflowTool.py

Modified: erp5/trunk/products/ERP5Type/patches/WorkflowTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/patches/WorkflowTool.py?rev=15475&r1=15474&r2=15475&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/patches/WorkflowTool.py (original)
+++ erp5/trunk/products/ERP5Type/patches/WorkflowTool.py Fri Aug  3 18:46:53 2007
@@ -181,7 +181,7 @@
   """
   assert possible_worklist_id_dict is None or len(possible_worklist_id_dict) != 0
   my_priority_list = priority_list[:]
-  my_criterion_id = my_priority_list.pop(0)
+  my_criterion_id = my_priority_list.pop()
   query_list = []
   append = query_list.append
   my_criterion_dict = criterion_dict[my_criterion_id]
@@ -240,7 +240,6 @@
     return cmp(max([len(x) for x in total_criterion_id_dict[criterion_id_a].itervalues()]),
                max([len(x) for x in total_criterion_id_dict[criterion_id_b].itervalues()]))
   total_criterion_id_list.sort(criterion_id_cmp)
-  total_criterion_id_list.reverse()
   query = generateNestedQuery(priority_list=total_criterion_id_list, criterion_dict=total_criterion_id_dict, securityQueryHook=securityQueryHook)
   assert query is not None
   group_by_expression = ', '.join([x for x in total_criterion_id_dict.keys() if x != SECURITY_PARAMETER_ID])