[CHG] limit search result when the goal is to work only on the first id

acsone · Jul 31, 2015 · a506361 · a506361
1 parent bd5102c
commit a506361
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/distribution_list/distribution_list.py b/distribution_list/distribution_list.py
@@ -112,7 +112,7 @@ def _order_del(self, cr, uid, lst, context=None):
             cr, uid, 'distribution.list', context=c),
         'dst_model_id': lambda self, cr, uid, c:
         self.pool.get('ir.model').search(
-            cr, uid, [('model', '=', 'res.partner')], context=c)[0],
+            cr, uid, [('model', '=', 'res.partner')], limit=1, context=c)[0],
         'bridge_field': DEFAULT_BRIDGE_FIELD,
     }
 
@@ -416,7 +416,7 @@ def _get_record(self, record_or_list):
         'domain': "[]",
         'src_model_id': lambda self, cr, uid, c:
             self.pool.get('ir.model').search(
-            cr, uid, [('model', '=', 'res.partner')], context=c)[0],
+            cr, uid, [('model', '=', 'res.partner')], limit=1, context=c)[0],
     }
 
     def onchange_src_model_id(self, cr, uid, ids, context=None):

diff --git a/mass_mailing_distribution_list/distribution_list.py b/mass_mailing_distribution_list/distribution_list.py
@@ -331,7 +331,7 @@ def generate_alias(self, cr, uid, dl_id, dl_name, context=None):
                 _('Please contact your Administrator '
                   'to configure a "catchall" email alias'))
         distribution_list_model_id = self.pool['ir.model'].search(
-            cr, uid, [('model', '=', 'distribution.list')])[0]
+            cr, uid, [('model', '=', 'distribution.list')], limit=1)[0]
         vals = {
             'alias_name': '%s+%s' % (catchall_alias, dl_name),
             'alias_defaults': '{"distribution_list_id": %s}' % str(dl_id),

diff --git a/mass_mailing_distribution_list/tests/test_mail_compose_message.py b/mass_mailing_distribution_list/tests/test_mail_compose_message.py
@@ -39,7 +39,7 @@ def setUp(self):
         self.mass_mailing_obj = self.registry['mail.mass_mailing']
         self.mail_compose_message_obj = self.registry['mail.compose.message']
         self.dst_model_id = self.registry('ir.model').search(
-            self.cr, self.uid, [('model', '=', SRC_MODEL)])[0]
+            self.cr, self.uid, [('model', '=', SRC_MODEL)], limit=1)[0]
 
         self.registry('ir.model').clear_caches()
         self.registry('ir.model.data').clear_caches()