discourse
/
discourse-assign
mirror of https://github.com/discourse/discourse-assign.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

DEV: use argument instead of instance variable

This commit is contained in:
zogstrip 2025-03-20 10:31:45 +01:00
parent a024e26d2d
commit 06985c8da9
No known key found for this signature in database
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
plugin.rb
View File

@ -887,11 +887,13 @@ after_initialize do
Assignment.active_for_group(group).destroy_all
end
add_filter_custom_filter("assigned") do |scope, filter_values|
add_filter_custom_filter("assigned") do |scope, filter_values, guardian|
return if !guardian.can_assign? || filter_values.blank?
user_or_group_name = filter_values.compact.first
return scope if user_or_group_name.blank?
return if user_or_group_name.blank?
if user_id = User.find_by_username(user_or_group_name)&.id
scope.where(<<~SQL, user_id)
topics.id IN (SELECT a.topic_id FROM assignments a WHERE a.assigned_to_id = ? AND a.assigned_to_type = 'User' AND a.active)
@ -900,8 +902,6 @@ after_initialize do
scope.where(<<~SQL, group_id)
topics.id IN (SELECT a.topic_id FROM assignments a WHERE a.assigned_to_id = ? AND a.assigned_to_type = 'Group' AND a.active)
SQL
else
scope
end
end
@ -913,7 +913,7 @@ after_initialize do
register_search_advanced_filter(/in:unassigned/) do |posts|
return if !@guardian.can_assign?
posts.where("topics.id NOT IN (SELECT a.topic_id FROM assignments a WHERE a.active)")
end