mirror of
https://github.com/discourse/discourse.git
synced 2025-02-20 23:11:38 +08:00
a9d20610d4
We relied on backticks to identify and replace site setting names with links. Unfortunately, some translations don't follow this convention, breaking this feature. Additionally, this lets us linkify `category settings` and `watched words` without using HTML in the translations. You may notice that I split the texts we want to linkify into two groups. I did this on purpose to emphasize those that should be translated (regular_links) from those who don't (site_settings_link). If you can think of a better solution, I'm open to suggestions.
74 lines
2.3 KiB
Ruby
74 lines
2.3 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
class ReviewableScoreSerializer < ApplicationSerializer
|
|
|
|
attributes :id, :score, :agree_stats, :status, :reason, :created_at, :reviewed_at
|
|
has_one :user, serializer: BasicUserSerializer, root: 'users'
|
|
has_one :score_type, serializer: ReviewableScoreTypeSerializer
|
|
has_one :reviewable_conversation, serializer: ReviewableConversationSerializer
|
|
has_one :reviewed_by, serializer: BasicUserSerializer, root: 'users'
|
|
|
|
def agree_stats
|
|
{
|
|
agreed: user.user_stat.flags_agreed,
|
|
disagreed: user.user_stat.flags_disagreed,
|
|
ignored: user.user_stat.flags_ignored
|
|
}
|
|
end
|
|
|
|
def reason
|
|
return unless object.reason
|
|
|
|
link_text = I18n.t("reviewables.reasons.site_setting_links.#{object.reason}", default: nil)
|
|
link_text = I18n.t("reviewables.reasons.regular_links.#{object.reason}", default: nil) if link_text.nil?
|
|
|
|
if link_text
|
|
link = build_link_for(object.reason, link_text)
|
|
text = I18n.t("reviewables.reasons.#{object.reason}", link: link, default: nil)
|
|
else
|
|
text = I18n.t("reviewables.reasons.#{object.reason}", default: nil)
|
|
# TODO(roman): Remove after the 2.8 release.
|
|
# The discourse-antivirus and akismet plugins still use the backtick format for settings.
|
|
# It'll be hard to migrate them to the new format without breaking backwards compatibility, so I'm keeping the old behavior for now.
|
|
# Will remove after the 2.8 release.
|
|
linkify_backticks(object.reason, text)
|
|
end
|
|
|
|
text
|
|
end
|
|
|
|
def include_reason?
|
|
reason.present?
|
|
end
|
|
|
|
private
|
|
|
|
def url_for(reason, text)
|
|
case reason
|
|
when 'watched_word'
|
|
"#{Discourse.base_url}/admin/customize/watched_words"
|
|
when 'category'
|
|
"#{Discourse.base_url}/c/#{object.reviewable.category&.name}/edit/settings"
|
|
else
|
|
"#{Discourse.base_url}/admin/site_settings/category/all_results?filter=#{text}"
|
|
end
|
|
end
|
|
|
|
def build_link_for(reason, text)
|
|
return text.gsub('_', ' ') unless scope.is_staff?
|
|
|
|
"<a href=\"#{url_for(reason, text)}\">#{text.gsub('_', ' ')}</a>"
|
|
end
|
|
|
|
def linkify_backticks(reason, text)
|
|
text.gsub!(/`[a-z_]+`/) do |m|
|
|
if scope.is_staff?
|
|
setting = m[1..-2]
|
|
"<a href=\"#{url_for(reason, setting)}\">#{setting.gsub('_', ' ')}</a>"
|
|
else
|
|
m.gsub('_', ' ')
|
|
end
|
|
end
|
|
end
|
|
end
|