Le plugin Category experts plante lors de la fusion des utilisateurs

zogstrip · Avril 2, 2026, 3:41

Merci pour le rapport @RGJ

Cela sera corrigé par

github.com/discourse/discourse-category-experts

FIX: Use `save_custom_fields` instead of `save!` when only custom fie… (#230)

main ← fix/save-custom-fields-during-owner-change

opened 03:41PM - 02 Apr 26 UTC

ZogStriP

+23 -8

…lds changed During user merges, `PostOwnerChanger` reassigns all post types …including small action posts which have `raw: nil` by design. The core `set_owner` method handles this correctly via `skip_validations: true`, but the plugin's `post_owner_changed` handler and `PostHandler` methods were calling `save!` after only modifying custom fields. This triggered full AR validations on the post body, raising `ActiveRecord::RecordInvalid` for posts with content that doesn't pass current validation rules. Replace all `save!` calls with `save_custom_fields` in places where only custom fields are being modified — 8 occurrences across `plugin.rb` and `PostHandler` (`mark_post_for_approval`, `mark_post_as_approved`, `mark_topic_as_question`, `correct_topic_custom_fields_after_removal`, `correct_topic_custom_fields_after_addition`, `handle_topic_category_change`). https://meta.discourse.org/t/399785

Sujet		Réponses	Vues
Merge users not possible from UI or CLI Support merge-user	2	462	Mai 14, 2025
User merge failed, corrupted database entries due to third party plugin Support merge-user	32	2556	Novembre 5, 2020
Discourse Category Experts Plugin official , category-experts	3	9485	Août 17, 2025
Issue merging user accounts, web interace and rake console Support merge-user	10	379	Mai 14, 2025
Merge script error when copying tags Bug	7	784	Mars 29, 2019

Le plugin Category experts plante lors de la fusion des utilisateurs

Sujets connexes