diff --git a/mail_debrand/models/mail_render_mixin.py b/mail_debrand/models/mail_render_mixin.py index 3123de2..dbedcd0 100644 --- a/mail_debrand/models/mail_render_mixin.py +++ b/mail_debrand/models/mail_render_mixin.py @@ -26,37 +26,28 @@ class MailRenderMixin(models.AbstractModel): # We don't want to change what was explicitly added in the message body, # so we will only change what is before and after it. if to_keep: - to_change = value.split(to_keep) - else: - to_change = [value] - to_keep = "" - new_parts = [] - for part in to_change: - tree = html.fromstring(part) - if tree is None: - new_parts.append(part) - continue - odoo_anchors = tree.xpath('//a[contains(@href,"odoo.com")]') - for elem in odoo_anchors: - parent = elem.getparent() - previous = elem.getprevious() - - if remove_before and not remove_parent and previous is not None: - # remove 'using' that is before - previous.tail = "" - if remove_parent and len(parent.getparent()): - # anchor ") + tree = html.fromstring(value) + odoo_anchors = tree.xpath('//a[contains(@href,"odoo.com")]') + for elem in odoo_anchors: + parent = elem.getparent() + previous = elem.getprevious() + if remove_before and not remove_parent and previous is not None: + # remove 'using' that is before + previous.tail = "" + if remove_parent and len(parent.getparent()): + # anchor ", to_keep) return value @api.model