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 /addons/project/models | |
| 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 'addons/project/models')
| -rw-r--r-- | addons/project/models/project.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/addons/project/models/project.py b/addons/project/models/project.py index bc415334515..dca8b1407ef 100644 --- a/addons/project/models/project.py +++ b/addons/project/models/project.py @@ -1030,7 +1030,7 @@ class Task(models.Model): days = list(DAYS.keys()) week_start = fields.Datetime.today().weekday() - if all([d in default_fields for d in days]): + if all(d in default_fields for d in days): vals[days[week_start]] = True if 'repeat_day' in default_fields: vals['repeat_day'] = str(fields.Datetime.today().day) |