后端允许版主将帖子合并/移动到私信，但前端不允许

Hooksmith · 2020 年10 月 4 日 21:17

此功能仅在您是管理员时显示，对版主不显示：

前端代码位于此处：

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

52672b9ea


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

请注意其中的“this.currentUser.admin”。

然而，后端允许版主（所有工作人员）正常执行此操作：

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

我认为前端代码应进行调整：

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

如果您也认为这是一个缺陷，我很乐意提交 PR 进行修复。我们的版主团队需要此修复。

codinghorror · 2020 年10 月 4 日 23:02

我不太确定，但这似乎合理。你觉得呢，@techAPJ？

techAPJ · 2020 年10 月 5 日 07:16

决定不对版主显示“移至新私信”选项是明确在以下提交中做出的：

这是因为版主没有权限查看私信，相关代码将在以下位置失败：

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 · 2020 年10 月 5 日 12:32

如果通过代码修改，将代理版主/管理员添加为新创建的主题（私信）的收件人，以便他们能在主题创建后立即看到，这是否可行？（你们是否对接收此类修改感兴趣？）

techAPJ · 2020 年10 月 6 日 02:51

我会研究将版主作为私信接收人的可能性，并随后为版主启用“将帖子移至新私信”功能。

Hooksmith · 2020 年10 月 6 日 11:18

太好了！如果这件事最终被搁置或优先级降低，且欢迎提交拉取请求（PR），请告诉我，我可以协助实现。

techAPJ · 2020 年10 月 8 日 05:50

我可能需要一点时间才能处理这个。在此非常欢迎提交 PR！:+

话题		回复	浏览量
How can I move posts to a new message? Support	7	1141	2023 年10 月 9 日
Convert a private message to public topic or vice versa Feature feedback	15	7059	2016 年10 月 21 日
Moderators can see the move posts to PM button but can't use it Bug	4	643	2019 年2 月 18 日
Splitting topic into a new message for moderation Community Building	8	962	2022 年5 月 8 日
Move posts to private conversation Feature	21	3304	2022 年1 月 13 日

后端允许版主将帖子合并/移动到私信，但前端不允许

相关话题