From 2c626b61b6fa4400934311bef996423d55b48508 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adri=C3=A1n=20Bernardi?= Date: Wed, 28 Dec 2022 11:00:09 -0300 Subject: [PATCH] lims_administrative_task: fix user email control --- lims_administrative_task/task.py | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/lims_administrative_task/task.py b/lims_administrative_task/task.py index aec91d1..e7e3111 100644 --- a/lims_administrative_task/task.py +++ b/lims_administrative_task/task.py @@ -372,7 +372,9 @@ class AdministrativeTask(Workflow, ModelSQL, ModelView): return for task in tasks: - to_addrs = [task.responsible.email] + to_addrs = [] + if task.responsible.email: + to_addrs.append(task.responsible.email) if not to_addrs: logger.error("Missing address for '%s' to send email", task.responsible.rec_name) @@ -392,7 +394,11 @@ class AdministrativeTask(Workflow, ModelSQL, ModelView): return for task in tasks: - to_addrs = [task.responsible.email, task.create_uid.email] + to_addrs = [] + if task.responsible.email: + to_addrs.append(task.responsible.email) + if task.create_uid.email: + to_addrs.append(task.create_uid.email) for user in task.notified_users: to_addrs.append(user.email) if not to_addrs: