[#2301] Quick fix: users with is_discoverable == false (default!) are included in search results.

This commit is contained in:
Ivan Tashkinov 2020-11-19 18:08:22 +03:00 committed by lain
parent 897b62091b
commit 5cfa32cb50
2 changed files with 8 additions and 5 deletions

View File

@ -85,7 +85,7 @@ defp search_query(query_string, for_user, following, top_user_ids) do
|> base_query(following)
|> filter_blocked_user(for_user)
|> filter_invisible_users()
|> filter_discoverable_users()
|> filter_non_discoverable_users()
|> filter_internal_users()
|> filter_blocked_domains(for_user)
|> fts_search(query_string)
@ -163,8 +163,10 @@ defp filter_invisible_users(query) do
from(q in query, where: q.invisible == false)
end
defp filter_discoverable_users(query) do
from(q in query, where: q.discoverable == true)
defp filter_non_discoverable_users(query) do
# Note: commented out — can't do it with users being non-discoverable by default
# from(q in query, where: q.is_discoverable == true)
query
end
defp filter_internal_users(query) do

View File

@ -65,12 +65,13 @@ test "excludes invisible users from results" do
assert found_user.id == user.id
end
test "excludes users when discoverable is false" do
test "does NOT exclude non-discoverable users from results (as long as it's the default)" do
# NOTE: as long as users are non-discoverable by default, we can't filter out most users: #2301
insert(:user, %{nickname: "john 3000", discoverable: false})
insert(:user, %{nickname: "john 3001"})
users = User.search("john")
assert Enum.count(users) == 1
assert Enum.count(users) == 2
end
test "excludes service actors from results" do