J’ai trouvé ce sujet en cherchant avant de signaler un bug, donc je l’ajoute.
TL;DR la requête pour la notification est incorrecte car elle compte les utilisateurs rejetés.
Notification : 16 utilisateurs en attente d’approbation, cliquez sur le lien, ne voyez que 2.
La requête utilisée pour la notification est la suivante :
puts AdminUserIndexQuery.new(query: "pending", stats: false).find_users_query.to_sql
SELECT "users".* FROM "users"
WHERE (suspended_till IS NULL OR suspended_till <= '2023-11-13 11:05:23.225614')
AND "users"."approved" = FALSE
AND "users"."active" = TRUE
ORDER BY users.created_at DESC,users.username
ce qui, dans mon cas, me donne 16 utilisateurs :
[4479, 4472, 4456, 4446, 4443, 4430, 4302, 4291, 4206, 4199, 4178, 4168, 4131, 4061, 3677, 3642]
En lançant ces identifiants dans la file d’attente des éléments révisables, je trouve 2 utilisateurs qui doivent réellement être approuvés (statut 0) et 14 utilisateurs qui ont déjà été rejetés (statut 2) :
ReviewableUser.where(type: 'ReviewableUser')
.where(target_id: ids)
.pluck(:target_id, :status)
[[3642, 2], [3677, 2], [4061, 2], [4131, 2], [4168, 2],
[4178, 2], [4199, 2], [4206, 2], [4291, 2], [4302, 2],
[4430, 2], [4443, 2], [4446, 2], [4456, 2], [4472, 0], [4479, 0]]