Les mods de catégorie ne peuvent pas voir la conversation complète dans d'autres indicateurs

Je vais classer cela comme un bug, bien que je ne sois pas certain qu’il s’agisse du comportement prévu.

Actuellement, les modérateurs de catégories peuvent traiter les signalements dans leur propre catégorie. Cependant, il semble que lorsqu’une personne signale un message comme « autre chose », ils peuvent traiter le signalement mais ne peuvent pas voir la conversation complète. Bien que le bouton soit disponible, il indique simplement que cette page n’existe pas ou est privée.


Reproduction (vous avez besoin de deux comptes. L’utilisateur 2 est modérateur de catégorie)

  • Un utilisateur signale un message comme « autre chose »
  • L’utilisateur 2 se rend sur /review, traite le signalement mais ne peut pas voir la conversation complète
5 « J'aime »

Merci pour le signalement du bug. Je confirme avoir réussi à reproduire le problème.

Lorsqu’un message est signalé comme « Autre chose », un sujet de message privé est créé entre l’utilisateur ayant effectué le signalement et le groupe « modérateurs ». Le bouton « Voir toute la conversation » dans votre capture d’écran pointe vers ce sujet de message privé. Si un modérateur de groupe de catégorie clique dessus actuellement, il recevra une erreur.

La solution consiste à ajouter quelque chose comme le code suivant vers la ligne 309 de post_action_creator.rb, ce qui ajoutera le groupe concerné au message privé si l’option est activée, etc. :

if SiteSetting.enable_category_group_moderation? && @post.topic&.category&.reviewable_by_group_id?
  create_args[:target_group_names] << @post.topic.category.reviewable_by_group.name
end

Si ce groupe a l’option « Qui peut envoyer des messages à ce groupe ? » définie sur « Tout le monde », le modérateur de groupe de catégorie pourra alors lire ces rapports « Autre chose ».

Si ce n’est pas défini sur « Tout le monde », une erreur sera renvoyée indiquant que vous ne pouvez pas créer le message lors de la tentative de création du signalement.

Je vois deux solutions possibles :

  1. Exiger que les groupes de modérateurs de groupe de catégorie aient « Tout le monde » pour la permission d’envoi de messages.
  2. Ajouter un nouveau drapeau aux arguments dans le bloc de code ci-dessus, appelé par exemple skip_group_validation, qui sautera la vérification des permissions d’envoi de messages lors de la création des signalements « Autre chose ».

Laissez-moi réfléchir à cela et voir ce que je peux proposer…

5 « J'aime »

Une correction pour le problème ci-dessus a été fusionnée.

Merci @ondrej !

8 « J'aime »