Konsistenz bei Erstellen und Aktualisieren von Beiträgen

angus · 21. April 2025 um 09:07

Das WP Discourse-Plugin verwendet derzeit eine einzelne Anfrage zum Erstellen von Beiträgen, wenn Beiträge in Discourse veröffentlicht werden, und (potenziell) zwei Anfragen beim Aktualisieren von Beiträgen.

Die erste Anfrage bei der Aktualisierung ist an PostController#update für die Hauptbeitragsattribute und die zweite an TopicController#update, um den hervorgehobenen Link des Themas zu aktualisieren, falls diese Funktion verwendet wird.

Es gibt einige Versuche in PostController#update, die Konsistenz mit PostController#create aufrechtzuerhalten, z. B.

github.com/discourse/discourse

app/controllers/posts_controller.rb

main


      
          changes = {
            raw: params[:post][:raw],
            edit_reason: params[:post][:edit_reason],
            locale: params[:post][:locale],
          }
          
          Post.plugin_permitted_update_params.keys.each { |param| changes[param] = params[:post][param] }
          
          # keep `raw_old` for backwards compatibility
          original_text = params[:post][:original_text] || params[:post][:raw_old]
          if original_text.present? && original_text != post.raw
            return render_json_error(I18n.t("edit_conflict"), status: 409)
          end
          
          # to stay consistent with the create api, we allow for title & category changes here
          if post.is_first_post?
            changes[:title] = params[:title] if params[:title]
            changes[:category_id] = params[:post][:category_id] if params[:post][:category_id]
          
            if changes[:category_id] && changes[:category_id].to_i != post.topic.category_id.to_i
              category = Category.find_by(id: changes[:category_id])

Ein hervorgehobener Link eines Themas ist jedoch derzeit nicht Teil dieser Konsistenz.

Ich möchte einen PR an discourse/discourse machen, um Aktualisierungen des hervorgehobenen Links von Themen über PostController#update zu ermöglichen. Dies würde die Aktualisierungslogik des WP Discourse-Plugins vereinfachen. Ich wollte die Änderung hier zuerst abfragen, falls ich etwas übersehe.

Thema		Antworten	Aufrufe
WP Discourse "Update Discourse topic" error WordPress	25	3410	7. Juli 2019
Editing linked Discourse posts/topics without changes being overridden by Wordpress? WordPress	14	1836	16. Oktober 2017
Wrong path after update of permalinks - bug? WordPress	8	1202	4. Juli 2020
A specific Discourse topic isn't updated when the WP article is updated WordPress	3	603	12. Juli 2021
Custom post types failing to update associated Discourse posts WordPress	5	692	2. März 2022

Konsistenz bei Erstellen und Aktualisieren von Beiträgen

Verwandte Themen