NewsLetter Controller

This topic was automatically closed 365 days after the last reply. New replies are no longer allowed.
12 years ago
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");
                    }
12 years ago
You're absolutely right. Thanks for reporting!
This topic was automatically closed 365 days after the last reply. New replies are no longer allowed.