use only the catalog for better performance

limit result to 1 because they are sorted by version and date, so if there is some results, the first one is the best.


git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@27795 20353a03-c40f-0410-a6d1-a30d3c3de9de

product/ERP5/Document/TradeCondition.py

@@ -358,17 +358,15 @@ class TradeCondition(Path, Transformation, XMLMatrix):
       model_object_list = [result.getObject() for result in \
           self.portal_catalog(portal_type=self.portal_type,
                               reference=reference,
-                              sort_on=(('version','descending'),))]
-
-      # if there is model which has effective period containing
-      # the start_date and the stop date of the paysheet, return it
-      for current_model in model_object_list:
-        if current_model.getEffectiveDate() <= start_date and \
-            current_model.getExpirationDate() >= stop_date:
-          effective_model_list.append(current_model)
-      if len(effective_model_list):
-        return effective_model_list[0]
-      # if no effective model are found (ex. because dates are None), return self
+                              sort_on=(('version','descending'),),
+                              effective_date="<=%s"%start_date,
+                              expiration_date=">=%s"%stop_date,
+                              limit=1)]
+      if len(model_object_list):
+        return model_object_list[0].getObject()
+      else:
+        # if no effective model are found (ex. because dates are None), 
+        # return self
         return self
 
     security.declareProtected(Permissions.AccessContentsInformation,