Fix locales that require a 'host' variable (#5273)

rubygems · Dec 13, 2024 · 0a81b21 · 0a81b21
1 parent 8d341cc
commit 0a81b21
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 8 deletions.
diff --git a/app/mailers/mailer.rb b/app/mailers/mailer.rb
@@ -43,20 +43,20 @@ def admin_manual(user, subject, body)
 
   def deletion_complete(email)
     mail to: email,
-         subject: I18n.t("mailer.deletion_complete.subject")
+         subject: I18n.t("mailer.deletion_complete.subject", host: Gemcutter::HOST_DISPLAY)
   end
 
   def deletion_failed(email)
     mail to: email,
-         subject: I18n.t("mailer.deletion_failed.subject")
+         subject: I18n.t("mailer.deletion_failed.subject", host: Gemcutter::HOST_DISPLAY)
   end
 
   def notifiers_changed(user_id)
     @user = User.find(user_id)
     @ownerships = @user.ownerships.by_indexed_gem_name
 
     mail to: @user.email,
-         subject: I18n.t("mailer.notifiers_changed.subject",
+         subject: I18n.t("mailer.notifiers_changed.subject", host: Gemcutter::HOST_DISPLAY,
            default: "You changed your RubyGems.org email notification settings")
   end
 

diff --git a/app/mailers/web_hooks_mailer.rb b/app/mailers/web_hooks_mailer.rb
@@ -6,7 +6,7 @@ def webhook_deleted(user_id, rubygem_id, url, failure_count)
     @failure_count = failure_count
 
     mail to: @user.email,
-         subject: t("mailer.web_hook_deleted.subject") do |format|
+         subject: t("mailer.web_hook_deleted.subject", host: Gemcutter::HOST_DISPLAY) do |format|
            format.html
            format.text
          end
@@ -17,7 +17,7 @@ def webhook_disabled(web_hook)
     @delete_command = "gem install gemcutter && gem webhook#{" #{web_hook.rubygem.name}" unless web_hook.global?} --remove '#{web_hook.url}'"
 
     mail to: web_hook.user.email,
-         subject: t("mailer.web_hook_disabled.subject") do |format|
+         subject: t("mailer.web_hook_disabled.subject", host: Gemcutter::HOST_DISPLAY) do |format|
            format.html
            format.text
          end

diff --git a/app/views/mailer/email_confirmation.text.erb b/app/views/mailer/email_confirmation.text.erb
@@ -1,4 +1,4 @@
 Hi <%= @user.handle %>
 
-<%= t("mailer.email_confirmation.welcome_message") %>
+<%= t("mailer.email_confirmation.welcome_message", host: Gemcutter::HOST_DISPLAY) %>
 <%= update_email_confirmations_url(token: @user.confirmation_token.html_safe) %>
diff --git a/test/integration/notification_settings_test.rb b/test/integration/notification_settings_test.rb
@@ -42,7 +42,7 @@ class NotificationSettingsTest < SystemTest
 
     assert_emails 1
 
-    assert_equal I18n.t("mailer.notifiers_changed.subject"), last_email.subject
+    assert_equal I18n.t("mailer.notifiers_changed.subject", host: Gemcutter::HOST_DISPLAY), last_email.subject
 
     assert_selector "#flash_notice", text: I18n.t("notifiers.update.success")
 

diff --git a/test/system/avo/web_hooks_test.rb b/test/system/avo/web_hooks_test.rb
@@ -65,6 +65,6 @@ class Avo::WebHooksSystemTest < ApplicationSystemTestCase
 
     perform_enqueued_jobs only: ActionMailer::MailDeliveryJob
 
-    assert_equal I18n.t("mailer.web_hook_deleted.subject"), last_email.subject
+    assert_equal I18n.t("mailer.web_hook_deleted.subject", host: Gemcutter::HOST_DISPLAY), last_email.subject
   end
 end