diff options
author | Côme Chilliet <come.chilliet@nextcloud.com> | 2025-01-20 17:12:19 +0100 |
---|---|---|
committer | Côme Chilliet <come.chilliet@nextcloud.com> | 2025-01-20 17:12:19 +0100 |
commit | b8de5e0d0973d5fcf72898245ef430ff85806861 (patch) | |
tree | 8c7a775019729dc3bb41a9facaa263e2af3d4412 | |
parent | c1b3f6a6172bd232a0d2d5bbbf0d750d5ced310a (diff) | |
download | nextcloud-server-fix/fix-email-setupcheck-with-null-smtpmode.tar.gz nextcloud-server-fix/fix-email-setupcheck-with-null-smtpmode.zip |
fix(settings): Fix setup check when mail_smptmode is set to "null"fix/fix-email-setupcheck-with-null-smtpmode
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
-rw-r--r-- | apps/settings/lib/SetupChecks/EmailTestSuccessful.php | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/apps/settings/lib/SetupChecks/EmailTestSuccessful.php b/apps/settings/lib/SetupChecks/EmailTestSuccessful.php index a8f527f9dba..8cad8e82156 100644 --- a/apps/settings/lib/SetupChecks/EmailTestSuccessful.php +++ b/apps/settings/lib/SetupChecks/EmailTestSuccessful.php @@ -46,7 +46,9 @@ class EmailTestSuccessful implements ISetupCheck { } public function run(): SetupResult { - if ($this->wasEmailTestSuccessful()) { + if ($this->config->getSystemValueString('mail_smtpmode', 'smtp') === 'null') { + return SetupResult::success($this->l10n->t('Mail delivery is disabled by instance config "%s".', ['mail_smtpmode'])); + } elseif ($this->wasEmailTestSuccessful()) { return SetupResult::success($this->l10n->t('Email test was successfully sent')); } else { // If setup check could link to settings pages, this one should link to OC.generateUrl('/settings/admin') |