Полиморфная загрузка

Что не так с этим кодом при использовании Rails 3.2?

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

Это вызывает эту ошибку:

Не могу сразу загрузить полиморфную ассоциацию: доступен для просмотра

Если я уберу reviewable.shop_type = ?условие, оно сработает.

Как я могу фильтровать по reviewable_typeи reviewable.shop_type(что на самом деле shop.shop_type)?

Я предполагаю, что ваши модели выглядят так:

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

Вы не можете выполнить этот запрос по нескольким причинам.

1. ActiveRecord не может создать соединение без дополнительной информации.
2. Нет таблицы с названием для проверки

Чтобы решить эту проблему, вам необходимо явно определить связь между Reviewи 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

Тогда вы можете запросить вот так:

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

Обратите внимание, что имя таблицы - shopsа не reviewable. В базе данных не должно быть таблицы, которая может быть проверена.

Я считаю, что это проще и гибче, чем явное определение joinмежду Reviewи, Shopпоскольку это позволяет вам выполнять загрузку в дополнение к запросам по связанным полям.

Причина, по которой это необходимо, заключается в том, что ActiveRecord не может построить соединение на основе только проверяемого, поскольку несколько таблиц представляют другой конец соединения, а SQL, насколько мне известно, не позволяет вам присоединиться к таблице, названной сохраненным значением в столбце. Определяя дополнительную связь belongs_to :shop, вы даете ActiveRecord информацию, необходимую для завершения соединения.

Если вы получили ActiveRecord :: EagerLoadPolymorphicError, это связано с тем, что было includesрешено вызывать, eager_loadкогда полиморфные ассоциации поддерживаются только preload. Это в документации здесь: http://api.rubyonrails.org/v5.1/classes/ActiveRecord/EagerLoadPolymorphicError.html

Поэтому всегда используйте preloadдля полиморфных ассоциаций. Здесь есть одно предостережение: вы не можете запрашивать полиморфную ассоциацию в предложениях where (что имеет смысл, поскольку полиморфная ассоциация представляет несколько таблиц).

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

Когда вы используете фрагменты SQL с WHERE, ссылки необходимы для присоединения к вашей ассоциации.

В качестве дополнения ответ вверху, который превосходен, вы также можете указать :includeсвязь, если по какой-то причине используемый вами запрос не включает таблицу модели, и вы получаете неопределенные ошибки таблицы.

Вот так:

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

Без этой :includeопции, если вы просто обращаетесь к ассоциации review.shopв приведенном выше примере, вы получите ошибку UndefinedTable (проверено в Rails 3, а не 4), потому что ассоциация будет работать SELECT FROM shops WHERE shop.id = 1 AND ( reviews.review_type = 'Shop' ).

:includeОпция заставит JOIN вместо этого. :)