Erreur lors de l'ajout d'un synonyme de balise lorsque les deux existent déjà sur un sujet

Overgrow · Février 29, 2020, 1:14

Il semble que le code actuel ne puisse pas gérer la création de synonymes si le sujet contenait déjà les deux balises avant la création du synonyme.

Démarrage de POST "/tag/solstrip/synonyms" pour 3.14.14.5 à 2020-02-29 12:07:28 +0000
Traitement par TagsController#create_synonyms en tant que */*
  Paramètres : {"synonymes"=>["solstrips"], "tag_id"=>"solstrip"}
Terminé avec erreur 500 Internal Server Error en 46 ms (ActiveRecord : 0,0 ms | Allocations : 6385)
ActiveRecord::RecordNotUnique (PG::UniqueViolation: ERREUR : la valeur de clé en double viole la contrainte d'unicité "index_topic_tags_on_topic_id_and_tag_id"
DÉTAIL : La clé (topic_id, tag_id)=(14651, 310) existe déjà.
)
/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'
Échec de la gestion de l'exception dans le middleware d'application d'exception : PG::UniqueViolation: ERREUR : la valeur de clé en double viole la contrainte d'unicité "index_topic_tags_on_topic_id_and_tag_id"
DÉTAIL : La clé (topic_id, tag_id)=(14651, 310) existe déjà.

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

Quelqu’un peut-il s’il vous plaît confirmer cela et probablement déplacer vers #bugs ? Merci !

justani · Avril 28, 2020, 3:22

Oui, ce problème se produit également sur notre installation Discourse.

supermathie · Juillet 17, 2020, 9:17

Confirmé : cela se produit toujours sur le code le plus récent.

sam · Juillet 21, 2020, 12:40

@vinothkannans peux-tu régler ça ?

vinothkannans · Juillet 21, 2020, 3:33

Ceci est maintenant corrigé dans le commit ci-dessous

Sujet		Réponses	Vues
Internal Server Error on using some tags (ERROR: duplicate key value violates unique constraint "index_tags_on_name") Bug	20	3311	Novembre 12, 2019
Adding synonym to the same tag gets stuck Bug	11	1339	Novembre 23, 2020
Main tag sharing same name and url with synonym Support	0	600	Juin 28, 2020
New Tags Creation Feature tags , completed	20	1205	Avril 16, 2024
There are some duplicated tags due to database corruption and I can't merge Support	13	1786	Décembre 24, 2023

Erreur lors de l'ajout d'un synonyme de balise lorsque les deux existent déjà sur un sujet

Sujets connexes