fix(APIv2): RHINENG-11984 swap conditions for safer checks

RedHatInsights · Aug 20, 2024 · 50ee9cf · 50ee9cf
1 parent 1092f62
commit 50ee9cf
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/app/models/v2/system.rb b/app/models/v2/system.rb
@@ -123,9 +123,9 @@ def self.first_group_name(table = arel_table)
 
     searchable_by :profile_ref_ids, %i[neq notin] do |_key, _op, val|
       values = val.split.map(&:strip)
-      ids = ::V2::PolicySystem.joins(policy: :profile).where.not(profile: { ref_id: values }).select(:system_id)
+      ids = ::V2::PolicySystem.joins(policy: :profile).where(profile: { ref_id: values }).select(:system_id)
 
-      { conditions: "inventory.hosts.id IN (#{ids.to_sql})" }
+      { conditions: "inventory.hosts.id NOT IN (#{ids.to_sql})" }
     end
 
     scope :with_groups, lambda { |groups, key = :id|