diff options
| author | Victor Feyens <vfe@odoo.com> | 2020-08-18 12:26:51 +0000 |
|---|---|---|
| committer | Thibault Delavallée <tde@odoo.com> | 2020-08-31 11:45:43 +0000 |
| commit | 1cc6bb1231401ea7f501d2f5b5e9641ec8734850 (patch) | |
| tree | 9c6da86e0a5a13368365c895e77ede2b1ebac3d9 /odoo/models.py | |
| parent | df06d7f9c488ab3e463baed10b55bea8bbfb5908 (diff) | |
[IMP] various: improve code style and performance by improve ``all()`` usage
PURPOSE
Clean code. Be more performance oriented.
SPECIFICATIONS
Improvements applied in this commit
* not all() --> any(not) for earlier returns;
* all([generator]) --> all(generator) to avoid unnecessary list casting.
This code construct is better managed by all;
This commit will probably not have a big performance effect on standard
production databases. However each performance and cleaning improvement
is welcomed.
LINKS
Task ID-2328619
closes odoo/odoo#56060
Related: odoo/enterprise#12486
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
Diffstat (limited to 'odoo/models.py')
| -rw-r--r-- | odoo/models.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/odoo/models.py b/odoo/models.py index 329e64bca7c..0bccf401912 100644 --- a/odoo/models.py +++ b/odoo/models.py @@ -5895,7 +5895,7 @@ Fields: first_call = not names - if not all(name in self._fields for name in names): + if any(name not in self._fields for name in names): return {} def PrefixTree(model, dotnames): |