תחביר Ruby hash מוצג במיילים שנשלחים למשתמשים שנמחקו

סיכום מבוסס בינה מלאכותית

A bug in Discourse causes Ruby hash syntax to appear in the email sent to users when their accounts are deleted after rejecting their first post. The issue stems from a missing translation key for the :needs_approval flag in the flag_reasons hash, leading to improper rendering of the email content. Richie reported the issue with screenshots, and selase identified the root cause in the code. The bug also affects custom flags, as noted by Moin. A fix has been merged, and a related issue for staged users has been moved to a separate topic. selase confirmed the resolution.

כאשר משתמש נמחק על ידי דחיית הפוסט הראשון שלו, נשלחת אליו אימייל עם שורת הנושא חשבונך נמחק.

קיימת שגיאה בגוף / בתוכן האימייל, נראה שהוא מציג סוג של תחביר Ruby hash בחלק האמצעי של האימייל:


שלבים לשחזור:

  1. ודא ש-Discourse דורש אישור של הפוסט הראשון של כל משתמש
  2. צור משתמש חדש
  3. צור פוסט חדש עם המשתמש החדש
  4. באמצעות חשבון מנהל, דחה את הפוסט עם האפשרות “מחק משתמש”
  5. המשתמש החדש מקבל אימייל עם התוכן הנ"ל
2 לייקים

If it helps, a couple of screen shots from the admin view of the rejected post:

Before deleting the user:

After rejection and deletion:

לייק 1

A post was split to a new topic: Fix deleted user email handling for staged users

This looks like a bug in how flag_reason is generated here:

PostActionTypeView.new.types doesn’t include :needs_approval flag (which gets created in the flow described). Because of this, the translation key resolves to flag_reasons. (with no suffix), which returns the entire YAML section instead of a single entry. That’s why there is a Ruby hash in place of the flag reason.

Relatedly, this likely impacts custom flags as well, since their translations would also be missing.

7 לייקים

I am sure it does. I mentioned that in May

3 לייקים

We’ve merged a fix for the issue in the OP. I’ve moved the report related to staged users into its own topic to make it easier to track pending a fix.

https://github.com/discourse/discourse/pull/34955

לייק 1