Le back-end permet aux modérateurs d'utiliser la fusion/déplacement de messages vers un message privé, mais le front-end ne le fait pas

Hooksmith · Octobre 4, 2020, 9:17

Cette fonctionnalité s’affiche lorsque vous êtes administrateur, mais pas pour les modérateurs :

Le code front-end pour cela se trouve ici :

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

52672b9ea


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

Notez le « this.currentUser.admin ».

Cependant, le back-end permet aux modérateurs (tout le personnel) d’effectuer cette action sans problème :

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

Je pense que le code front-end devrait être ajusté :

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

Si vous êtes d’accord pour considérer cela comme un bug, je serais ravi de créer une PR pour le corriger. Notre équipe de modération aurait besoin de cette correction.

codinghorror · Octobre 4, 2020, 11:02

Je ne suis pas sûr, mais cela semble plausible. Qu’en penses-tu @techAPJ ?

techAPJ · Octobre 5, 2020, 7:16

La décision de ne pas afficher l’option « déplacer vers un nouveau MP » pour les modérateurs a été prise explicitement dans :

https://github.com/discourse/discourse/commit/dc2776bcddc07e69e5a4ab92526f6238f3a5ccd7

Cela s’explique par le fait que les modérateurs n’ont pas la permission de consulter les MP, et le code pertinent échouera à l’adresse suivante :

github.com/discourse/discourse

app/models/post_mover.rb

310952fd6


      
            topic
          end
          
          private
          
          def update_topic_excerpt(topic)
            topic.update_excerpt(topic.first_post.excerpt_for_topic)
          end
          
          def move_posts_to(topic)
            Guardian.new(user).ensure_can_see! topic
            @destination_topic = topic
          
            moving_all_posts = (@original_topic.posts.pluck(:id).sort == @post_ids.sort)
          
            create_temp_table
            delete_invalid_post_timings
            move_each_post
            notify_users_that_posts_have_moved
            update_statistics
            update_user_actions

Hooksmith · Octobre 5, 2020, 12:32

Serait-il envisageable de modifier le code pour que le modérateur ou administrateur en exercice soit ajouté en tant que destinataire du nouveau sujet (message privé), afin qu’il puisse voir le nouveau sujet dès sa création ? (Et cette modification serait-elle quelque chose qui vous intéresserait ?)

techAPJ · Octobre 6, 2020, 2:51

Je vais examiner la possibilité d’ajouter le rôle de modérateur en tant que destinataire de MP et, par la suite, activer la fonctionnalité « déplacer les messages vers un nouveau MP » pour les modérateurs.

Hooksmith · Octobre 6, 2020, 11:18

Super ! Si cela se retrouve en fin de liste d’attente ou est dépriorisé et qu’une PR est la bienvenue, faites-le-moi savoir et je pourrais aider à la mise en œuvre.

techAPJ · Octobre 8, 2020, 5:50

Cela peut me prendre un peu de temps avant d’y arriver. Les PR sont les bienvenues ici !

Sujet		Réponses	Vues
How can I move posts to a new message? Support	7	1141	Octobre 9, 2023
Convert a private message to public topic or vice versa Feature feedback	15	7059	Octobre 21, 2016
Moderators can see the move posts to PM button but can't use it Bug	4	643	Février 18, 2019
Splitting topic into a new message for moderation Community Building	8	962	Mai 8, 2022
Move posts to private conversation Feature	21	3302	Janvier 13, 2022

Le back-end permet aux modérateurs d'utiliser la fusion/déplacement de messages vers un message privé, mais le front-end ne le fait pas

Sujets connexes