FIX: Notifications shouldn't use user locale unless allow_user_locale is enabled

This commit is contained in:
Gerhard Schlager 2018-09-04 11:47:05 +02:00
parent d9be4f47e8
commit c788737eed

View File

@ -20,10 +20,15 @@ class UserNotifications < ActionMailer::Base
end end
def signup_after_approval(user, opts = {}) def signup_after_approval(user, opts = {})
locale = user_locale(user)
tips = I18n.t('system_messages.usage_tips.text_body_template',
base_url: Discourse.base_url,
locale: locale)
build_email(user.email, build_email(user.email,
template: 'user_notifications.signup_after_approval', template: 'user_notifications.signup_after_approval',
locale: user_locale(user), locale: locale,
new_user_tips: I18n.t('system_messages.usage_tips.text_body_template', base_url: Discourse.base_url, locale: locale)) new_user_tips: tips)
end end
def notify_old_email(user, opts = {}) def notify_old_email(user, opts = {})
@ -320,7 +325,7 @@ class UserNotifications < ActionMailer::Base
protected protected
def user_locale(user) def user_locale(user)
(user.locale.present? && I18n.available_locales.include?(user.locale.to_sym)) ? user.locale : nil user.effective_locale
end end
def email_post_markdown(post, add_posted_by = false) def email_post_markdown(post, add_posted_by = false)