Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

ActiveRecord::EagerLoadPolymorphicError: Can not eagerly load the polymorphic association

class Transaction < ActiveRecord::Base
  belongs_to :account, :polymorphic => true
end

class Bankaccount < ActiveRecord::Base
  has_many :transactions, :as => :account
end

class Creditcard < ActiveRecord::Base
  has_many :transactions, :as => :account
end

Trying to do a summation of transactions where the account is active.

Transaction.sum(:all, :conditions => "account.status = 'active'", :include => :account)

So after some reading I came across this: The reason is that the parent model‘s type is a column value so its corresponding table name cannot be put in the FROM/JOIN clauses of that query. The table name is bankaccounts and creditcards so does that mean they should be singular? Also the account_type is a string either Bankaccount or Creditcard to reflect the model but should it be the tablename instead?

like image 266
bordicon Avatar asked Jan 17 '10 03:01

bordicon


2 Answers

There are two issues here:

  1. Summing over a polymorphic association.
  2. Condition over a polymorphic association.

You can't actually do either of these things. So you should get the same error by performing these two queries:

  1. Transactions.count(:all, :joins => :account)
  2. Transactions.find(:all, :conditions => "accounts.status = 'active'", :joins => :account)

To actually get the information you need you must explicitly list out the possible parent polymorphic associations. One way to do this is to simply use SQL and LEFT JOINS, so that you can use a single query. Using Rails this can be performed with two queries:

Creditcard.sum(
  :all, 
  :select => "transactions.amount", 
  :conditions => "creditcards.status = 'active'", 
  :joins => :transaction
) + Bankaccount.sum(
  :all, 
  :select => "transactions.amount", 
  :conditions => "bankaccounts.status = 'active'", 
  :joins => :transaction
)

P.S: It's best to use :join instead of :include if you don't plan on accessing the joined objects after the query.

like image 153
Pan Thomakos Avatar answered Sep 22 '22 12:09

Pan Thomakos


For those that got this error even when they aren't trying to query conditions on the polymorphic association, it's because includes decided to call eager_load when polymorphic associations are only supported by preload. It's in the documentation here: http://api.rubyonrails.org/v5.1/classes/ActiveRecord/EagerLoadPolymorphicError.html

So you should always use preload for polymorphic associations.

like image 44
seanmorton Avatar answered Sep 22 '22 12:09

seanmorton