[PR #475] [MERGED] Fixed default SMTP server when editing a Draft #833

New issue

Closed

opened 2026-02-25 21:36:22 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 21:36:22 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/475
Author: @henrique-borba
Created: 5/14/2021
Status: ✅ Merged
Merged: 5/20/2021
Merged by: @jasonmunro

Base: master ← Head: fix-draft-default-smtp

📝 Commits (1)

87305bf Fixed default SMTP server when editing a Draft

📊 Changes

1 file changed (+8 additions, -3 deletions)

View changed files

📝 modules/smtp/modules.php (+8 -3)

📄 Description

When editing a previously saved draft, the default SMTP server was lost in the dropdown select, causing duplication of the draft or saving it in another mailbox. This PR solve this bug by updating the get_primary_recipient method to work with the "Reply-To" header field.

Issues

None

Checklist

None

How2Test

None

Todo

None

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/cypht-org/cypht/pull/475 **Author:** [@henrique-borba](https://github.com/henrique-borba) **Created:** 5/14/2021 **Status:** ✅ Merged **Merged:** 5/20/2021 **Merged by:** [@jasonmunro](https://github.com/jasonmunro) **Base:** `master` ← **Head:** `fix-draft-default-smtp` --- ### 📝 Commits (1) - [`87305bf`](https://github.com/cypht-org/cypht/commit/87305bfaca562e79a8381bc2c36d5e3c71de5eb3) Fixed default SMTP server when editing a Draft ### 📊 Changes **1 file changed** (+8 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `modules/smtp/modules.php` (+8 -3) </details> ### 📄 Description When editing a previously saved draft, the default SMTP server was lost in the dropdown select, causing duplication of the draft or saving it in another mailbox. This PR solve this bug by updating the get_primary_recipient method to work with the "Reply-To" header field. ### Issues  - [X] None ### Checklist  - [X] None ### How2Test   - [X] None ### Todo  - [X] None --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>