From f6fff16e759ec206c9ea4151cc7f903dc2306384 Mon Sep 17 00:00:00 2001 From: evazion Date: Thu, 13 Apr 2017 00:06:02 -0500 Subject: [PATCH] /bans: avoid N+1 queries for user, banner. Avoids an N+1 issue when rendering users with link_to_user. --- app/controllers/bans_controller.rb | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/app/controllers/bans_controller.rb b/app/controllers/bans_controller.rb index b3eba6c9c..9ce26ebba 100644 --- a/app/controllers/bans_controller.rb +++ b/app/controllers/bans_controller.rb @@ -11,9 +11,10 @@ class BansController < ApplicationController end def index - @bans = Ban.search(params[:search]) - @bans = @bans.paginate(params[:page], :limit => params[:limit]) - respond_with(@bans) + @bans = Ban.search(params[:search]).paginate(params[:page], :limit => params[:limit]) + respond_with(@bans) do |fmt| + fmt.html { @bans = @bans.includes(:user, :banner) } + end end def show