ditkrg/active_model_serializers
5
0
Fork 0
mirror of https://github.com/ditkrg/active_model_serializers.git synced 2026-01-25 07:16:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

There's no need for target_serializer method

Browse Source
This commit is contained in:
Santiago Pastorino
2013-05-15 16:33:12 -07:00
parent feaefeeef3
commit 1c3f14407c
1 changed files with 6 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
lib/active_model/serializer/associations.rb
View File

@@ -10,6 +10,9 @@ module ActiveModel
@embed_key = options[:embed_key] || :id @embed_key = options[:embed_key] || :id
@embed_in_root = options[:include] @embed_in_root = options[:include]
serializer = options[:serializer]
@serializer = serializer.is_a?(String) ? serializer.constantize : serializer
@options = options @options = options
@serializer_options = serializer_options @serializer_options = serializer_options
end end
@@ -42,16 +45,11 @@ module ActiveModel
private private
attr_reader :object, :embed, :embed_key attr_reader :object, :embed, :embed_key, :serializer
def target_serializer
serializer = options[:serializer]
serializer.is_a?(String) ? serializer.constantize : serializer
end
def find_serializable(object) def find_serializable(object)
if target_serializer if serializer
target_serializer.new(object, serializer_options) serializer.new(object, serializer_options)
elsif object.respond_to?(:active_model_serializer) && (ams = object.active_model_serializer) elsif object.respond_to?(:active_model_serializer) && (ams = object.active_model_serializer)
ams.new(object, serializer_options) ams.new(object, serializer_options)
else else