diff options
| author | Victor Feyens <vfe@odoo.com> | 2020-08-10 13:17:00 +0000 |
|---|---|---|
| committer | Victor Feyens <vfe@odoo.com> | 2020-08-14 09:56:10 +0000 |
| commit | fdb23e282b90ad95e4f909f7232eb0aa36038836 (patch) | |
| tree | c29267645405b1fabd0b12c1276b89e6c7058f58 /addons/purchase_requisition | |
| parent | 65bf6988f941be6fe1e88202bc814e4b645abf79 (diff) | |
[IMP] *: wrong any() usage
Replace wrong usages of any(list|recordset), by any(generator)
to speed up computations, avoiding list creations and/or looping twice on a recordset
for nothing.
any([generator]) => any(generator)
any(filtered) => any(generator)
closes odoo/odoo#55768
Related: odoo/enterprise#12360
Signed-off-by: Victor Feyens (vfe) <vfe@odoo.com>
Diffstat (limited to 'addons/purchase_requisition')
| -rw-r--r-- | addons/purchase_requisition/models/purchase_requisition.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/addons/purchase_requisition/models/purchase_requisition.py b/addons/purchase_requisition/models/purchase_requisition.py index 0a4e042a6e4..24e2f30500c 100644 --- a/addons/purchase_requisition/models/purchase_requisition.py +++ b/addons/purchase_requisition/models/purchase_requisition.py @@ -187,7 +187,7 @@ class PurchaseRequisitionLine(models.Model): ('product_id', '=', vals.get('product_id')), ('name', '=', res.requisition_id.vendor_id.id), ]) - if not any([s.purchase_requisition_id for s in supplier_infos]): + if not any(s.purchase_requisition_id for s in supplier_infos): res.create_supplier_info() if vals['price_unit'] <= 0.0: raise UserError(_('You cannot confirm the blanket order without price.')) |