Moin
17 أكتوبر 2025، 2:50م
1
في FIX: Correct user deletion wording by hlashbrooke · Pull Request #34680 · discourse/discourse · GitHub ، تمت إزالة “والمشرفين” من الملاحظة التي تشرح سبب عدم إمكانية حذف المستخدم:
delete_forbidden_because_staff: “المسؤولون والمشرفون لا يمكن حذفهم.”
أتفهم أن هذا مفيد لأن المشرفين يمكن حذفهم. لكن لدي انطباع بأن شيئًا ما تم التغاضي عنه في هذا التعديل، مما أربكني اليوم.
كان المستخدم الخاص بي مشرفًا، ولكنه ليس مسؤولًا، ومع ذلك قالت الملاحظة “المسؤولون لا يمكن حذفهم”.
بعد التحقق من الكود، أعتقد أنني وجدت سبب حدوث ذلك: إذا لم يكن من الممكن حذف مستخدم، فهناك سببان محتملان:
المستخدم مسؤول.
المستخدم لديه منشورات أقدم من الإعدادات.
السبب الحقيقي لعدم تمكني من حذف مستخدمي المشرف كان [2]، ولكن السبب المعروض هو واحد، لأنه بينما تمت إزالة المشرفين من نص التحذير، لا يزال الكود يتحقق من staff بدلاً من admin.
});
}
}
@discourseComputed("model.canBeDeleted", "model.staff")
deleteExplanation(canBeDeleted, staff) {
if (canBeDeleted) {
return null;
}
if (staff) {
return i18n("admin.user.delete_forbidden_because_staff");
} else {
return i18n("admin.user.delete_forbidden", {
count: this.siteSettings.delete_user_max_post_age,
});
}
}
@discourseComputed("model.username")
postEditsByEditorFilter(username) {
3 إعجابات
Lilly
(Lillian Louis)
17 أكتوبر 2025، 3:00م
2
نعم، يمكنني إعادة إنتاج هذا على موقع الاختبار المستضاف الخاص بي أيضًا. شكراً لك على الإبلاغ يا موين
إعجاب واحد (1)
لتوضيح الأمر بشكل أكبر.. لا يمكن حذف مستخدم ليس مشرفًا، والسبب المعروض أدناه. الخطأ هو أنه يتم عرض السبب الخاطئ عندما يكون المستخدم مشرفًا.
لا يمكن حذف المستخدمين إذا كان لديهم منشورات. احذف جميع المنشورات قبل محاولة حذف مستخدم. (لا يمكن حذف المنشورات التي يزيد عمرها عن 365 يومًا.)
Moin
17 أكتوبر 2025، 6:32م
5
يرى المسؤولون السبب الخطأ أيضًا. لا يتعلق الأمر بمن ينظر إلى صفحة المسؤول الخاصة بالمستخدم، ولكن ما هي الحالة التي ينظر إليها المستخدم.
نعم، هذا ما قصدته، لكنني أخطأت في بنية الجملة! أعتقد أنني قمت بإصلاحها الآن.
Moin
17 أكتوبر 2025، 6:36م
7
شكراً لك. كنت قلقاً من أن يؤدي توضيحك إلى سوء فهم، على الرغم من أنني كنت أعتقد أن العنوان يصف المشكلة بشكل جيد للغاية.
إعجاب واحد (1)
نأمل أن يؤدي هذا إلى جعل واجهة المستخدم متسقة مع الواجهة الخلفية
main ← fix-ui-for-deleting-moderators-account
opened 10:10AM - 20 Oct 25 UTC
The UI wasn't "synced" with the backend and showed the wrong error message when … trying to delete a moderator.
In the back-end, only admins can't be deleted.
In the front-end, it was tied to a "staff" check.
Ref - https://meta.discourse.org/t/-/386013
3 إعجابات