jaculabilis/danbooru
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

search: fix post count caching for the date: metatag.

Browse Source 
The `date:` metatag depends on the current user's time zone, so the post count for `date:` searches
needs to be cached separately for different users, otherwise it could lead to incorrect page counts
if two users with different time zones search for the same `date:` metatag at the same time.
This commit is contained in:
evazion
2022-11-20 23:45:50 -06:00
parent ba1cf14c7e
commit bc5ca71bdf
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/logical/post_query.rb
View File

@@ -138,7 +138,8 @@ class PostQuery
# True if the search depends on the current user because of permissions or privacy settings. # True if the search depends on the current user because of permissions or privacy settings.
def is_user_dependent_search? def is_user_dependent_search?
metatags.any? do |metatag| metatags.any? do |metatag|
metatag.name.in?(%w[upvoter upvote downvoter downvote commenter comm search flagger fav ordfav favgroup ordfavgroup]) || # XXX date: is user dependent because it depends on the current user's time zone
metatag.name.in?(%w[date upvoter upvote downvoter downvote commenter comm search flagger fav ordfav favgroup ordfavgroup]) ||
metatag.name == "status" && metatag.value == "unmoderated" || metatag.name == "status" && metatag.value == "unmoderated" ||
metatag.name == "disapproved" && !metatag.value.downcase.in?(PostDisapproval::REASONS) metatag.name == "disapproved" && !metatag.value.downcase.in?(PostDisapproval::REASONS)
end end