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

posts: reduce string allocations during thumbnail generation.

Browse Source 
Further micro-optimize thumbnails to reduce string allocations.

`Post#levelblocked?` gets called once per thumbnail. Before it split the
tag string, which meant one string allocation for each tag on each post.
This added up to thousands of string allocations per pageload.
This commit is contained in:
evazion
2021-12-16 16:30:31 -06:00
parent 1c5786d20f
commit 993965b654
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
app/models/post.rb
View File

@@ -8,7 +8,7 @@ class Post < ApplicationRecord
NOTE_COPY_TAGS = %w[translated partially_translated check_translation translation_request reverse_translation NOTE_COPY_TAGS = %w[translated partially_translated check_translation translation_request reverse_translation
annotated partially_annotated check_annotation annotation_request] annotated partially_annotated check_annotation annotation_request]
RESTRICTED_TAGS = Danbooru.config.restricted_tags RESTRICTED_TAGS_REGEX = /(?:^| )(?:#{Danbooru.config.restricted_tags.join("|")})(?:$| )/o
self.ignored_columns = [:pool_string, :fav_string] self.ignored_columns = [:pool_string, :fav_string]
@@ -1289,7 +1289,8 @@ class Post < ApplicationRecord
end end
def levelblocked?(user = CurrentUser.user) def levelblocked?(user = CurrentUser.user)
!user.is_gold? && RESTRICTED_TAGS.any? { |tag| tag.in?(tag_array) } #!user.is_gold? && RESTRICTED_TAGS.any? { |tag| has_tag?(tag) }
!user.is_gold? && tag_string.match?(RESTRICTED_TAGS_REGEX)
end end
def banblocked?(user = CurrentUser.user) def banblocked?(user = CurrentUser.user)