Motivo incorreto mostrado para exclusão de usuário moderador não poder ser excluído

Em FIX: Correct user deletion wording by hlashbrooke · Pull Request #34680 · discourse/discourse · GitHub , “e moderadores” foi removido da nota que explica por que um usuário não pode ser excluído:

delete_forbidden_because_staff: “Administradores e moderadores não podem ser excluídos.”

Entendo que isso é útil porque moderadores podem ser excluídos. Mas tenho a impressão de que algo foi esquecido neste ajuste, o que me confundiu hoje.
Meu usuário é um moderador, mas não um administrador, e ainda assim a nota dizia “Administradores não podem ser excluídos”.

Após verificar o código, acho que encontrei o motivo disso acontecer: se um usuário não pode ser excluído, existem duas razões possíveis:

  1. O usuário é um administrador.
  2. O usuário tem posts mais antigos do que o configurado.

O motivo real pelo qual não pude excluir meu usuário moderador foi [2], mas o motivo mostrado é um, porque embora os moderadores tenham sido removidos do texto do aviso, o código ainda verificava staff em vez de admin.

3 curtidas

Sim, eu consigo reproduzir isso no meu site de teste hospedado também. Obrigado por relatar, Moin :slight_smile:

1 curtida

Para esclarecer ainda mais… um usuário que não é moderador não pode ser excluído e o motivo mostrado é o abaixo. O bug é que o motivo errado é mostrado quando o usuário é um moderador.

Usuários não podem ser excluídos se tiverem posts. Exclua todos os posts antes de tentar excluir um usuário. (Posts com mais de 365 dias não podem ser excluídos.)

Os administradores também veem o motivo errado. Não se trata de quem olha a página de administração do usuário, mas de qual status o usuário tem que você está olhando.

Sim, era isso que eu queria dizer, mas errei a estrutura da frase! :rofl: Acho que corrigi agora.

Obrigado. Eu estava preocupado que seu esclarecimento pudesse levar a mal-entendidos, embora eu achasse que o título descrevia bem o problema.

1 curtida

Isso, esperançosamente, tornará a interface do usuário consistente com o back-end

3 curtidas