[Zope-Checkins] SVN: Zope/branches/andig-catalog-report/src/Products/ZCatalog/ZCatalog.py Simplify the sort code

Hanno Schlichting hannosch at hannosch.eu
Sat Jul 24 10:57:59 EDT 2010


Log message for revision 115012:
  Simplify the sort code
  

Changed:
  U   Zope/branches/andig-catalog-report/src/Products/ZCatalog/ZCatalog.py

-=-
Modified: Zope/branches/andig-catalog-report/src/Products/ZCatalog/ZCatalog.py
===================================================================
--- Zope/branches/andig-catalog-report/src/Products/ZCatalog/ZCatalog.py	2010-07-24 14:51:33 UTC (rev 115011)
+++ Zope/branches/andig-catalog-report/src/Products/ZCatalog/ZCatalog.py	2010-07-24 14:57:59 UTC (rev 115012)
@@ -16,6 +16,7 @@
 """
 
 import logging
+import operator
 import sys
 import string
 import time
@@ -1008,7 +1009,7 @@
 
     def delColumn(self, name):
         return self._catalog.delColumn(name)
-    
+
     #
     # Catalog report methods
     #
@@ -1017,11 +1018,7 @@
     def getCatalogReport(self):
         """ Reports about the duration of queries """
         rval = self._catalog.getCatalogReport().report()
-
-        sort_by = 'duration'
-        rval.sort(lambda e1, e2, sort_by=sort_by:
-                  cmp(e1[sort_by], e2[sort_by]))
-        rval.reverse()
+        rval.sort(key=operator.itemgetter('duration'), reverse=True)
         return rval
 
     security.declareProtected(manage_zcatalog_entries,



More information about the Zope-Checkins mailing list