Les notifications "doivent approuver les utilisateurs" sont déclenchées incorrectement par les anciens utilisateurs

J’ai remarqué le même comportement et je pense avoir trouvé pourquoi cela se produit (pas pourquoi ces utilisateurs n’apparaissent pas dans la file d’attente de révision, mais pourquoi ils n’ont pas été approuvés du tout) :

D’après ce que je comprends du code[1], lorsque vous activez ‘must_approve_users’, la plupart des utilisateurs créés avant que le paramètre ne soit activé devraient être approuvés :

Cela a fonctionné pour la plupart de mes utilisateurs, mais certains n’ont pas été marqués comme approuvés.

Requête Data Explorer
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

Lorsque vous regardez la table reviewables, vous pouvez remarquer que les utilisateurs dont l’ID correspond à l’ID cible d’un message de chat n’ont pas été approuvés.

Requête Data Explorer
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

Je pense que c’est un bug que les utilisateurs ne soient pas approuvés lorsqu’il existe un ID cible correspondant dont le type cible est différent de ‘user’.


  1. Je n’ai quasiment aucune compétence en programmation ↩︎

3 « J'aime »