PERF: N+1 query on user summary page.

2016-09-23 12:44:08 +08:00 · 2016-09-23 12:44:08 +08:00 · 4e663998af
parent bddbfa56f6
commit 4e663998af
1 changed files with 5 additions and 1 deletions
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@ -33,7 +33,11 @@ class UsersController < ApplicationController
  def show
    raise Discourse::InvalidAccess if SiteSetting.hide_user_profiles_from_public && !current_user

-    @user = fetch_user_from_params(include_inactive: current_user.try(:staff?))
+    @user = fetch_user_from_params(
+      { include_inactive: current_user.try(:staff?) },
+      [{ user_profile: :card_image_badge }]
+    )
+
    user_serializer = UserSerializer.new(@user, scope: guardian, root: 'user')

    # TODO remove this options from serializer