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

FIX: n+1 query on list of private messages assigned (#216) 

When getting a list of private messages assigned to groups/users, we should include :allowed_users to avoid n+1
This commit is contained in:
Krzysztof Kotlarek 2021-09-29 11:01:15 +10:00 committed by GitHub
parent dd37f66f5d
commit 3a58df219c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
plugin.rb
View File

@ -294,7 +294,7 @@ after_initialize do
sql = "topics.id IN (#{topic_ids_sql})" sql = "topics.id IN (#{topic_ids_sql})"
list = list.where(sql, user_id: user.id) list = list.where(sql, user_id: user.id).includes(:allowed_users)
create_list(:assigned, { unordered: true }, list) create_list(:assigned, { unordered: true }, list)
end end
@ -319,7 +319,7 @@ after_initialize do
sql = "topics.id IN (#{topic_ids_sql})" sql = "topics.id IN (#{topic_ids_sql})"
list = list.where(sql, group_id: group.id) list = list.where(sql, group_id: group.id).includes(:allowed_users)
create_list(:assigned, { unordered: true }, list) create_list(:assigned, { unordered: true }, list)
end end