Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion app/jobs/regular/translate_categories.rb
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ class TranslateCategories < ::Jobs::Base

def execute(args)
return unless SiteSetting.translator_enabled
return unless SiteSetting.experimental_category_translation
return unless SiteSetting.experimental_content_translation

locales = SiteSetting.automatic_translation_target_languages.split("|")
return if locales.blank?
Expand Down
2 changes: 1 addition & 1 deletion app/jobs/scheduled/automatic_category_translation.rb
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ class AutomaticCategoryTranslation < ::Jobs::Scheduled

def execute(args)
return unless SiteSetting.translator_enabled
return unless SiteSetting.experimental_category_translation
return unless SiteSetting.experimental_content_translation

locales = SiteSetting.automatic_translation_target_languages.split("|")
return if locales.blank?
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ def self.translate_translatable!(translatable, target_locale_sym = I18n.locale)
raise "Not Implemented"
end

def self.translate_text(text, target_locale_sym = I18n.locale)
def self.translate_text!(text, target_locale_sym = I18n.locale)
raise "Not Implemented"
end

Expand Down
2 changes: 1 addition & 1 deletion config/settings.yml
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ discourse_translator:
experimental_inline_translation:
default: false
client: true
experimental_category_translation:
experimental_content_translation:
default: false
hidden: true
discourse_translator_verbose_logs:
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
# frozen_string_literal: true

class RenameSiteSettingContentTranslation < ActiveRecord::Migration[7.2]
def up
execute <<~SQL
UPDATE site_settings
SET name = 'experimental_content_translation'
WHERE name = 'experimental_category_translation';
SQL
end

def down
execute <<~SQL
UPDATE site_settings
SET name = 'experimental_category_translation'
WHERE name = 'experimental_content_translation';
SQL
end
end
8 changes: 3 additions & 5 deletions spec/jobs/translate_categories_spec.rb
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
# frozen_string_literal: true

require "rails_helper"

describe Jobs::TranslateCategories do
subject(:job) { described_class.new }

Expand All @@ -15,7 +13,7 @@ def localize_all_categories(*locales)

before do
SiteSetting.translator_enabled = true
SiteSetting.experimental_category_translation = true
SiteSetting.experimental_content_translation = true
SiteSetting.automatic_translation_backfill_rate = 100
SiteSetting.automatic_translation_target_languages = "pt|zh_CN"

Expand All @@ -31,8 +29,8 @@ def localize_all_categories(*locales)
job.execute({})
end

it "does nothing when experimental_category_translation is disabled" do
SiteSetting.experimental_category_translation = false
it "does nothing when experimental_content_translation is disabled" do
SiteSetting.experimental_content_translation = false

translator.expects(:translate_text!).never

Expand Down
Loading