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/wizard | |
| 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/wizard')
| -rw-r--r-- | addons/project/wizard/project_delete_wizard.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/addons/project/wizard/project_delete_wizard.py b/addons/project/wizard/project_delete_wizard.py index 9dff0454549..00bb4aa5ff4 100644 --- a/addons/project/wizard/project_delete_wizard.py +++ b/addons/project/wizard/project_delete_wizard.py @@ -14,7 +14,7 @@ class ProjectDelete(models.TransientModel): def _compute_projects_archived(self): for wizard in self.with_context(active_test=False): - wizard.projects_archived = all([not p.active for p in wizard.project_ids]) + wizard.projects_archived = all(not p.active for p in wizard.project_ids) def _compute_task_count(self): for wizard in self: |