ar_lazy_preload 1.1.2 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5c0652f697df796459ee3750a5a5e54d94bbc1539f0a330f06c24a507f14004e
-  data.tar.gz: ca559bf8f481486ce7fadf75c0e0fa2e2ec1d25babdb1dee590fecbc117c5d3c
+  metadata.gz: 82d5aeed661b9ab42f1d15f78cb8db84cc8b426ddfe26ae1442f297800ec0707
+  data.tar.gz: 1c89c0b2f8025143186551e06629c40a37bf8c207b7605c71481b02c158f89f5
 SHA512:
-  metadata.gz: c3a2bc64e535ff7ba87fa57b568414c87b24ce358bd6587128bfc7d153d055a9187b8162964639d72adf53b22f92fe22970c5a906ce891054161cea5a9e754f2
-  data.tar.gz: aef7c0d8d88f8fea6de1f81abc89baa9aa2fa5fed2b74a9d3eb16e21cd057efacc735c8d46406c54cd4bf3c2e3ac3fed025b95e2a3e9c7188554eff2a6afe442
+  metadata.gz: acbedff3a73e2b26ee0a905c9b5059e9e27297abccd2a16c0a68c3bd67001a4d45721b83aaad21738b36d6c7bce8d41ea6756bd0f1bd54973f339ab2cc26ea68
+  data.tar.gz: 5285e4116f8ba49bc7aa2cbfc1962bfff112785ebd7dbed823ebadf606e9eef1f0772f7f30312344a00cbe8c76af37fc0378d422933073d8758bebe5ae6bcd85

data/README.md

@@ -7,6 +7,12 @@
 - **Perfect fit for GraphQL**. Define a list of associations to load at the top-level resolver and let the gem do its job
 - **Auto-preload support**. If you don't want to specify the association list–set `ArLazyPreload.config.auto_preload` to `true`
 
+Used in production by:
+
+- [Fund that flip](https://evilmartians.com/chronicles/big-refactoring-fix-that-app-for-fund-that-flip)
+- Toptal
+- _Want to be here? Let me know_ 🙂
+
 ## Why should I use it?
 
 Lazy loading is super helpful when the list of associations to load is determined dynamically. For instance, in GraphQL this list comes from the API client, and you'll have to inspect the selection set to find out what associations are going to be used.
@@ -68,6 +74,7 @@ posts = User.preload_associations_lazily.flat_map(&:posts)
 
 2. When `#size` is called on association (e.g., `User.lazy_preload(:posts).map { |u| u.posts.size }`), lazy preloading won't happen, because `#size` method performs `SELECT COUNT()` database request instead of loading the association when association haven't been loaded yet ([here](https://github.com/DmitryTsepelev/ar_lazy_preload/pull/42) is the issue, and [here](https://blazarblogs.wordpress.com/2019/07/27/activerecord-size-vs-count-vs-length/) is the explanation article about `size`, `length` and `count`).
 
+3. Lazy preloading for **ActiveStorage** variants is not working automatically because of the way how it is implemented ([here](https://github.com/DmitryTsepelev/ar_lazy_preload/pull/70) is the issue). You can preload them manually by calling `#with_all_variant_records`/`#with_attached_#{attachment_name}` on association. Example: `User.with_attached_avatar.first(10).map { |u| u.avatar.variant(:small).processed.url }`
 
 ## Installation
 

data/lib/ar_lazy_preload/contexts/base_context.rb

@@ -92,7 +92,7 @@ module ArLazyPreload
 
       def preloadable_reflection?(klass, reflection)
         scope = reflection.scope
-        preloadable_scope = scope&.arity&.zero?
+        preloadable_scope = scope&.arity&.zero? || ::ActiveRecord::VERSION::MAJOR >= 7
         through_reflection =
           reflection.options[:through] && klass.reflect_on_association(reflection.options[:through])
         preloadable_through_reflection =

data/lib/ar_lazy_preload/railtie.rb

@@ -1,32 +1,10 @@
 # frozen_string_literal: true
 
-require "ar_lazy_preload/active_record/base"
-require "ar_lazy_preload/active_record/relation"
-require "ar_lazy_preload/active_record/association"
-require "ar_lazy_preload/active_record/collection_association"
-require "ar_lazy_preload/active_record/merger"
-require "ar_lazy_preload/active_record/association_relation"
-require "ar_lazy_preload/active_record/collection_proxy"
-
 module ArLazyPreload
   class Railtie < Rails::Railtie
     config.to_prepare do |_app|
       ActiveSupport.on_load(:active_record) do
-        ActiveRecord::Base.include(Base)
-
-        ActiveRecord::Relation.prepend(Relation)
-        ActiveRecord::AssociationRelation.prepend(AssociationRelation)
-        ActiveRecord::Relation::Merger.prepend(Merger)
-
-        [
-          ActiveRecord::Associations::CollectionAssociation,
-          ActiveRecord::Associations::Association
-        ].each { |klass| klass.prepend(Association) }
-
-        ActiveRecord::Associations::CollectionAssociation.prepend(CollectionAssociation)
-        ActiveRecord::Associations::CollectionProxy.prepend(CollectionProxy)
-
-        ArLazyPreload::Preloader.patch_for_rails_7! if ActiveRecord::VERSION::MAJOR >= 7
+        ArLazyPreload.install_hooks
       end
     end
   end

data/lib/ar_lazy_preload/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ArLazyPreload
-  VERSION = "1.1.2"
+  VERSION = "2.1.0"
 end

data/lib/ar_lazy_preload.rb

@@ -1,12 +1,36 @@
 # frozen_string_literal: true
 
 require "ar_lazy_preload/configuration"
-require "ar_lazy_preload/railtie"
+require "ar_lazy_preload/railtie" if defined?(::Rails)
+
+require "ar_lazy_preload/active_record/base"
+require "ar_lazy_preload/active_record/relation"
+require "ar_lazy_preload/active_record/association"
+require "ar_lazy_preload/active_record/collection_association"
+require "ar_lazy_preload/active_record/merger"
+require "ar_lazy_preload/active_record/association_relation"
+require "ar_lazy_preload/active_record/collection_proxy"
 
 module ArLazyPreload
   class << self
     def config
       @config ||= Configuration.new
     end
+
+    def install_hooks
+      ActiveRecord::Base.include(Base)
+
+      ActiveRecord::Relation.prepend(Relation)
+      ActiveRecord::AssociationRelation.prepend(AssociationRelation)
+      ActiveRecord::Relation::Merger.prepend(Merger)
+
+      ActiveRecord::Associations::CollectionAssociation.prepend(Association)
+      ActiveRecord::Associations::Association.prepend(Association)
+
+      ActiveRecord::Associations::CollectionAssociation.prepend(CollectionAssociation)
+      ActiveRecord::Associations::CollectionProxy.prepend(CollectionProxy)
+
+      ArLazyPreload::Preloader.patch_for_rails_7! if ActiveRecord::VERSION::MAJOR >= 7
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ar_lazy_preload
 version: !ruby/object:Gem::Version
-  version: 1.1.2
+  version: 2.1.0
 platform: ruby
 authors:
 - DmitryTsepelev
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-12-05 00:00:00.000000000 Z
+date: 2024-03-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.2'
-  type: :runtime
+        version: '6.1'
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '6.1'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -58,14 +58,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.81.0
+        version: 1.57.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.81.0
+        version: 1.57.2
 - !ruby/object:Gem::Dependency
   name: db-query-matchers
   requirement: !ruby/object:Gem::Requirement
@@ -212,7 +212,7 @@ homepage: https://github.com/DmitryTsepelev/ar_lazy_preload
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -220,15 +220,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.6'
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key: 
+rubygems_version: 3.4.10
+signing_key:
 specification_version: 4
 summary: lazy_preload implementation for ActiveRecord models
 test_files: []