graphql-preload 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0f2b5fb449b2e31a81bf5bf828987a57c355372c
-  data.tar.gz: 170b4686a7424c01e650844b73181496aaf3f3ce
+  metadata.gz: 6bbbde895d17e5914f30802362a2453cc4f3de89
+  data.tar.gz: b11f2782b0a1cb286d867e83fff3da24c1650c5c
 SHA512:
-  metadata.gz: 0632e5d4d71b0f3087f56e659f2243008c0d4ee9ded0e62cf6d71fd54037e489cd3e20ec8482a9f3defdacf749d591b23082f60d5e42d798d9b84d2357442ee1
-  data.tar.gz: 30027b5d03c0346771062ba8d8a4db3eb8e80e4bfbd0d18e439b49b9f883e9e1ab704075ba273bd45e5dd4a0128e7d0c1da8f7bb5e5e42e17be9d88fc376be04
+  metadata.gz: 2b6f8a34abdaf6ce6624d38cd230d49864dde3e2b3a93561019b23562d6870819605da476125512c70d621933af1019f0c508bee872baaf518308820b4854a4a
+  data.tar.gz: a89501116230eca584019915d5e66b2c4ef019ed70c12151cc7a474587e1189f3f6b584a8c739ef5887b0869be357524afc3525883e4689435654dec642ae11f

data/README.md

@@ -40,8 +40,8 @@ Call `preload` when defining your field:
         # Post.includes(:comments, :authors)
         preload [:comments, :authors]
 
-        # Post.includes(:comments, authors: [:posts, :followers])
-        preload [:comments, { authors: [:posts, :followers] }]
+        # Post.includes(:comments, authors: [:followers, :posts])
+        preload [:comments, { authors: [:followers, :posts] }]
 
         resolve ->(obj, args, ctx) { obj.comments }
       end

data/lib/graphql/preload/instrument.rb

@@ -33,17 +33,17 @@ module GraphQL
               promises << preload(record, sub_association)
             end
           when Hash
-            association.each do |sub_association, property|
+            association.each do |sub_association, nested_association|
               promises << preload_single_association(record, sub_association).then do
                 associated_records = record.public_send(sub_association)
 
                 case associated_records
                 when ActiveRecord::Base
-                  preload(associated_records, property)
+                  preload(associated_records, nested_association)
                 else
                   Promise.all(
                     Array.wrap(associated_records).map do |associated_record|
-                      preload(associated_record, property)
+                      preload(associated_record, nested_association)
                     end
                   )
                 end
@@ -56,7 +56,6 @@ module GraphQL
       end
 
       private def preload_single_association(record, association)
-        return Promise.resolve(record) if record.association(association).loaded?
         GraphQL::Preload::Loader.for(record.class, association).load(record)
       end
     end

data/lib/graphql/preload/loader.rb

@@ -4,6 +4,10 @@ module GraphQL
     class Loader < GraphQL::Batch::Loader
       attr_reader :association, :model
 
+      def cache_key(record)
+        record.object_id
+      end
+
       def initialize(model, association)
         @association = association
         @model = model
@@ -13,38 +17,41 @@ module GraphQL
 
       def load(record)
         unless record.is_a?(model)
-          raise TypeError, "loader for #{model.name} can't load associations for #{record.class.name} objects"
+          raise TypeError, "Loader for #{model} can't load associations for #{record.class} objects"
         end
 
-        if record.association(association).loaded?
-          Promise.resolve(record)
-        else
-          super
-        end
+        return Promise.resolve(record) if association_loaded?(record)
+        super
       end
 
       def perform(records)
+        preload_association(records)
+        records.each { |record| fulfill(record, record) }
+      end
+
+      private def association_loaded?(record)
+        record.association(association).loaded?
+      end
+
+      private def preload_association(records)
         if ActiveRecord::VERSION::MAJOR > 3
           ActiveRecord::Associations::Preloader.new.preload(records, association)
         else
           ActiveRecord::Associations::Preloader.new(records, association).run
         end
-
-        records.each { |record| fulfill(record, record) }
       end
 
       private def validate_association
         unless association.is_a?(Symbol)
-          raise ArgumentError, 'association must be a Symbol object'
+          raise ArgumentError, 'Association must be a Symbol object'
         end
 
         unless model < ActiveRecord::Base
-          raise ArgumentError, 'model must be an ActiveRecord::Base descendant'
+          raise ArgumentError, 'Model must be an ActiveRecord::Base descendant'
         end
 
         return if model.reflect_on_association(association)
-
-        raise TypeError, "association :#{association} does not exist on #{model.name}"
+        raise TypeError, "Association :#{association} does not exist on #{model}"
       end
     end
   end

data/lib/graphql/preload/version.rb

@@ -1,5 +1,5 @@
 module GraphQL
   module Preload
-    VERSION = '1.0.2'.freeze
+    VERSION = '1.0.3'.freeze
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: graphql-preload
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - Ryan Foster, Etienne Tripier
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-07-14 00:00:00.000000000 Z
+date: 2017-08-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord