Back-end permite que moderadores usem posts de mesclagem/movimentação para mensagem privada, mas front-end não

Hooksmith · Outubro 4, 2020, 9:17pm

Essa funcionalidade aparece quando você é administrador, mas não para moderadores:

O código do front-end para isso está aqui:

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

52672b9ea


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

Observe o “this.currentUser.admin”.

No entanto, o back-end permite que moderadores (toda a equipe) realizem essa ação sem problemas:

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

Acho que o código do front-end deveria ser ajustado:

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

Se você concordar que isso é um bug, ficarei feliz em abrir um PR para corrigi-lo. Nossa equipe de moderação precisaria dessa correção.

codinghorror · Outubro 4, 2020, 11:02pm

Não tenho certeza, mas parece plausível. O que você acha, @techAPJ?

techAPJ · Outubro 5, 2020, 7:16am

A decisão de não exibir a opção ‘mover para nova MP’ para moderadores foi tomada explicitamente em:

Isso ocorre porque os moderadores não têm permissão para ver MPs, e o código relevante falhará em:

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 · Outubro 5, 2020, 12:32pm

Seria viável que, com uma alteração no código, o moderador/administrador atuante fosse adicionado como destinatário do tópico recém-criado (mensagem privada), para que ele pudesse visualizar o novo tópico assim que fosse criado? (E essa seria uma alteração que vocês estariam interessados em receber?)

techAPJ · Outubro 6, 2020, 2:51am

Vou verificar a possibilidade de adicionar o moderador atuante como destinatário de MP e, em seguida, habilitar o recurso “mover posts para uma nova MP” para moderadores.

Hooksmith · Outubro 6, 2020, 11:18am

Ótimo! Caso isso acabe indo para o backlog ou seja despriorizado e uma PR seja bem-vinda, por favor, me avise e eu posso ajudar com a implementação.

techAPJ · Outubro 8, 2020, 5:50am

Pode demorar um pouco para eu chegar nisso. PR é 100% bem-vindo aqui!

Tópico		Respostas	Visualizações
How can I move posts to a new message? Support	7	1141	9 de Outubro de 2023
Convert a private message to public topic or vice versa Feature feedback	15	7059	21 de Outubro de 2016
Moderators can see the move posts to PM button but can't use it Bug	4	643	18 de Fevereiro de 2019
Splitting topic into a new message for moderation Community Building	8	962	8 de Maio de 2022
Move posts to private conversation Feature	21	3304	13 de Janeiro de 2022

Back-end permite que moderadores usem posts de mesclagem/movimentação para mensagem privada, mas front-end não

Tópicos relacionados