Carico desideroso polimorfico

Usando Rails 3.2, cosa c'è di sbagliato in questo codice?

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

Solleva questo errore:

Impossibile caricare con entusiasmo l'associazione polimorfica: revisionabile

Se rimuovo la reviewable.shop_type = ?condizione, funziona.

Come posso filtrare in base a reviewable_typee reviewable.shop_type(che è effettivamente shop.shop_type)?

La mia ipotesi è che i tuoi modelli abbiano questo aspetto:

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

Non sei in grado di eseguire quella query per diversi motivi.

1. ActiveRecord non è in grado di creare il join senza ulteriori informazioni.
2. Non esiste una tabella denominata revisionabile

Per risolvere questo problema, è necessario definire in modo esplicito la relazione tra 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

Quindi puoi eseguire una query in questo modo:

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

Notare che il nome della tabella è shopse non reviewable. Non dovrebbe esserci una tabella chiamata revisionabile nel database.

Credo che questo sia più facile e più flessibile rispetto alla definizione esplicita del jointra Reviewe Shoppoiché consente di caricare con entusiasmo oltre a eseguire query per campi correlati.

Il motivo per cui ciò è necessario è che ActiveRecord non può creare un join basato sul solo revisionabile, poiché più tabelle rappresentano l'altra estremità del join e SQL, per quanto ne so, non ti consente di unirti a una tabella denominata dal valore memorizzato in una colonna. Definendo la relazione extra belongs_to :shop, si forniscono ad ActiveRecord le informazioni necessarie per completare il join.

Se ottieni un ActiveRecord :: EagerLoadPolymorphicError, è perché ha includesdeciso di chiamare eager_loadquando le associazioni polimorfiche sono supportate solo da preload. È nella documentazione qui: http://api.rubyonrails.org/v5.1/classes/ActiveRecord/EagerLoadPolymorphicError.html

Quindi usa sempre preloadper associazioni polimorfiche. C'è un avvertimento per questo: non è possibile interrogare l'associazione polimorfica nelle clausole where (il che ha senso, poiché l'associazione polimorfica rappresenta più tabelle).

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

Quando si utilizzano frammenti SQL con WHERE, i riferimenti sono necessari per unirsi alla propria associazione.

Come addendum la risposta in alto, che è ottima, puoi anche specificare :includesull'associazione se per qualche motivo la query che stai utilizzando non include la tabella del modello e stai ottenendo errori di tabella non definiti.

Così:

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

Senza l' :includeopzione, se si accede semplicemente all'associazione review.shopnell'esempio sopra, si otterrà un errore UndefinedTable (testato in Rails 3, non 4) perché l'associazione funzionerà SELECT FROM shops WHERE shop.id = 1 AND ( reviews.review_type = 'Shop' ).

L' :includeopzione forzerà invece un JOIN. :)