mirror of
https://github.com/discourse/discourse.git
synced 2026-03-04 01:15:08 +08:00
e154a90427
Some upcoming changes are all-or-nothing — they affect the whole site or only admins, so enabling them per-group makes no sense. This adds a `disallow_enabled_for_groups` flag to upcoming change metadata that restricts the "Enabled for" dropdown to only "Everyone" and "No One", and clears any existing group assignments server-side as a safety net.
96 lines
2.7 KiB
Ruby
96 lines
2.7 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
class UpcomingChanges::Toggle
|
|
include Service::Base
|
|
|
|
# For cases like the UpcomingChanges::Promote where we don't want to log
|
|
# the change again since it's already being logged there.
|
|
options { attribute :log_change, default: true }
|
|
|
|
params do
|
|
attribute :setting_name, :string
|
|
attribute :enabled, :boolean
|
|
validates :setting_name, presence: true
|
|
validates :enabled, inclusion: [true, false]
|
|
|
|
def upcoming_change_event
|
|
enabled ? :upcoming_change_enabled : :upcoming_change_disabled
|
|
end
|
|
end
|
|
|
|
policy :current_user_is_admin
|
|
policy :setting_is_available
|
|
transaction { step :toggle }
|
|
|
|
step :clear_groups_if_disallowed
|
|
|
|
only_if(:should_log_change) do
|
|
step :log_change
|
|
step :log_event
|
|
end
|
|
|
|
step :trigger_event
|
|
|
|
private
|
|
|
|
def current_user_is_admin(guardian:)
|
|
guardian.is_admin?
|
|
end
|
|
|
|
def setting_is_available(params:)
|
|
SiteSetting.respond_to?(params.setting_name)
|
|
end
|
|
|
|
def toggle(params:, guardian:, options:)
|
|
context[:previous_value] = SiteSetting.public_send(params.setting_name)
|
|
|
|
# TODO (martin) Remove this once we release upcoming changes,
|
|
# otherwise it will be confusing for people to see log messages
|
|
# about upcoming changes via "What's new?" experimental toggles
|
|
# before we update that UI.
|
|
if SiteSetting.enable_upcoming_changes
|
|
SiteSetting.send("#{params.setting_name}=", params.enabled)
|
|
else
|
|
SiteSetting.public_send("#{params.setting_name}=", params.enabled)
|
|
end
|
|
end
|
|
|
|
def clear_groups_if_disallowed(params:)
|
|
metadata = SiteSetting.upcoming_change_metadata[params.setting_name.to_sym]
|
|
return if !metadata || !metadata[:disallow_enabled_for_groups]
|
|
|
|
SiteSettingGroup.find_by(name: params.setting_name)&.destroy!
|
|
SiteSetting.refresh_site_setting_group_ids!
|
|
end
|
|
|
|
def should_log_change(options:)
|
|
options.log_change
|
|
end
|
|
|
|
def log_change(params:, guardian:, options:)
|
|
if SiteSetting.enable_upcoming_changes
|
|
StaffActionLogger.new(guardian.user).log_upcoming_change_toggle(
|
|
params.setting_name,
|
|
context[:previous_value],
|
|
params.enabled,
|
|
{ context: I18n.t("staff_action_logs.upcoming_changes.log_manually_toggled") },
|
|
)
|
|
else
|
|
SiteSetting.log(params.setting_name, params.enabled, context[:previous_value], guardian.user)
|
|
end
|
|
end
|
|
|
|
def log_event(params:, guardian:, options:)
|
|
return unless SiteSetting.enable_upcoming_changes
|
|
|
|
UpcomingChangeEvent.create!(
|
|
event_type: params.enabled ? :manual_opt_in : :manual_opt_out,
|
|
upcoming_change_name: params.setting_name,
|
|
acting_user: guardian.user,
|
|
)
|
|
end
|
|
|
|
def trigger_event(params:)
|
|
DiscourseEvent.trigger(params.upcoming_change_event, params.setting_name)
|
|
end
|
|
end
|
