From d37f6f0b365d191b36db023be7d03a3024bfed97 Mon Sep 17 00:00:00 2001
From: "teodoralexandru@nexterp.ro" <teodoralexandru@nexterp.ro>
Date: Fri, 26 Feb 2021 13:15:40 +0200
Subject: [PATCH] I have a fix for the issue, here: ursais@6a3933f

---
 mail_debrand/models/mail_render_mixinANDmail_mail.py | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/mail_debrand/models/mail_render_mixinANDmail_mail.py b/mail_debrand/models/mail_render_mixinANDmail_mail.py
index 81621668c..61f67e7d3 100644
--- a/mail_debrand/models/mail_render_mixinANDmail_mail.py
+++ b/mail_debrand/models/mail_render_mixinANDmail_mail.py
@@ -33,7 +33,7 @@ class MailRenderMixin(models.AbstractModel):
                     only_what_is_in_tags = bytes_text[: bytes_text.rfind(b">") + 1]
                     data_formatted = html.fromstring(only_what_is_in_tags)
                     parent.replace(previous, data_formatted)
-                if parent.getparent() and remove_parent:
+                if len(parent.getparent()) and remove_parent:
                     # anchor <a href odoo has a parent powered by that must be removed
                     parent.getparent().remove(parent)
                 else:
@@ -42,12 +42,10 @@ class MailRenderMixin(models.AbstractModel):
                     else:
                         parent.remove(elem)
             value = etree.tostring(tree, pretty_print=True, method="html")
-        if type(value) is str:
-            value = re.sub("[^(<)(</)]odoo", "", value, flags=re.IGNORECASE)
-        elif type(value) is bytes:
-            value = re.sub(b"[^(<)(</)]odoo", b"", value, flags=re.IGNORECASE)
-
-        return value
+            # etree can return bytes; ensure we get a proper string
+            if type(value) is bytes:
+                value = value.decode()
+        return re.sub("[^(<)(</)]odoo", "", value, flags=re.IGNORECASE)
 
     @api.model
     def _render_template(