From 463a817853ee8056bfe4c39b15ed1c7f2d14e451 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Mon, 16 Nov 2020 21:22:13 +0100
Subject: [PATCH] Skip record when it is empty

The Reference value may be None if the target has been deleted despite being
required.

issue9813
review331131002
---
 marketing_automation.py | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/marketing_automation.py b/marketing_automation.py
index 08fc68bbb9..210f1b4c6f 100644
--- a/marketing_automation.py
+++ b/marketing_automation.py
@@ -492,6 +492,10 @@ class Activity(ModelSQL, ModelView):
         RecordActivity = pool.get('marketing.automation.record.activity')
         record = activity.record
 
+        # As it is a reference, the record may have been deleted
+        if not record:
+            return
+
         # XXX: use domain
         if self.condition and not record.eval(self.condition):
             return
@@ -691,7 +695,10 @@ class Record(ModelSQL, ModelView):
         cls.write(records, {'blocked': True})
 
     def get_rec_name(self, name):
-        return self.record.rec_name
+        if self.record:
+            return self.record.rec_name
+        else:
+            return '(%s)' % self.id
 
     @classmethod
     def create(cls, vlist):