maverick/docbrown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Optimize FeedbackMessagesController#index (#7736)

Browse source
This commit is contained in:
Mac Siri 2020-05-11 12:35:08 -04:00 • committed by GitHub
parent e28664c67f
commit 1c195b02de
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
app/controllers/internal/feedback_messages_controller.rb
View file

@ -11,7 +11,7 @@ class Internal::FeedbackMessagesController < Internal::ApplicationController
@status = params[:status] || "Open"
@email_messages = EmailMessage.find_for_reports(@feedback_messages)
@new_articles = Article.published.includes(:user).
@new_articles = Article.published.select(:title, :user_id, :path).includes(:user).
order(created_at: :desc).
where("score > ? AND score < ?", -10, 8).
limit(120)
@ -22,6 +22,7 @@ class Internal::FeedbackMessagesController < Internal::ApplicationController
).
where("created_at > ?", 48.hours.ago).
order(created_at: :desc).
select(:username, :name, :id).
where.not("username LIKE ?", "%spam_%").
limit(150)
@ -79,20 +80,15 @@ class Internal::FeedbackMessagesController < Internal::ApplicationController
private
def get_vomits
q = Reaction.includes(:user, :reactable).
select(:id, :user_id, :reactable_type, :reactable_id).
order(updated_at: :desc)
if params[:status] == "Open" || params[:status].blank?
Reaction.where(category: "vomit", status: "valid").
includes(:user, :reactable).
order(updated_at: :desc)
q.where(category: "vomit", status: "valid")
elsif params[:status] == "Resolved"
Reaction.where(category: "vomit", status: "confirmed").
includes(:user, :reactable).
order(updated_at: :desc).
limit(10)
q.where(category: "vomit", status: "confirmed").limit(10)
else
Reaction.where(category: "vomit", status: "invalid").
includes(:user, :reactable).
order(updated_at: :desc).
limit(10)
q.where(category: "vomit", status: "invalid").limit(10)
end
end