spree/spree — pull request #3687 by huoxito (2013-09-07 06:09:47+00:00)
Title: Fix variant eager loading in Shipment `manifest`

Deleted variants were not being fetched due to a default scope. Adding a
`joins` seems to override the default scope and fix exceptions in
backend when trying to view orders with deleted variants

default_scope seems to be evil. It might save us from adding scope in some places but also create issues in many others.

This one should fix #3673