Test form modifies active MailSystem settings

Created on 1 July 2024, 6 months ago

Problem/Motivation

When using the test form at /admin/config/system/htmlmail/test the mailsystem.settings config is modified with the selection of the "Test mail sending class" field value, which should not happen.

Steps to reproduce

- Install and enable MimeMail and MailSystem modules.
- Configure the default MailSystem to use different classes for "Formatter" and "Sender". For example formatter - "HTML Mail mailer", sender - "Default PHP mailer"
- Send a test mail at /admin/config/system/htmlmail/test selection "HTML Mail mailer" as "Test mail sending class"
- Check MailSystem configuration, both formatter and sender have been changed to "HTML Mail mailer".

Proposed resolution

- Toogle back to the previous configuration after sending the test e-mail in HtmlMailTestForm::submitForm()

🐛 Bug report

Status

Needs review

Version

4.0

Component

Code

Created by

🇪🇸Spain budalokko Girona

Live updates comments and jobs are added and updated live.

Merge Requests

!10Test form modifies active MailSystem settings
Open
🇪🇸Spain budalokko
updated 6 months ago

Comments & Activities

Issue created by @budalokko
Comment 6 months ago →
🇪🇸Spain budalokko Girona
Sorry the noise ... wrong module :)
Merge request !10Issue #3458278: Fix test form modifies active MailSystem settings. → (Open) created by budalokko
Status changed to Needs review 6 months ago12:29pm 1 July 2024
Comment 6 months ago →
🇪🇸Spain budalokko Girona
Pipeline finished with Success
6 months ago
Total: 197s
#212887

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024