Erro ao adicionar sinônimo de tag quando ambas já existem em um tópico

Overgrow · Fevereiro 29, 2020, 1:14pm

Parece que o código atual não consegue criar sinônimos caso o tópico já contenha ambas as tags antes de criar o sinônimo.

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

Alguém poderia confirmar isso e, provavelmente, mover para #bugs? Obrigado!

justani · Abril 28, 2020, 3:22pm

Sim, esse problema também está ocorrendo na nossa instalação do Discourse.

supermathie · Julho 17, 2020, 9:17pm

Confirmado que isso ainda está acontecendo no código mais recente.

sam · Julho 21, 2020, 12:40am

@vinothkannans você consegue resolver isso?

vinothkannans · Julho 21, 2020, 3:33pm

Isso já foi corrigido no commit abaixo

Tópico		Respostas	Visualizações
Internal Server Error on using some tags (ERROR: duplicate key value violates unique constraint "index_tags_on_name") Bug	20	3311	12 de Novembro de 2019
Adding synonym to the same tag gets stuck Bug	11	1339	23 de Novembro de 2020
Main tag sharing same name and url with synonym Support	0	600	28 de Junho de 2020
New Tags Creation Feature tags , completed	20	1205	16 de Abril de 2024
There are some duplicated tags due to database corruption and I can't merge Support	13	1786	24 de Dezembro de 2023

Erro ao adicionar sinônimo de tag quando ambas já existem em um tópico

Tópicos relacionados