[PR #800] [MERGED] Fixed adding one server multiple times [Only nux module] #1054

New issue

Closed

opened 2026-02-25 21:37:05 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 21:37:05 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/800
Author: @josaphatim
Created: 10/20/2023
Status: ✅ Merged
Merged: 10/23/2023
Merged by: @kroky

Base: master ← Head: fixed-nux-issues-and-add-postale-migadu-sieve

📝 Commits (1)

be6706c Fixed adding one server multiple times [Only nux module]

📊 Changes

2 files changed (+14 additions, -2 deletions)

View changed files

📝 modules/imap/handler_modules.php (+1 -1)
📝 modules/nux/modules.php (+13 -1)

📄 Description

There was a bug on nux module as no check was done when adding server. One server could be added many times contrarily to adding server from iMAP or SMTP sections. With this merge request now if IMAP server is already added an error is returned and process is stop but if a smtp server is already added we don't add it again.

_{🔄 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/800 **Author:** [@josaphatim](https://github.com/josaphatim) **Created:** 10/20/2023 **Status:** ✅ Merged **Merged:** 10/23/2023 **Merged by:** [@kroky](https://github.com/kroky) **Base:** `master` ← **Head:** `fixed-nux-issues-and-add-postale-migadu-sieve` --- ### 📝 Commits (1) - [`be6706c`](https://github.com/cypht-org/cypht/commit/be6706c37c13f051efab5b678bd1814adb3cd104) Fixed adding one server multiple times [Only nux module] ### 📊 Changes **2 files changed** (+14 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `modules/imap/handler_modules.php` (+1 -1) 📝 `modules/nux/modules.php` (+13 -1) </details> ### 📄 Description There was a bug on nux module as no check was done when adding server. One server could be added many times contrarily to adding server from iMAP or SMTP sections. With this merge request now if IMAP server is already added an error is returned and process is stop but if a smtp server is already added we don't add it again. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>