# Notificações de "deve aprovar usuários" são acionadas incorretamente por usuários antigos

**URL:** https://meta.discourse.org/t/must-approve-users-notifications-are-incorrectly-triggered-by-old-users/240731
**Category:** Bug
**Created:** [Outubro 3, 2022, 12:35pm UTC](https://meta.discourse.org/t/must-approve-users-notifications-are-incorrectly-triggered-by-old-users/240731 "2022-10-03T12:35:44Z")
**Posts on this page:** 1
**Showing post:** 4

<div class="post-metadata">

### Author: ![Moin](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/moin/32/554653_2.png) [@Moin](https://meta.discourse.org/u/Moin)
#### Post date: [Abril 20, 2024, 2:08pm UTC](https://meta.discourse.org/t/must-approve-users-notifications-are-incorrectly-triggered-by-old-users/240731/4 "2024-04-20T14:08:37Z")

</div>

Notei o mesmo comportamento e acho que descobri por que isso acontece (não por que esses usuários não aparecem na fila de revisão, mas por que eles não foram aprovados):

Pelo que entendi do código\[1\], quando você habilita ‘must\_approve\_users’, a maioria dos usuários criados antes da configuração ser habilitada deve ser aprovada:

> <https://github.com/discourse/discourse/blob/a564274ba277a834b8a3ab4c28b2c1022fd9521e/config/initializers/014-track-setting-changes.rb#L8-L16>

Isso funcionou para a maioria dos meus usuários, mas alguns não foram marcados como aprovados.

> **Consulta do Explorador de Dados**
>
> ```plaintext
> SELECT
> id as UserID, approved
> FROM
> users
> ORDER BY id
> 
> ```

| UserID | Approved |
| --- | --- |
| 1 | true |
| 3 | true |
| … | … |
| 8 | true |
| 10 | false |
| 11 | false |
| 12 | false |
| 13 | true |

Quando você olha para a tabela `reviewables`, pode notar que os usuários com um ID de usuário correspondente ao `target_id` de uma mensagem de chat não foram aprovados.

> **Consulta do Explorador de Dados**
>
> ```plaintext
> SELECT id, target_id, target_type
> FROM reviewables
> ORDER BY target_id
> 
> ```

| id | target\_id | target\_type |
| --- | --- | --- |
| 6 | 9 | ChatMessage |
| 7 | 10 | ChatMessage |
| 8 | 11 | ChatMessage |
| 9 | 12 | ChatMessage |
| 1 | 2901 | Post |
| 2 | 2909 | Post |
| 5 | 2991 | Post |

Acho que é um bug que os usuários não sejam aprovados quando há um `target_id` correspondente que tem um `target_type` diferente de ‘user’.

* * *

1. Tenho pouquíssimas habilidades de programação

---

_[View the full topic](https://meta.discourse.org/t/must-approve-users-notifications-are-incorrectly-triggered-by-old-users/240731)._