Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix erroneous media search #593

Merged
merged 11 commits into from
Feb 16, 2024
15 changes: 0 additions & 15 deletions app/controllers/media_controller.rb
Splines marked this conversation as resolved.
Show resolved Hide resolved
Original file line number Diff line number Diff line change
Expand Up @@ -228,21 +228,6 @@ def search
results = search.results
@total = search.total

# in the case of a search with tag_operator 'or', we
# execute two searches and merge the results, where media
# with the selected tags are now shown at the front of the list
if (search_params[:tag_operator] == "or") \
&& (search_params[:all_tags] == "0") \
&& (search_params[:fulltext].size >= 2)
params["search"]["all_tags"] = "1"
search_no_tags = Medium.search_by(search_params, params[:page])
search_no_tags.execute
results_no_tags = search_no_tags.results
results = (results + results_no_tags).uniq
@total = results.size
params["search"]["all_tags"] = "0"
end
Splines marked this conversation as resolved.
Show resolved Hide resolved

if filter_media
search_arel = Medium.where(id: results.pluck(:id))
visible_search_results = current_user.filter_visible_media(search_arel)
Expand Down
13 changes: 5 additions & 8 deletions app/models/medium.rb
Original file line number Diff line number Diff line change
Expand Up @@ -336,15 +336,12 @@ def self.search_by(search_params, _page)
with(:release_state, search_params[:access])
end
end
if !search_params[:all_tags] == "1" &&
!search_params[:tag_operator] == "or" && (search_params[:tag_ids])
if search_params[:tag_operator] == "or" || search_params[:all_tags] == "1"
fosterfarrell9 marked this conversation as resolved.
Show resolved Hide resolved
search.build do
with(:tag_ids).any_of(search_params[:tag_ids])
end
else
search.build do
if search_params[:all_tags] == "0" && search_params[:tag_ids].any?
search.build do
if search_params[:tag_operator] == "and"
with(:tag_ids).all_of(search_params[:tag_ids])
else
with(:tag_ids).any_of(search_params[:tag_ids])
end
end
end
Expand Down