diff --git a/src/Umbraco.Core/Configuration/Models/GlobalSettings.cs b/src/Umbraco.Core/Configuration/Models/GlobalSettings.cs
index c880830274..94b8d88ed3 100644
--- a/src/Umbraco.Core/Configuration/Models/GlobalSettings.cs
+++ b/src/Umbraco.Core/Configuration/Models/GlobalSettings.cs
@@ -159,6 +159,10 @@ namespace Umbraco.Cms.Core.Configuration.Models
public bool IsSmtpServerConfigured => !string.IsNullOrWhiteSpace(Smtp?.Host);
///
+ /// Gets a value indicating whether there is a physical pickup directory configured.
+ ///
+ public bool IsPickupDirectoryLocationConfigured => !string.IsNullOrWhiteSpace(Smtp?.PickupDirectoryLocation);
+
/// Gets a value indicating whether TinyMCE scripting sanitization should be applied
///
[DefaultValue(StaticSanitizeTinyMce)]
@@ -174,4 +178,4 @@ namespace Umbraco.Cms.Core.Configuration.Models
[DefaultValue(StaticSqlWriteLockTimeOut)]
public TimeSpan SqlWriteLockTimeOut { get; } = TimeSpan.Parse(StaticSqlWriteLockTimeOut);
}
-}
+}
\ No newline at end of file
diff --git a/src/Umbraco.Infrastructure/Mail/EmailSender.cs b/src/Umbraco.Infrastructure/Mail/EmailSender.cs
index 854c2fb51e..5da5e5097d 100644
--- a/src/Umbraco.Infrastructure/Mail/EmailSender.cs
+++ b/src/Umbraco.Infrastructure/Mail/EmailSender.cs
@@ -75,15 +75,13 @@ namespace Umbraco.Cms.Infrastructure.Mail
}
}
- var isPickupDirectoryConfigured = !string.IsNullOrWhiteSpace(_globalSettings.Smtp?.PickupDirectoryLocation);
-
- if (_globalSettings.IsSmtpServerConfigured == false && !isPickupDirectoryConfigured)
+ if (!_globalSettings.IsSmtpServerConfigured && !_globalSettings.IsPickupDirectoryLocationConfigured)
{
_logger.LogDebug("Could not send email for {Subject}. It was not handled by a notification handler and there is no SMTP configured.", message.Subject);
return;
}
- if (isPickupDirectoryConfigured && !string.IsNullOrWhiteSpace(_globalSettings.Smtp?.From))
+ if (_globalSettings.IsPickupDirectoryLocationConfigured && !string.IsNullOrWhiteSpace(_globalSettings.Smtp?.From))
{
// The following code snippet is the recommended way to handle PickupDirectoryLocation.
// See more https://github.com/jstedfast/MailKit/blob/master/FAQ.md#q-how-can-i-send-email-to-a-specifiedpickupdirectory
@@ -154,7 +152,10 @@ namespace Umbraco.Cms.Infrastructure.Mail
///
///
/// We assume this is possible if either an event handler is registered or an smtp server is configured
+ /// or a pickup directory location is configured
///
- public bool CanSendRequiredEmail() => _globalSettings.IsSmtpServerConfigured || _notificationHandlerRegistered;
+ public bool CanSendRequiredEmail() => _globalSettings.IsSmtpServerConfigured
+ || _globalSettings.IsPickupDirectoryLocationConfigured
+ || _notificationHandlerRegistered;
}
-}
+}
\ No newline at end of file