Falscher Grund angezeigt, warum Moderator-Benutzer nicht gelöscht werden kann

In FIX: Correct user deletion wording by hlashbrooke · Pull Request #34680 · discourse/discourse · GitHub wurde „und Moderatoren“ aus dem Hinweis entfernt, der erklärt, warum ein Benutzer nicht gelöscht werden kann:

delete_forbidden_because_staff: „Admins und Moderatoren können nicht gelöscht werden.“

Ich verstehe, dass dies hilfreich ist, da Moderatoren gelöscht werden können. Aber ich habe den Eindruck, dass bei dieser Anpassung etwas übersehen wurde, was mich heute verwirrt hat.
Mein Benutzer ist ein Moderator, aber kein Administrator, und trotzdem stand in der Notiz: „Admins können nicht gelöscht werden“.

Nachdem ich den Code überprüft hatte, glaube ich, den Grund dafür gefunden zu haben: Wenn ein Benutzer nicht gelöscht werden kann, gibt es zwei mögliche Gründe:

  1. Der Benutzer ist ein Administrator.
  2. Der Benutzer hat Beiträge, die älter als konfiguriert sind.

Der eigentliche Grund, warum ich meinen Moderator-Benutzer nicht löschen konnte, war [2], aber der angezeigte Grund ist eins, denn obwohl Moderatoren aus dem Text der Warnung entfernt wurden, prüfte der Code immer noch auf staff statt auf admin.

3 „Gefällt mir“

Ja, ich kann das auch auf meiner gehosteten Testseite reproduzieren. Danke für die Meldung, Moin :slight_smile:

1 „Gefällt mir“

Um dies weiter zu verdeutlichen: Ein Benutzer, der kein Moderator ist, kann nicht gelöscht werden, und der Grund dafür wird unten angezeigt. Der Fehler besteht darin, dass der falsche Grund angezeigt wird, wenn der Benutzer ein Moderator ist.

Benutzer können nicht gelöscht werden, wenn sie Beiträge haben. Löschen Sie alle Beiträge, bevor Sie versuchen, einen Benutzer zu löschen. (Beiträge, die älter als 365 Tage sind, können nicht gelöscht werden.)

Administratoren sehen ebenfalls den falschen Grund. Es geht nicht darum, wer die Admin-Seite des Benutzers betrachtet, sondern welchen Status der Benutzer hat, den Sie betrachten.

Ja, das meinte ich, aber ich hatte die Satzstruktur falsch! :rofl: Ich glaube, ich habe es jetzt behoben.

Danke. Ich war besorgt, dass Ihre Klarstellung zu Missverständnissen führen würde, obwohl ich dachte, der Titel beschreibe das Problem ziemlich gut.

1 „Gefällt mir“

Dies sollte hoffentlich die Benutzeroberfläche mit dem Backend konsistent machen

3 „Gefällt mir“