Back-End erlaubt Moderatoren, Beiträge in private Nachrichten zusammenzuführen/zu verschieben, aber Front-End nicht

Hooksmith · 4. Oktober 2020 um 21:17

Diese Funktion wird nur angezeigt, wenn Sie Administrator sind, nicht jedoch für Moderatoren:

Der Frontend-Code dafür befindet sich hier:

app/assets/javascripts/discourse/app/controllers/move-to-topic.js

52672b9ea


      
          @discourseComputed("canSplitTopic")
          canSplitToPM(canSplitTopic) {
            return canSplitTopic && this.currentUser && this.currentUser.admin;
          },

Beachten Sie das „this.currentUser.admin“.

Der Backend-Code erlaubt es Moderatoren (allen Mitarbeitern) jedoch problemlos, diese Aktion auszuführen:

github.com/discourse/discourse

app/controllers/topics_controller.rb

5cf411c3a


      
          raise Discourse::InvalidAccess if params[:archetype] == "private_message" && !guardian.is_staff?
          
          topic = Topic.find_by(id: topic_id)
          guardian.ensure_can_move_posts!(topic)
          
          destination_topic = Topic.find_by(id: destination_topic_id)
          guardian.ensure_can_create_post_on_topic!(destination_topic)

github.com/discourse/discourse

app/controllers/topics_controller.rb

5cf411c3a


      
          raise Discourse::InvalidAccess if params[:archetype] == "private_message" && !guardian.is_staff?
          
          topic = Topic.with_deleted.find_by(id: topic_id)
          guardian.ensure_can_move_posts!(topic)

github.com/discourse/discourse

lib/guardian/topic_guardian.rb

9f73e8779


      
          def can_perform_action_available_to_group_moderators?(topic)
            return false if anonymous? || topic.nil?
            return true if is_staff?
            return true if @user.has_trust_level?(TrustLevel[4])
          
            is_category_group_moderator?(topic.category)
          end
          alias :can_archive_topic? :can_perform_action_available_to_group_moderators?
          alias :can_close_topic? :can_perform_action_available_to_group_moderators?
          alias :can_split_merge_topic? :can_perform_action_available_to_group_moderators?
          alias :can_edit_staff_notes? :can_perform_action_available_to_group_moderators?
          
          def can_move_posts?(topic)
            return false if is_silenced?
            can_perform_action_available_to_group_moderators?(topic)
          end

Ich denke, der Frontend-Code sollte angepasst werden:

@discourseComputed("canSplitTopic")
canSplitToPM(canSplitTopic) {
-  return canSplitTopic && this.currentUser && this.currentUser.admin;
+  return canSplitTopic && this.currentUser && this.currentUser.staff;
},

Wenn Sie zustimmen, dass dies ein Fehler ist, würde ich mich freuen, einen PR zur Behebung einzureichen. Unser Moderationsteam wäre auf diese Korrektur angewiesen.

codinghorror · 4. Oktober 2020 um 23:02

Ich bin mir nicht sicher, aber es scheint plausibel. Was denkst du, @techAPJ?

techAPJ · 5. Oktober 2020 um 07:16

Die Entscheidung, die Option „In neue PN verschieben

Hooksmith · 5. Oktober 2020 um 12:32

Wäre es machbar, wenn durch eine Code-Änderung der amtierende Moderator/Administrator als Empfänger auf das neu erstellte Thema (Private Nachricht) gesetzt wird, damit er das neue Thema sofort nach der Erstellung sehen kann? (Und wäre dies eine Änderung, die Sie gerne erhalten würden?)

techAPJ · 6. Oktober 2020 um 02:51

Ich werde prüfen, ob es möglich ist, den Moderator als PM-Empfänger hinzuzufügen und anschließend die Funktion „Beiträge in eine neue PM verschieben

Hooksmith · 6. Oktober 2020 um 11:18

Toll! Falls dies am Ende in den Backlog rutscht oder zurückgestellt wird und ein PR willkommen ist, lass es mich bitte wissen, dann kann ich bei der Umsetzung helfen.

techAPJ · 8. Oktober 2020 um 05:50

Es kann ein bisschen dauern, bis ich dazu komme. PRs sind hier zu 100 % willkommen!

Thema		Antworten	Aufrufe
How can I move posts to a new message? Support	7	1140	9. Oktober 2023
Convert a private message to public topic or vice versa Feature feedback	15	7059	21. Oktober 2016
Moderators can see the move posts to PM button but can't use it Bug	4	643	18. Februar 2019
Splitting topic into a new message for moderation Community Building	8	962	8. Mai 2022
Move posts to private conversation Feature	21	3301	13. Januar 2022

Back-End erlaubt Moderatoren, Beiträge in private Nachrichten zusammenzuführen/zu verschieben, aber Front-End nicht

Verwandte Themen