Skip to content

Instantly share code, notes, and snippets.

@guilleiguaran
Created October 15, 2014 21:40
Show Gist options
  • Save guilleiguaran/374661df31b122926e4b to your computer and use it in GitHub Desktop.
Save guilleiguaran/374661df31b122926e4b to your computer and use it in GitHub Desktop.
diff --git a/lib/active_model/serializer/adapter.rb b/lib/active_model/serializer/adapter.rb
index 8eacc39..f0a8257 100644
--- a/lib/active_model/serializer/adapter.rb
+++ b/lib/active_model/serializer/adapter.rb
@@ -17,13 +17,7 @@ module ActiveModel
end
def to_json(options = {})
- result = serializable_hash(options)
-
- if root = options.fetch(:root, serializer.json_key)
- result = { root => result }
- end
-
- result.to_json
+ serializable_hash(options).to_json
end
end
end
diff --git a/lib/active_model/serializer/adapter/json.rb b/lib/active_model/serializer/adapter/json.rb
index 82bc319..bab3e44 100644
--- a/lib/active_model/serializer/adapter/json.rb
+++ b/lib/active_model/serializer/adapter/json.rb
@@ -3,18 +3,25 @@ module ActiveModel
class Adapter
class Json < Adapter
def serializable_hash(options = {})
- @hash = serializer.attributes(options)
+ if serializer.respond_to?(:each)
+ @result = serializer.map{|s| self.class.new(s).serializable_hash }
+ else
+ @result = serializer.attributes(options)
- serializer.each_association do |name, association, options|
- if association.respond_to?(:each)
- array_serializer = association
- @hash[name] = array_serializer.map { |item| item.attributes(options) }
- else
- @hash[name] = association.attributes(options)
+ serializer.each_association do |name, association, opts|
+ if association.respond_to?(:each)
+ array_serializer = association
+ @result[name] = array_serializer.map { |item| item.attributes(opts) }
+ else
+ @result[name] = association.attributes(options)
+ end
end
end
- @hash
+ if root = options.fetch(:root, serializer.json_key)
+ @result = { root => @result }
+ end
+ @result
end
end
end
diff --git a/lib/active_model/serializer/adapter/json_api.rb b/lib/active_model/serializer/adapter/json_api.rb
index 7634061..2e361cc 100644
--- a/lib/active_model/serializer/adapter/json_api.rb
+++ b/lib/active_model/serializer/adapter/json_api.rb
@@ -4,22 +4,30 @@ module ActiveModel
class JsonApi < Adapter
def initialize(serializer, options = {})
super
- serializer.root ||= true
+ serializer.root = true
end
- def serializable_hash(opts = {})
- @hash = serializer.attributes
+ def serializable_hash(options = {})
+ @root = options[:root] || serializer.json_key
- serializer.each_association do |name, association, options|
- @hash[:links] ||= {}
- unless options[:embed] == :ids
- @hash[:linked] ||= {}
- end
+ @hash = {}
+
+ if serializer.respond_to?(:each)
+ @hash[@root] = serializer.map{|s| self.class.new(s).serializable_hash }
+ else
+ @hash[@root] = serializer.attributes
+
+ serializer.each_association do |name, association, opts|
+ @hash[@root][:links] ||= {}
+ unless options[:embed] == :ids
+ @hash[:linked] ||= {}
+ end
- if association.respond_to?(:each)
- add_links(name, association, options)
- else
- add_link(name, association, options)
+ if association.respond_to?(:each)
+ add_links(name, association, opts)
+ else
+ add_link(name, association, opts)
+ end
end
end
@@ -27,8 +35,8 @@ module ActiveModel
end
def add_links(name, serializers, options)
- @hash[:links][name] ||= []
- @hash[:links][name] += serializers.map(&:id)
+ @hash[@root][:links][name] ||= []
+ @hash[@root][:links][name] += serializers.map(&:id)
unless options[:embed] == :ids
@hash[:linked][name] ||= []
@@ -37,7 +45,7 @@ module ActiveModel
end
def add_link(name, serializer, options)
- @hash[:links][name] = serializer.id
+ @hash[@root][:links][name] = serializer.id
unless options[:embed] == :ids
plural_name = name.to_s.pluralize.to_sym
diff --git a/lib/active_model/serializer/array_serializer.rb b/lib/active_model/serializer/array_serializer.rb
index 697c445..c7af8d3 100644
--- a/lib/active_model/serializer/array_serializer.rb
+++ b/lib/active_model/serializer/array_serializer.rb
@@ -10,6 +10,10 @@ module ActiveModel
serializer_class.new(object)
end
end
+
+ def json_key
+ @objects.first.json_key
+ end
end
end
end
diff --git a/test/action_controller/serialization_test.rb b/test/action_controller/serialization_test.rb
index dcfd9ea..d0e9244 100644
--- a/test/action_controller/serialization_test.rb
+++ b/test/action_controller/serialization_test.rb
@@ -23,6 +23,14 @@ module ActionController
ensure
ActiveModel::Serializer.config.adapter = old_adapter
end
+
+ def render_array_using_implicit_serializer
+ array = [
+ Profile.new({ name: 'Name 1', description: 'Description 1', comments: 'Comments 1' }),
+ Profile.new({ name: 'Name 2', description: 'Description 2', comments: 'Comments 2' })
+ ]
+ render json: array
+ end
end
tests MyController
@@ -48,6 +56,24 @@ module ActionController
assert_equal 'application/json', @response.content_type
assert_equal '{"profile":{"name":"Name 1","description":"Description 1"}}', @response.body
end
+
+ def test_render_array_using_implicit_serializer
+ get :render_array_using_implicit_serializer
+ assert_equal 'application/json', @response.content_type
+
+ expected = [
+ {
+ name: 'Name 1',
+ description: 'Description 1',
+ },
+ {
+ name: 'Name 2',
+ description: 'Description 2',
+ }
+ ]
+
+ assert_equal expected.to_json, @response.body
+ end
end
end
end
diff --git a/test/array_serializer_test.rb b/test/array_serializer_test.rb
index 66840a4..4c9b63e 100644
--- a/test/array_serializer_test.rb
+++ b/test/array_serializer_test.rb
@@ -5,7 +5,7 @@ module ActiveModel
class ArraySerializerTest < Minitest::Test
def setup
@comment = Comment.new
- @post= Post.new
+ @post = Post.new
@serializer = ArraySerializer.new([@comment, @post])
end
@@ -13,7 +13,7 @@ module ActiveModel
assert_respond_to @serializer, :each
end
- def test_each_object_should_be_serializer_with_appropriate_serializer
+ def test_each_object_should_be_serialized_with_appropriate_serializer
serializers = @serializer.to_a
assert_kind_of CommentSerializer, serializers.first
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment