Careful with Email Campaigns in SuiteCRM 8.9.x!

tapio · 20 February 2026 07:25

Nice to see that there are plans for Email Campaigns!

I did send a couple related PR:s to fix some issues.

I also provided a solution for this bug, which appears only when using legacy path for Email Campaigns.

github.com/SuiteCRM/SuiteCRM-Core

Fix #862 -Legacy campaign send to prefer Email Marketing content over template

hotfix ← eleknader:fix/issue-862-legacy-emailmarketing-content

opened 07:16AM - 20 Feb 26 UTC

eleknader

+11 -0

## Problem Legacy campaign delivery (`function::runMassEmailCampaign` -> `Email…Man`) can send subject/body from the linked Email Template even when Email Marketing content has been edited in UI. ## Root Cause In legacy `EmailMan::sendEmail()`, `current_emailtemplate` content was always used after template retrieval. `email_marketing.subject/body` did not override template values. ## Changes - File: `public/legacy/modules/EmailMan/EmailMan.php` - After template content unescape, prefer Email Marketing content when present: - override subject with `email_marketing.subject` if non-empty - override body_html/body with `email_marketing.body` if non-empty - Kept fallback behavior: if Email Marketing fields are empty, template content is still used. ## Validation - `php -l public/legacy/modules/EmailMan/EmailMan.php` - DEV evidence with test campaign data: - legacy path reached (`Run Nightly Mass Email Campaigns`) - generated campaign email record used Email Marketing subject/body marker (`DO YOU SEND THIS OR THE TEMPLATE?`) - template marker (`Issue 848 test`) not present in generated body ## Scope / Risk - Small, isolated change in one legacy file. - No schema changes. - No config changes.

PR needs two reviews by developers with write-access.

Best Regards,

Tapio