Don't know if it makes a big difference or not but in the Nop.Web\Controllers\NewsletterController.cs under the SubscribeNewsletter method,
if (subscribe)
{
if (!subscription.Active)
{
_workflowMessageService.SendNewsLetterSubscriptionActivationMessage(subscription, _workContext.WorkingCurrency.Id);
}
result = _localizationService.GetResource("Newsletter.SubscribeEmailSent");
}
Shouldn't it be:
if (subscribe)
{
if (!subscription.Active)
{
_workflowMessageService.SendNewsLetterSubscriptionActivationMessage(subscription, _workContext.WorkingLanguage.Id);
}
result = _localizationService.GetResource("Newsletter.SubscribeEmailSent");
}