refactor wo pluck

config/locales/server.en.yml

···
 en:
   site_settings:
     shadowban_enabled: "Enable shadowban"
-    shadowban_approved_group_name: "Group name for approved users (users NOT in this group will be shadowbanned)"
+    shadowban_allowed_group: "Group name for approved users (users NOT in this group will be shadowbanned)"
     shadowban_show_to_staff: "Show shadowbanned posts to staff"

config/locales/server.fr.yml

···
 fr:
   site_settings:
     shadowban_enabled: "Activer le shadowban"
-    shadowban_approved_group_name: "Nom du groupe d'utilisateurs approuvés (les utilisateurs PAS dans ce groupe seront en shadowban)"
+    shadowban_allowed_group: "Nom du groupe d'utilisateurs approuvés (les utilisateurs PAS dans ce groupe seront en shadowban)"
     shadowban_show_to_staff: "Afficher les publications shadowban au staff"

config/settings.yml

···
 # config/settings.yml
-shadowban:
+shadowban-all-plugin:
   shadowban_enabled:
     default: false
-    client: true
-  shadowban_approved_group_name:
-    default: "approved_users"
-    client: false
+    type: bool
+  shadowban_allowed_group:
+    default: ""
     type: string
   shadowban_show_to_staff:
     default: true
-    client: true
+    type: bool

plugin.rb

···
-# name: shadowban
-# about: Hide posts from users NOT in a specific group (whitelist approach)
-# version: 0.2.0
-# authors: Michael Howell (notriddle), Modified for whitelist groups
+# name: shadowban-group
+# about: Hide posts from all users except those in a specific group
+# version: 0.0.2
+# authors: Modified from Michael Howell (notriddle)
 # url: https://github.com/notriddle/discourse-shadowban
 enabled_site_setting :shadowban_enabled
 
 after_initialize do
 
-  # Méthode helper pour récupérer les IDs des utilisateurs autorisés (non-shadowban)
-  def approved_user_ids
-    return [] unless SiteSetting.shadowban_approved_group_name.present?
+  # Méthode helper pour vérifier si un utilisateur est dans le groupe autorisé
+  def self.user_in_allowed_group?(user)
+    return false unless user
+    return true if user.staff? && SiteSetting.shadowban_show_to_staff
     
-    Rails.cache.fetch("approved_user_ids_#{SiteSetting.shadowban_approved_group_name}", expires_in: 5.minutes) do
-      group = Group.find_by(name: SiteSetting.shadowban_approved_group_name)
-      return [] unless group
-      
-      # Récupère les IDs des utilisateurs du groupe approuvé
-      group.users.pluck(:id)
-    end
-  end
-
-  # Méthode pour vérifier si un utilisateur est shadowban (NOT in approved group)
-  def user_is_shadowbanned?(user_id)
-    return false unless SiteSetting.shadowban_enabled
-    return false unless SiteSetting.shadowban_approved_group_name.present?
-    return false if user_id.nil?
+    group_name = SiteSetting.shadowban_allowed_group
+    return true if group_name.blank? # Si aucun groupe n'est défini, tous les utilisateurs sont autorisés
     
-    # Si l'utilisateur n'est PAS dans le groupe approuvé, il est shadowban
-    !approved_user_ids.include?(user_id)
-  end
-
-  # Invalide le cache quand les membres du groupe changent
-  DiscourseEvent.on(:user_added_to_group) do |user, group|
-    if group.name == SiteSetting.shadowban_approved_group_name
-      Rails.cache.delete("approved_user_ids_#{SiteSetting.shadowban_approved_group_name}")
-    end
-  end
-
-  DiscourseEvent.on(:user_removed_from_group) do |user, group|
-    if group.name == SiteSetting.shadowban_approved_group_name
-      Rails.cache.delete("approved_user_ids_#{SiteSetting.shadowban_approved_group_name}")
-    end
+    user.groups.where(name: group_name).exists?
   end
 
-  # Invalide le cache quand le nom du groupe change
-  DiscourseEvent.on(:site_setting_changed) do |name, old_value, new_value|
-    if name == :shadowban_approved_group_name
-      Rails.cache.delete("approved_user_ids_#{old_value}") if old_value.present?
-    end
+  # Méthode helper pour vérifier si un utilisateur est shadowban
+  def self.user_shadowbanned?(user)
+    return false unless user
+    return false if user.staff? && SiteSetting.shadowban_show_to_staff
+    
+    group_name = SiteSetting.shadowban_allowed_group
+    return false if group_name.blank? # Si aucun groupe n'est défini, personne n'est shadowban
+    
+    !user.groups.where(name: group_name).exists?
   end
 
   module ::DiscourseShadowbanTopicView
     def filter_post_types(posts)
       result = super(posts)
       
-      # Le staff peut toujours voir tous les posts
+      # Si l'utilisateur actuel peut voir les posts shadowban, on ne filtre pas
       if SiteSetting.shadowban_show_to_staff && @user&.staff?
         return result
       end
-
-      # Filtre les posts des utilisateurs shadowban
-      if SiteSetting.shadowban_enabled && SiteSetting.shadowban_approved_group_name.present?
-        approved_ids = approved_user_ids
-        if approved_ids.any?
-          # Montre seulement les posts des utilisateurs approuvés + les posts de l'utilisateur actuel
-          result = result.where(
-            'posts.user_id IN (?) OR posts.user_id = ?',
-            approved_ids,
-            @user&.id || 0
-          )
-        else
-          # Si aucun utilisateur approuvé, montre seulement les posts de l'utilisateur actuel
-          result = result.where('posts.user_id = ?', @user&.id || 0)
-        end
-      end
+      
+      group_name = SiteSetting.shadowban_allowed_group
+      return result if group_name.blank?
       
-      result
+      # Exclure les posts des utilisateurs qui ne sont PAS dans le groupe autorisé
+      # sauf si c'est l'utilisateur lui-même qui regarde ses propres posts
+      result.where(
+        'posts.user_id IN (
+          SELECT u.id FROM users u 
+          INNER JOIN group_users gu ON u.id = gu.user_id 
+          INNER JOIN groups g ON gu.group_id = g.id 
+          WHERE g.name = ?
+        ) OR posts.user_id = ?',
+        group_name,
+        @user&.id || 0
+      )
     end
   end
 
···
     def default_results(options = {})
       result = super(options)
       
-      # Le staff peut toujours voir tous les topics
+      # Si l'utilisateur actuel peut voir les topics shadowban, on ne filtre pas
       if SiteSetting.shadowban_show_to_staff && @user&.staff?
         return result
       end
-
-      # Filtre les topics des utilisateurs shadowban
-      if SiteSetting.shadowban_enabled && SiteSetting.shadowban_approved_group_name.present?
-        approved_ids = approved_user_ids
-        if approved_ids.any?
-          # Montre seulement les topics des utilisateurs approuvés + les topics de l'utilisateur actuel
-          result = result.where(
-            'topics.user_id IN (?) OR topics.user_id = ?',
-            approved_ids,
-            @user&.id || 0
-          )
-        else
-          # Si aucun utilisateur approuvé, montre seulement les topics de l'utilisateur actuel
-          result = result.where('topics.user_id = ?', @user&.id || 0)
-        end
-      end
+      
+      group_name = SiteSetting.shadowban_allowed_group
+      return result if group_name.blank?
       
-      result
+      # Exclure les topics créés par des utilisateurs qui ne sont PAS dans le groupe autorisé
+      # sauf si c'est l'utilisateur lui-même qui regarde ses propres topics
+      result.where(
+        'topics.user_id IN (
+          SELECT u.id FROM users u 
+          INNER JOIN group_users gu ON u.id = gu.user_id 
+          INNER JOIN groups g ON gu.group_id = g.id 
+          WHERE g.name = ?
+        ) OR topics.user_id = ?',
+        group_name,
+        @user&.id || 0
+      )
     end
   end
 
···
 
   module ::DiscourseShadowbanPostAlerter
     def create_notification(user, type, post, opts = {})
-      # Le staff reçoit toujours les notifications
-      if SiteSetting.shadowban_show_to_staff && user&.staff?
-        return super(user, type, post, opts)
-      end
-
-      # Ne pas envoyer de notification si le post vient d'un utilisateur shadowban
-      unless user_is_shadowbanned?(post.user_id)
+      # Ne pas créer de notification si l'auteur du post est shadowban
+      # sauf si le destinataire est staff et peut voir les posts shadowban
+      if (SiteSetting.shadowban_show_to_staff && user&.staff?) || 
+         !DiscourseShadowbanGroup.user_shadowbanned?(post.user)
         super(user, type, post, opts)
       end
     end
···
 
   module ::DiscourseShadowbanPostCreator
     def update_topic_stats
-      # Ne pas mettre à jour les stats si l'utilisateur est shadowban
-      unless user_is_shadowbanned?(@post.user_id)
+      # Ne pas mettre à jour les statistiques si l'utilisateur est shadowban
+      unless DiscourseShadowbanGroup.user_shadowbanned?(@post.user)
         super
       end
     end
     
     def update_user_counts
       # Ne pas mettre à jour les compteurs si l'utilisateur est shadowban
-      unless user_is_shadowbanned?(@post.user_id)
+      unless DiscourseShadowbanGroup.user_shadowbanned?(@post.user)
         super
       end
     end
···
     prepend ::DiscourseShadowbanPostCreator
   end
 
-  # Ajoute une méthode helper disponible globalement
-  def self.approved_user_ids
-    return [] unless SiteSetting.shadowban_approved_group_name.present?
-    
-    Rails.cache.fetch("approved_user_ids_#{SiteSetting.shadowban_approved_group_name}", expires_in: 5.minutes) do
-      group = Group.find_by(name: SiteSetting.shadowban_approved_group_name)
-      return [] unless group
-      group.users.pluck(:id)
+  # Créer un module pour les méthodes helper
+  module ::DiscourseShadowbanGroup
+    def self.user_in_allowed_group?(user)
+      return false unless user
+      return true if user.staff? && SiteSetting.shadowban_show_to_staff
+      
+      group_name = SiteSetting.shadowban_allowed_group
+      return true if group_name.blank?
+      
+      user.groups.where(name: group_name).exists?
     end
-  end
 
-  def self.user_is_shadowbanned?(user_id)
-    return false unless SiteSetting.shadowban_enabled
-    return false unless SiteSetting.shadowban_approved_group_name.present?
-    return false if user_id.nil?
-    
-    !approved_user_ids.include?(user_id)
+    def self.user_shadowbanned?(user)
+      return false unless user
+      return false if user.staff? && SiteSetting.shadowban_show_to_staff
+      
+      group_name = SiteSetting.shadowban_allowed_group
+      return false if group_name.blank?
+      
+      !user.groups.where(name: group_name).exists?
+    end
   end
 end