Motivo errato mostrato per cui l'utente moderatore non può essere eliminato

In FIX: Correct user deletion wording by hlashbrooke · Pull Request #34680 · discourse/discourse · GitHub “e moderatori” è stato rimosso dalla nota che spiega perché un utente non può essere eliminato:

delete_forbidden_because_staff: “Gli amministratori e i moderatori non possono essere eliminati.”

Capisco che questo sia utile perché i moderatori possono essere eliminati. Ma ho l’impressione che qualcosa sia stato trascurato in questo aggiustamento, il che mi ha confuso oggi.
Il mio utente è un moderatore, ma non un amministratore, e la nota diceva ancora “Gli amministratori non possono essere eliminati”.

Dopo aver controllato il codice, penso di aver trovato perché succede questo: Se un utente non può essere eliminato, ci sono due ragioni possibili:

  1. L’utente è un amministratore.
  2. L’utente ha post più vecchi di quelli configurati.

La vera ragione per cui non ho potuto eliminare il mio utente moderatore è stata [2], ma la ragione mostrata è una, perché mentre i moderatori sono stati rimossi dal testo dell’avviso, il codice controllava ancora staff invece di admin.

3 Mi Piace

Sì, posso riprodurlo anche sul mio sito di test ospitato. Grazie per la segnalazione Moin :slight_smile:

1 Mi Piace

Per chiarire ulteriormente… un utente che non è un moderatore non può essere eliminato e il motivo mostrato è il seguente. Il bug è che viene mostrato il motivo errato quando l’utente è un moderatore.

Gli utenti non possono essere eliminati se hanno post. Elimina tutti i post prima di tentare di eliminare un utente. (I post più vecchi di 365 giorni non possono essere eliminati.)

Anche gli amministratori vedono il motivo sbagliato. Non si tratta di chi guarda la pagina di amministrazione dell’utente, ma di quale stato ha l’utente che stai guardando.

Sì, era quello che intendevo, ma ho sbagliato la struttura della frase! :rofl: Penso di averla sistemata ora.

Grazie. Ero preoccupato che la tua precisazione potesse portare a malintesi, anche se pensavo che il titolo descrivesse abbastanza bene il problema.

1 Mi Piace

Si spera che questo renda l’interfaccia utente coerente con il back-end

3 Mi Piace