mirror of
https://gh.wpcy.net/https://github.com/discourse/discourse.git
synced 2026-05-01 12:58:30 +08:00
71b96243e1
There exists a `localization_guardian` that checks if a user can localize based on settings like - `content_localization_allowed_groups` - `content_localization_allow_author_localization` However, it missed out checking if the user can even see the model. This commit fixes that by adding the checks. This issue was found as I was adding a new model (`tags`) and discovered they were absent for the older models. This commit also introduces a small refactor that `.find`s the model first on the controller and passes the object, so that the subsequent services do not have to `.find` them again.
12 lines
346 B
Ruby
12 lines
346 B
Ruby
# frozen_string_literal: true
|
|
|
|
class TopicLocalizationDestroyer
|
|
def self.destroy(topic:, locale:, acting_user:)
|
|
Guardian.new(acting_user).ensure_can_localize_topic!(topic)
|
|
|
|
localization = TopicLocalization.find_by(topic_id: topic.id, locale: locale)
|
|
raise Discourse::NotFound unless localization
|
|
|
|
localization.destroy
|
|
end
|
|
end
|
