Allow override of LegacyEmailBuilder with configured policy

Created on 18 January 2023, almost 2 years ago
Updated 9 February 2023, almost 2 years ago

Problem/Motivation

Normally email adjusters run after email builders and hence they can override the default values. However LegacyEmailBuilder uses preRender() instead of build() so this doesn't work - LegacyEmailBuilder overwrites the values from policy.

Proposed resolution

The fix is delicate - the code was moved to preRender() as part of 🐛 LegacyEmailBuilder calls hook_mail() too early Fixed . We can put the code back to the build() phase so long as we ensure the theme is switched first. This also allows removing of a hack from BaseEmailTrait::setAddress().

Remaining tasks

User interface changes

API changes

Data model changes

Feature request
Status

Fixed

Version

1.0

Component

Code

Created by

🇬🇧United Kingdom adamps

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024