当标签同义词在主题上已存在时添加错误

Overgrow · 2020 年2 月 29 日 13:14

看起来当前的代码无法处理在创建同义词时，主题已经同时包含这两个标签的情况。

Started POST "/tag/solstrip/synonyms" for 3.14.14.5 at 2020-02-29 12:07:28 +0000
Processing by TagsController#create_synonyms as */*
  Parameters: {"synonyms"=>["solstrips"], "tag_id"=>"solstrip"}
Completed 500 Internal Server Error in 46ms (ActiveRecord: 0.0ms | Allocations: 6385)
ActiveRecord::RecordNotUnique (PG::UniqueViolation: ERROR:  duplicate key value violates unique constraint "index_topic_tags_on_topic_id_and_tag_id"
DETAIL:  Key (topic_id, tag_id)=(14651, 310) already exists.
)
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-mini-profiler-1.1.6/lib/patches/db/pg.rb:69:in `exec_params'
Failed to handle exception in exception app middleware : PG::UniqueViolation: ERROR:  duplicate key value violates unique constraint "index_topic_tags_on_topic_id_and_tag_id"
DETAIL:  Key (topic_id, tag_id)=(14651, 310) already exists.

github.com/discourse/discourse

lib/discourse_tagging.rb

c31039d51


      
          # Returns true if all were added successfully, or an Array of the
          # tags that failed to be added, with errors on each Tag.
          def self.add_or_create_synonyms_by_name(target_tag, synonym_names)
            tag_names = DiscourseTagging.tags_for_saving(synonym_names, Guardian.new(Discourse.system_user)) || []
            existing = Tag.where_name(tag_names).all
            target_tag.synonyms << existing
            (tag_names - target_tag.synonyms.map(&:name)).each do |name|
              target_tag.synonyms << Tag.create(name: name)
            end
            successful = existing.select { |t| !t.errors.present? }
            TopicTag.where(tag_id: successful.map(&:id)).update_all(tag_id: target_tag.id)
            Scheduler::Defer.later "Update tag topic counts" do
              Tag.ensure_consistency!
            end
            (existing - successful).presence || true
          end
          
          def self.muted_tags(user)
            return [] unless user
            TagUser.lookup(user, :muted).joins(:tag).pluck('tags.name')
          end

有人能确认一下这个问题吗？如果可以的话，也许可以将其移到 #bugs 分类？谢谢！

justani · 2020 年4 月 28 日 15:22

是的，这个问题也出现在我们的 Discourse 安装中。

supermathie · 2020 年7 月 17 日 21:17

已确认此问题在最新代码中仍然存在。

sam · 2020 年7 月 21 日 00:40

@vinothkannans 你能处理一下这件事吗？

vinothkannans · 2020 年7 月 21 日 15:33

此问题已在以下提交中修复

话题		回复	浏览量
Internal Server Error on using some tags (ERROR: duplicate key value violates unique constraint "index_tags_on_name") Bug	20	3311	2019 年11 月 12 日
Adding synonym to the same tag gets stuck Bug	11	1339	2020 年11 月 23 日
Main tag sharing same name and url with synonym Support	0	600	2020 年6 月 28 日
New Tags Creation Feature tags , completed	20	1205	2024 年4 月 16 日
There are some duplicated tags due to database corruption and I can't merge Support	13	1786	2023 年12 月 24 日

当标签同义词在主题上已存在时添加错误

相关话题