From 136f7dbf78c19bd6af0fbd049eb233028312335b Mon Sep 17 00:00:00 2001 From: Roman Rizzi Date: Fri, 25 Mar 2022 09:36:39 -0300 Subject: [PATCH] DEV: Remove old link building code. (#16121) We have a new API introduced [here](https://github.com/discourse/discourse/pull/14553). --- app/serializers/reviewable_score_serializer.rb | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/app/serializers/reviewable_score_serializer.rb b/app/serializers/reviewable_score_serializer.rb index a16bd752c8..549bd4536a 100644 --- a/app/serializers/reviewable_score_serializer.rb +++ b/app/serializers/reviewable_score_serializer.rb @@ -40,12 +40,6 @@ class ReviewableScoreSerializer < ApplicationSerializer 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) if text end text @@ -86,15 +80,4 @@ class ReviewableScoreSerializer < ApplicationSerializer "#{text.gsub('_', ' ')}" end - - def linkify_backticks(reason, text) - text.gsub!(/`[a-z_]+`/) do |m| - if scope.is_staff? - setting = m[1..-2] - "#{setting.gsub('_', ' ')}" - else - m.gsub('_', ' ') - end - end - end end