5

我正在使用public_activity和我的一个部分我有这个电话：

<%= link_to "#{activity.trackable.user.name} " + "commented on " + "#{activity.trackable.node.name}", node_path(activity.trackable.node.family_tree_id,activity.trackable.node) %>

这是由这个调用执行的：

      <% @unread_act.each do |friend| %>
        <li><%= render_activity friend %> </li>
      <% end %>

该实例变量在我的ApplicationController此处分配：

  before_filter :handle_nofications

  def handle_nofications
    if current_user.present?
      @unread_act = Notification.where(owner_id: current_user).includes(:trackable => :user).unread_by(current_user)
    end
  end

我正在使用 gem Bullet 来查找 N+1 个查询，我得到的反馈是这样的：

N+1 Query detected
  Comment => [:node]
  Add to your finder: :include => [:node]
N+1 Query method call stack

我最初得到了这个消息 for:trackable和 for :user。我现在都渴望通过includes该赋值语句进行加载。

但是，我不知道如何在急切加载时深入 3 个级别 - 而不仅仅是两个级别。

鉴于该节点被称为 like activity.trackable.node.name，我想一些适当的急切加载分配看起来像：

  @unread_act = Notification.where(owner_id: current_user).includes(:trackable => [:user, :node]).unread_by(current_user)

但我得到的错误是：

ActiveRecord::AssociationNotFoundError at /
Association named 'node' was not found on Node; perhaps you misspelled it?

当我简单地这样做时，我什至得到了：

@unread_act = Notification.where(owner_id: current_user).includes(:trackable => :node).unread_by(current_user)

所以我怀疑这里出了点问题。

有任何想法吗？

编辑 1

请参阅下面的节点和通知模型和关联。

Node.rb

class Node < ActiveRecord::Base
  include PublicActivity::Model
  tracked except: :update, owner: ->(controller, model) { controller && controller.current_user }

  belongs_to :family_tree
  belongs_to :user
  belongs_to :media, polymorphic: true, dependent: :destroy
  has_many :comments, dependent: :destroy
  has_many :node_comments, dependent: :destroy

Notification.rb

class Notification < PublicActivity::Activity
  acts_as_readable :on => :created_at
end
4

2 回答 2

1

这有望在 Rails 5.0 中得到修复。已经有一个问题和一个拉取请求。

https://github.com/rails/rails/pull/17479

https://github.com/rails/rails/issues/8005

我已经分叉了导轨并将补丁应用于 4.2-stable，它对我来说非常有用。随意使用它，即使我不能保证定期与上游同步。

https://github.com/ttosch/rails/tree/4-2-stable

于 2015-01-19T12:01:05.790 回答
0

似乎解决此问题的正确方法是在trackable属性上使用数组，如下所示：

@unread_act = Notification.where(recipient_id: current_user).includes(:trackable => [:user, :node]).unread_by(current_user).order("created_at desc")

主要部分是：

includes(:trackable => [:user, :node])

这似乎工作得很好。

于 2014-11-07T21:12:35.867 回答