Carga rápida polimórfica

Usando o Rails 3.2, o que há de errado com esse código?

@reviews = @user.reviews.includes(:user, :reviewable)
.where('reviewable_type = ? AND reviewable.shop_type = ?', 'Shop', 'cafe')

Isso gera este erro:

Não é possível carregar avidamente a associação polimórfica: revisável

Se eu remover a reviewable.shop_type = ?condição, funciona.

Como posso filtrar com base no reviewable_typee reviewable.shop_type(que é na verdade shop.shop_type)?

Meu palpite é que seus modelos são assim:

class User < ActiveRecord::Base
  has_many :reviews
end

class Review < ActiveRecord::Base
  belongs_to :user
  belongs_to :reviewable, polymorphic: true
end

class Shop < ActiveRecord::Base
  has_many :reviews, as: :reviewable
end

Você não pode fazer essa consulta por vários motivos.

1. ActiveRecord não é capaz de construir a junção sem informações adicionais.
2. Não há nenhuma tabela chamada revisável

Para resolver esse problema, você precisa definir explicitamente a relação entre Reviewe Shop.

class Review < ActiveRecord::Base
   belongs_to :user
   belongs_to :reviewable, polymorphic: true
   # For Rails < 4
   belongs_to :shop, foreign_key: 'reviewable_id', conditions: "reviews.reviewable_type = 'Shop'"
   # For Rails >= 4
   belongs_to :shop, -> { where(reviews: {reviewable_type: 'Shop'}) }, foreign_key: 'reviewable_id'
   # Ensure review.shop returns nil unless review.reviewable_type == "Shop"
   def shop
     return unless reviewable_type == "Shop"
     super
   end
end

Então você pode consultar assim:

Review.includes(:shop).where(shops: {shop_type: 'cafe'})

Observe que o nome da tabela é shopse não reviewable. Não deve haver uma tabela chamada revisável no banco de dados.

Acredito que isso seja mais fácil e flexível do que definir explicitamente o joinentre Reviewe, Shoppois permite que você carregue com antecedência, além de consultar por campos relacionados.

A razão pela qual isso é necessário é que ActiveRecord não pode construir uma junção baseada apenas em revisável, uma vez que várias tabelas representam a outra extremidade da junção, e o SQL, até onde eu sei, não permite que você junte uma tabela nomeada pelo valor armazenado em uma coluna. Ao definir o relacionamento extra belongs_to :shop, você está fornecendo ao ActiveRecord as informações de que precisa para concluir a associação.

Se você obtiver um ActiveRecord :: EagerLoadPolymorphicError, é porque includesdecidiu chamar eager_loadquando as associações polimórficas são suportadas apenas por preload. Está na documentação aqui: http://api.rubyonrails.org/v5.1/classes/ActiveRecord/EagerLoadPolymorphicError.html

Portanto, sempre use preloadpara associações polimórficas. Há uma advertência para isso: você não pode consultar a associação polimórfica nas cláusulas where (o que faz sentido, já que a associação polimórfica representa várias tabelas).

@reviews = @user.reviews.includes(:user, :reviewable)
.where('reviewable_type = ? AND reviewable.shop_type = ?', 'Shop', 'cafe').references(:reviewable)

Quando você está usando fragmentos SQL com WHERE, referências são necessárias para ingressar em sua associação.

Como adendo a resposta no topo, que é excelente, você também pode especificar :includena associação se por algum motivo a consulta que você está usando não inclui a tabela do modelo e você está obtendo erros de tabela indefinida.

Igual a:

belongs_to :shop, 
           foreign_key: 'reviewable_id', 
           conditions: "reviews.reviewable_type = 'Shop'",
           include: :reviews

Sem a :includeopção, se você simplesmente acessar a associação review.shopno exemplo acima, obterá um erro UndefinedTable (testado no Rails 3, não 4) porque a associação servirá SELECT FROM shops WHERE shop.id = 1 AND ( reviews.review_type = 'Shop' ).

A :includeopção irá forçar um JOIN. :)