active_model_serializers_pg 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4bdfe8c6e1f9a0fbca6e444a7594474d43d9415008c86f6fe30964088fc8c1f7
-  data.tar.gz: 0e5a9f884cda2663bf2f36051ad42367381ea5f61798048cd4ae774c19c76265
+  metadata.gz: c61aa96dfaadcdbb0304b0a8ea50b05468ad7474c6d3b740419c8bf525f28c01
+  data.tar.gz: 1d5dfd08ff5168add108de4854246f48ed564b0548d8a66155068d26d41f4805
 SHA512:
-  metadata.gz: 44df1b79df5306e323a62c6fd0a6d0b52e6f2da6da8d6a6eb494fb440a9e09909ecd476cf563fa873a42bddedc40dbdc0efefb95cf13982ca69684b11c93050d
-  data.tar.gz: d08476c02adb38cb0978910b0d2448c0f09019ba52592132105612846e51d883d5ab0f26c6ad0f3eaf66c3757f3a93ab59831bfb6652459d47cdf8de37ba342d
+  metadata.gz: 32d433d676644d637de1cfda49d340f35221a6f1a08a5b8b74f605018e5f4169a45ec50bfcafe2c2fc20cb95e53cdaa98123e2b25b3246cafb276c2e68926301
+  data.tar.gz: d21913bd9baa516e016adcea39c28f4ce05446753553676485fd75cca1fbce1e36727df16bca2d6004b999ba6850d9ec14dc6bf29ac2f1c25660ee3b84dd31ae

data/lib/active_model_serializers_pg/collection_serializer.rb

@@ -15,6 +15,7 @@ module ActiveModelSerializersPg
     include Enumerable
     # PATCHED: implement this below so we don't need @serializers
     # delegate :each, to: :@serializers
+    delegate :each, to: :serializers
 
     attr_reader :object, :root
 
@@ -22,8 +23,13 @@ module ActiveModelSerializersPg
       @object                  = resources
       @options                 = options
       @root                    = options[:root]
-      # PATCHED: We don't want to iterate unless we have to:
-      # @serializers             = serializers_from_resources
+      # PATCHED: We don't want to materialize a Relation by iterating unless we have to.
+      # On the other hand, if we don't have a serializer we *do* want to `throw :no_serializer`
+      # right away. That should only happen for basic types (like a String or Hash),
+      # so we act lazy when we have a Relation, and eager otherwise:
+      unless resources.is_a? ActiveRecord::Relation
+        @serializers = serializers_from_resources
+      end
     end
 
     # PATCH: Give ourselves access to the serializer for the individual elements:
@@ -76,25 +82,17 @@ module ActiveModelSerializersPg
         object.respond_to?(:size)
     end
 
-    # PATCHED: Add a replacement to `each` so we don't change the interface:
-    def each
-      Rails.logger.debug caller.join("\n")
-      Enumerator.new do |y|
-        serializers_from_resources.each do |ser|
-          y.yield ser
-        end
-      end
-    end
-
     protected
 
-    attr_reader :serializers, :options
+    attr_reader :options
+
+    def serializers
+      @serializers ||= serializers_from_resources
+    end
 
     private
 
     def serializers_from_resources
-      puts "options here"
-      pp options
       serializer_context_class = options.fetch(:serializer_context_class, ActiveModel::Serializer)
       object.map do |resource|
         serializer_from_resource(resource, serializer_context_class, options)

data/lib/active_model_serializers_pg/version.rb

@@ -1,3 +1,3 @@
 module ActiveModelSerializersPg
-  VERSION = '0.0.2'
+  VERSION = '0.0.3'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_model_serializers_pg
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - Paul A. Jungwirth
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-10-15 00:00:00.000000000 Z
+date: 2019-10-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active_model_serializers