DEV: Remove the anon language switcher and update settings

.discourse-compatibility

@@ -1,3 +1,4 @@
+< 3.5.0.beta6-dev: 7e0a51707f98eb8ec49fa39c519075b1766d54be
 < 3.5.0.beta5-dev: 5c44f829ef82ded3416b0cddc521e9e6d62ed534
 < 3.5.0.beta4-dev: 14ca3c07efa0a80712a4cbb8ca455c32a727adec
 < 3.5.0.beta2-dev: 5f24835801fdc7cb98e1bcf42d2ab2e49e609921

app/jobs/regular/detect_translate_post.rb

@@ -25,7 +25,7 @@ def execute(args)
         return
       end
 
-      locales = SiteSetting.automatic_translation_target_languages.split("|")
+      locales = SiteSetting.experimental_content_localization_supported_locales.split("|")
       return if locales.blank?
 
       locales.each do |locale|

app/jobs/regular/detect_translate_topic.rb

@@ -26,7 +26,7 @@ def execute(args)
         return
       end
 
-      locales = SiteSetting.automatic_translation_target_languages.split("|")
+      locales = SiteSetting.experimental_content_localization_supported_locales.split("|")
       return if locales.blank?
 
       locales.each do |locale|

app/jobs/regular/translate_categories.rb

@@ -11,7 +11,7 @@ def execute(args)
       return unless SiteSetting.translator_enabled
       return unless SiteSetting.experimental_content_translation
 
-      locales = SiteSetting.automatic_translation_target_languages.split("|")
+      locales = SiteSetting.experimental_content_localization_supported_locales.split("|")
       return if locales.blank?
 
       cat_id = args[:from_category_id] || Category.order(:id).first&.id

app/jobs/regular/translate_posts.rb

@@ -11,7 +11,7 @@ def execute(args)
       return unless SiteSetting.translator_enabled
       return unless SiteSetting.experimental_content_translation
 
-      locales = SiteSetting.automatic_translation_target_languages.split("|")
+      locales = SiteSetting.experimental_content_localization_supported_locales.split("|")
       return if locales.blank?
 
       limit = args[:limit] || BATCH_SIZE

app/jobs/regular/translate_topics.rb

@@ -11,7 +11,7 @@ def execute(args)
       return unless SiteSetting.translator_enabled
       return unless SiteSetting.experimental_content_translation
 
-      locales = SiteSetting.automatic_translation_target_languages.split("|")
+      locales = SiteSetting.experimental_content_localization_supported_locales.split("|")
       return if locales.blank?
 
       limit = args[:limit] || BATCH_SIZE

app/jobs/scheduled/category_translation_backfill.rb

@@ -9,8 +9,7 @@ def execute(args)
       return unless SiteSetting.translator_enabled
       return unless SiteSetting.experimental_content_translation
 
-      locales = SiteSetting.automatic_translation_target_languages.split("|")
-      return if locales.blank?
+      return if SiteSetting.experimental_content_localization_supported_locales.blank?
 
       Jobs.enqueue(:translate_categories)
     end

app/jobs/scheduled/post_translation_backfill.rb

@@ -9,7 +9,7 @@ def execute(args)
       return unless SiteSetting.translator_enabled
       return unless SiteSetting.experimental_content_translation
 
-      return if SiteSetting.automatic_translation_target_languages.blank?
+      return if SiteSetting.experimental_content_localization_supported_locales.blank?
       return if SiteSetting.automatic_translation_backfill_rate == 0
 
       Jobs.enqueue(:translate_posts, limit: SiteSetting.automatic_translation_backfill_rate)

app/jobs/scheduled/topic_translation_backfill.rb

@@ -9,7 +9,7 @@ def execute(args)
       return unless SiteSetting.translator_enabled
       return unless SiteSetting.experimental_content_translation
 
-      return if SiteSetting.automatic_translation_target_languages.blank?
+      return if SiteSetting.experimental_content_localization_supported_locales.blank?
       return if SiteSetting.automatic_translation_backfill_rate == 0
 
       Jobs.enqueue(:translate_topics, limit: SiteSetting.automatic_translation_backfill_rate)

assets/javascripts/discourse/initializers/extend-for-translate-button.js

@@ -1,5 +1,4 @@
 import { withPluginApi } from "discourse/lib/plugin-api";
-import LanguageSwitcher from "../components/language-switcher";
 import ToggleTranslationButton from "../components/post-menu/toggle-translation-button";
 import TranslatedPost from "../components/translated-post";
 
@@ -9,20 +8,6 @@ function initializeTranslation(api) {
     return;
   }
 
-  const currentUser = api.getCurrentUser();
-
-  if (
-    !currentUser &&
-    siteSettings.experimental_anon_language_switcher &&
-    siteSettings.automatic_translation_target_languages
-  ) {
-    api.headerIcons.add(
-      "discourse-translator_language-switcher",
-      LanguageSwitcher,
-      { before: ["search"] }
-    );
-  }
-
   customizePostMenu(api);
 }
 

config/locales/server.en.yml

@@ -17,11 +17,6 @@ en:
     translator_azure_custom_subdomain: "Required if using a Virtual Network or Firewall for Azure Cognitive Services. Note: Only enter the custom subdomain not the full custom endpoint."
     restrict_translation_by_group: "Only allowed groups can translate"
     restrict_translation_by_poster_group: "Only allow translation of posts made by users in allowed groups. If empty, allow translations of posts from all users."
-    experimental_anon_language_switcher: "Enable experimental language switcher. This will allow site visitors who are not logged in to switch between translated versions of Discourse and user-contributed content in topics."
-    errors:
-      needs_nonzero_backfill: "Automatic language translation requires the 'automatic_translation_backfill_rate' hidden setting to be a non-zero value. Please approach your site administrator to increase this limit."
-      experimental_anon_language_switcher_requirements: "The experimental language switcher requires the `set locale from cookie` site setting to be enabled, and the `automatic translation target languages` to have at least one language."
-    automatic_translation_target_languages: "The languages to automatically translate user content (posts, topics) to. If empty, no languages will be automatically translated."
   translator:
     failed:
       topic: "The translator is unable to translate this topic's title (%{source_locale}) to your language (%{target_locale})."

config/settings.yml

@@ -14,14 +14,6 @@ discourse_translator:
       - Yandex
       - LibreTranslate
     validator: "DiscourseTranslator::Validators::TranslatorSelectionValidator"
-  automatic_translation_target_languages:
-    default: ""
-    type: list
-    client: true
-    list_type: named
-    allow_any: false
-    choices: "DiscourseTranslator::TranslatableLanguagesSetting.values"
-    validator: "DiscourseTranslator::Validators::TranslatableLanguagesValidator"
   automatic_translation_backfill_rate:
     default: 0
     client: false
@@ -122,10 +114,6 @@ discourse_translator:
     default: ""
     client: true
     type: group_list
-  experimental_anon_language_switcher:
-    default: false
-    client: true
-    validator: "DiscourseTranslator::Validators::LanguageSwitcherSettingValidator"
   experimental_content_translation:
     default: false
     hidden: true

db/migrate/20250528040217_rename_translation_target_languages_to_content_localization_supported_locales.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+class RenameTranslationTargetLanguagesToContentLocalizationSupportedLocales < ActiveRecord::Migration[
+  7.2
+]
+  def up
+    setting_exists =
+      DB.query_single(
+        "SELECT 1 FROM site_settings WHERE name = 'experimental_content_localization_supported_locales' LIMIT 1",
+      ).present?
+
+    if setting_exists
+      execute "DELETE FROM site_settings WHERE name = 'automatic_translation_target_languages'"
+    else
+      execute "UPDATE site_settings SET name = 'experimental_content_localization_supported_locales' WHERE name = 'automatic_translation_target_languages'"
+    end
+  end
+
+  def down
+    raise ActiveRecord::IrreversibleMigration
+  end
+end

lib/discourse_translator/parallel_text_translation.rb

@@ -3,20 +3,14 @@
 module DiscourseTranslator
   class ParallelTextTranslation
     def inject(plugin)
-      # in parallel text translations,
-      # we don't want to send the post for detection if automatic translation already happens,
-      # as automatic translations send content for language detection as a side effect of translating
-
       plugin.on(:post_process_cooked) do |_, post|
-        if SiteSetting.automatic_translation_target_languages.blank? &&
-             Guardian.new.can_detect_language?(post) && post.user_id > 0
+        if Guardian.new.can_detect_language?(post) && post.user_id > 0
           Jobs.enqueue(:detect_translatable_language, type: "Post", translatable_id: post.id)
         end
       end
 
       plugin.on(:topic_created) do |topic|
-        if SiteSetting.automatic_translation_target_languages.blank? &&
-             Guardian.new.can_detect_language?(topic.first_post) && topic.user_id > 0
+        if Guardian.new.can_detect_language?(topic.first_post) && topic.user_id > 0
           Jobs.enqueue(:detect_translatable_language, type: "Topic", translatable_id: topic.id)
         end
       end

spec/jobs/detect_translate_post_spec.rb

@@ -10,7 +10,7 @@
     SiteSetting.translator_enabled = true
     SiteSetting.experimental_content_translation = true
     SiteSetting.automatic_translation_backfill_rate = 1
-    SiteSetting.automatic_translation_target_languages = locales.join("|")
+    SiteSetting.experimental_content_localization_supported_locales = locales.join("|")
   end
 
   it "does nothing when translator is disabled" do
@@ -45,7 +45,7 @@
   end
 
   it "does not translate when no target languages are configured" do
-    SiteSetting.automatic_translation_target_languages = ""
+    SiteSetting.experimental_content_localization_supported_locales = ""
     DiscourseTranslator::PostLocaleDetector.expects(:detect_locale).with(post).returns("en")
     DiscourseTranslator::PostTranslator.expects(:translate).never
 

spec/jobs/detect_translate_topic_spec.rb

@@ -10,7 +10,7 @@
     SiteSetting.translator_enabled = true
     SiteSetting.experimental_content_translation = true
     SiteSetting.automatic_translation_backfill_rate = 1
-    SiteSetting.automatic_translation_target_languages = locales.join("|")
+    SiteSetting.experimental_content_localization_supported_locales = locales.join("|")
   end
 
   it "does nothing when translator is disabled" do
@@ -45,7 +45,7 @@
   end
 
   it "does not translate when no target languages are configured" do
-    SiteSetting.automatic_translation_target_languages = ""
+    SiteSetting.experimental_content_localization_supported_locales = ""
     DiscourseTranslator::TopicLocaleDetector.expects(:detect_locale).with(topic).returns("en")
     DiscourseTranslator::TopicTranslator.expects(:translate).never