[Checkins] SVN: Products.ZCatalog/trunk/src/Products/ZCatalog/Catalog.py remove some code duplication

Hano Schlichting cvs-admin at zope.org
Sun Apr 8 14:08:18 UTC 2012


Log message for revision 125106:
  remove some code duplication
  

Changed:
  U   Products.ZCatalog/trunk/src/Products/ZCatalog/Catalog.py

-=-
Modified: Products.ZCatalog/trunk/src/Products/ZCatalog/Catalog.py
===================================================================
--- Products.ZCatalog/trunk/src/Products/ZCatalog/Catalog.py	2012-04-08 14:04:36 UTC (rev 125105)
+++ Products.ZCatalog/trunk/src/Products/ZCatalog/Catalog.py	2012-04-08 14:08:14 UTC (rev 125106)
@@ -877,14 +877,12 @@
                             n += 1
                         worst = keys[0]
                 result = multisort(result, sort_spec)
+            sequence, _ = self._limit_sequence(result, 0, b_start, b_size,
+                switched_reverse)
             if merge:
-                sequence, _ = self._limit_sequence(result, 0, b_start, b_size,
-                    switched_reverse)
                 result = LazyValues(sequence)
                 result.actual_result_count = actual_result_count
             else:
-                sequence, _ = self._limit_sequence(result, 0, b_start, b_size,
-                    switched_reverse)
                 return sequence
         elif not first_reverse:
             # Limit / sort results using N-Best algorithm in reverse (N-Worst?)
@@ -932,14 +930,12 @@
                             n += 1
                         best = keys[-1]
                 result = multisort(result, sort_spec)
+            sequence, _ = self._limit_sequence(result, 0, b_start, b_size,
+                switched_reverse)
             if merge:
-                sequence, _ = self._limit_sequence(result, 0, b_start, b_size,
-                    switched_reverse)
                 result = LazyValues(sequence)
                 result.actual_result_count = actual_result_count
             else:
-                sequence, _ = self._limit_sequence(result, 0, b_start, b_size,
-                    switched_reverse)
                 return sequence
 
         return LazyMap(self.__getitem__, result, len(result),



More information about the checkins mailing list