aboutsummaryrefslogtreecommitdiffstats
path: root/lib/active_storage
diff options
context:
space:
mode:
authorDavid Heinemeier Hansson <david@loudthinking.com>2017-07-23 16:47:11 -0500
committerDavid Heinemeier Hansson <david@loudthinking.com>2017-07-23 16:47:11 -0500
commit68b5d274a365c1babdb92dedfcf2e600138be5eb (patch)
tree1ad85544cd7bda1fd8ab5547f37dec2ddf9efc91 /lib/active_storage
parentf96abd1dfc858ce73afcf54c87ed4e88401a7048 (diff)
downloadrails-68b5d274a365c1babdb92dedfcf2e600138be5eb.tar.gz
rails-68b5d274a365c1babdb92dedfcf2e600138be5eb.tar.bz2
rails-68b5d274a365c1babdb92dedfcf2e600138be5eb.zip
Add and test preloading scope
Diffstat (limited to 'lib/active_storage')
-rw-r--r--lib/active_storage/attached/macros.rb6
1 files changed, 6 insertions, 0 deletions
diff --git a/lib/active_storage/attached/macros.rb b/lib/active_storage/attached/macros.rb
index 0452089a5f..5915793f8a 100644
--- a/lib/active_storage/attached/macros.rb
+++ b/lib/active_storage/attached/macros.rb
@@ -33,6 +33,10 @@ module ActiveStorage::Attached::Macros
# There are no columns defined on the model side, Active Storage takes
# care of the mapping between your records and the attachments.
#
+ # To avoid N+1 queries, you can include the attached blobs in your query like so:
+ #
+ # Gallery.where(user: Current.user).with_attached_photos
+ #
# If the +:dependent+ option isn't set, all the attachments will be purged
# (i.e. destroyed) whenever the record is destroyed.
def has_many_attached(name, dependent: :purge_later)
@@ -44,6 +48,8 @@ module ActiveStorage::Attached::Macros
has_many :"#{name}_attachments", -> { where(name: name) }, as: :record, class_name: "ActiveStorage::Attachment"
has_many :"#{name}_blobs", through: :"#{name}_attachments"
+ scope :"with_attached_#{name}", -> { includes("#{name}_attachments": :blob) }
+
if dependent == :purge_later
before_destroy { public_send(name).purge_later }
end