Use around_action to set locale in admin/notification mailers (#38140)

This commit is contained in:
Matt Jankowski 2026-03-11 06:18:24 -04:00 committed by GitHub
parent d39f867931
commit d047a10cf5
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 25 additions and 39 deletions

View File

@ -11,30 +11,26 @@ class AdminMailer < ApplicationMailer
after_action :set_important_headers!, only: :new_critical_software_updates
around_action :set_locale
default to: -> { @me.user_email }
def new_report(report)
@report = report
locale_for_account(@me) do
mail subject: default_i18n_subject(instance: @instance, id: @report.id)
end
mail subject: default_i18n_subject(instance: @instance, id: @report.id)
end
def new_appeal(appeal)
@appeal = appeal
locale_for_account(@me) do
mail subject: default_i18n_subject(instance: @instance, username: @appeal.account.username)
end
mail subject: default_i18n_subject(instance: @instance, username: @appeal.account.username)
end
def new_pending_account(user)
@account = user.account
locale_for_account(@me) do
mail subject: default_i18n_subject(instance: @instance, username: @account.username)
end
mail subject: default_i18n_subject(instance: @instance, username: @account.username)
end
def new_trends(links, tags, statuses)
@ -42,31 +38,23 @@ class AdminMailer < ApplicationMailer
@tags = tags
@statuses = statuses
locale_for_account(@me) do
mail subject: default_i18n_subject(instance: @instance)
end
mail subject: default_i18n_subject(instance: @instance)
end
def new_software_updates
@software_updates = SoftwareUpdate.by_version
locale_for_account(@me) do
mail subject: default_i18n_subject(instance: @instance)
end
mail subject: default_i18n_subject(instance: @instance)
end
def new_critical_software_updates
@software_updates = SoftwareUpdate.urgent.by_version
locale_for_account(@me) do
mail subject: default_i18n_subject(instance: @instance)
end
mail subject: default_i18n_subject(instance: @instance)
end
def auto_close_registrations
locale_for_account(@me) do
mail subject: default_i18n_subject(instance: @instance)
end
mail subject: default_i18n_subject(instance: @instance)
end
private
@ -79,6 +67,10 @@ class AdminMailer < ApplicationMailer
@instance = Rails.configuration.x.local_domain
end
def set_locale(&block)
locale_for_account(@me, &block)
end
def set_important_headers!
headers(
'Importance' => 'high',

View File

@ -15,6 +15,8 @@ class NotificationMailer < ApplicationMailer
before_deliver :verify_functional_user
around_action :set_locale
default to: -> { email_address_with_name(@user.email, @me.username) }
layout 'mailer'
@ -22,45 +24,33 @@ class NotificationMailer < ApplicationMailer
def mention
return if @status.blank?
locale_for_account(@me) do
mail subject: default_i18n_subject(name: @status.account.acct)
end
mail subject: default_i18n_subject(name: @status.account.acct)
end
def quote
return if @status.blank?
locale_for_account(@me) do
mail subject: default_i18n_subject(name: @status.account.acct)
end
mail subject: default_i18n_subject(name: @status.account.acct)
end
def follow
locale_for_account(@me) do
mail subject: default_i18n_subject(name: @account.acct)
end
mail subject: default_i18n_subject(name: @account.acct)
end
def favourite
return if @status.blank?
locale_for_account(@me) do
mail subject: default_i18n_subject(name: @account.acct)
end
mail subject: default_i18n_subject(name: @account.acct)
end
def reblog
return if @status.blank?
locale_for_account(@me) do
mail subject: default_i18n_subject(name: @account.acct)
end
mail subject: default_i18n_subject(name: @account.acct)
end
def follow_request
locale_for_account(@me) do
mail subject: default_i18n_subject(name: @account.acct)
end
mail subject: default_i18n_subject(name: @account.acct)
end
private
@ -81,6 +71,10 @@ class NotificationMailer < ApplicationMailer
@account = @notification.from_account
end
def set_locale(&block)
locale_for_account(@me, &block)
end
def verify_functional_user
throw(:abort) unless @user.functional?
end