الواجهة الخلفية تسمح للمشرفين بدمج/نقل المشاركات إلى رسالة خاصة، لكن الواجهة الأمامية لا تسمح بذلك

Hooksmith · 4 أكتوبر 2020، 9: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 · 4 أكتوبر 2020، 11:02م

لست متأكدًا، لكن يبدو ذلك معقولاً. ما رأيك @techAPJ؟

techAPJ · 5 أكتوبر 2020، 7: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 · 5 أكتوبر 2020، 12:32م

هل من الممكن إجراء تعديل في الكود لإضافة المشرف/المدير الحالي كمرسل إليه في الموضوع الجديد (الرسالة الخاصة) بحيث يتمكن من رؤية الموضوع الجديد فور إنشائه؟ (وهل هذا تعديل تودون تلقيه؟)

techAPJ · 6 أكتوبر 2020، 2:51ص

سأبحث في إمكانية إضافة المشرف كمتلقٍ للرسائل الخاصة، ومن ثم تمكين ميزة “نقل المنشورات إلى رسالة خاصة جديدة” للمشرفين.

Hooksmith · 6 أكتوبر 2020، 11:18ص

عظيم! في حال انتهى الأمر به إلى قائمة الانتظار أو تم تأجيله، ويُرحب بطلب السحب (PR)، يرجى إعلامي وسأتمكن من المساعدة في التنفيذ.

techAPJ · 8 أكتوبر 2020، 5:50ص

قد يستغرق الأمر مني بعض الوقت للوصول إلى هذا. طلبات السحب (PR) مرحب بها بنسبة 100% هنا!

الموضوع		الردود	مرات العرض
How can I move posts to a new message? Support	7	1140	9 أكتوبر 2023
Convert a private message to public topic or vice versa Feature feedback	15	7059	21 أكتوبر 2016
Moderators can see the move posts to PM button but can't use it Bug	4	643	18 فبراير 2019
Splitting topic into a new message for moderation Community Building	8	962	8 مايو 2022
Move posts to private conversation Feature	21	3302	13 يناير 2022

الواجهة الخلفية تسمح للمشرفين بدمج/نقل المشاركات إلى رسالة خاصة، لكن الواجهة الأمامية لا تسمح بذلك

الموضوعات ذات الصلة