From 436f3eaf5d6af7c6f5abd86767cce87e1a71dcb5 Mon Sep 17 00:00:00 2001 From: evazion Date: Sun, 17 Apr 2022 23:50:33 -0500 Subject: [PATCH] posts: eliminate redundant tag load on posts show page. Eliminate an unnecessary `SELECT name, post_count, category FROM tags WHERE name IN (?)` query on the post show page. This query was used by `humanized_essential_tag_string` when generating the page title. This query was redundant because we already use another query to load the tag list with `SELECT * FROM tags WHERE name IN (?)`. --- app/presenters/tag_set_presenter.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/presenters/tag_set_presenter.rb b/app/presenters/tag_set_presenter.rb index e893b8f90..f0b4feb57 100644 --- a/app/presenters/tag_set_presenter.rb +++ b/app/presenters/tag_set_presenter.rb @@ -44,7 +44,7 @@ class TagSetPresenter private def tags - Tag.where(name: tag_names).select(:name, :post_count, :category) + Tag.where(name: tag_names) end def tags_by_category