ditkrg/active_model_serializers
4
0
Fork 0
mirror of https://github.com/ditkrg/active_model_serializers.git synced 2026-01-23 22:36:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix definition of serializer attributes with multiple calls to attribute instead of one single call to attributes.

Browse Source
This commit is contained in:
Lucas Hosseini 2015-08-28 21:06:10 +02:00
parent 64168cbecd
commit 995bbcc18d
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/active_model/serializer.rb
View File

@ -70,7 +70,7 @@ module ActiveModel
ActiveModelSerializers.silence_warnings do ActiveModelSerializers.silence_warnings do
define_method key do define_method key do
object.read_attribute_for_serialization(attr) object.read_attribute_for_serialization(attr)
end unless respond_to?(key, false) || _fragmented.respond_to?(attr) end unless (key != :id && method_defined?(key)) || _fragmented.respond_to?(attr)
end end
end end

3
test/fixtures/poro.rb vendored
View File

@ -134,7 +134,8 @@ end
AuthorSerializer = Class.new(ActiveModel::Serializer) do AuthorSerializer = Class.new(ActiveModel::Serializer) do
cache key:'writer', skip_digest: true cache key:'writer', skip_digest: true
attributes :id, :name attribute :id
attribute :name
has_many :posts, embed: :ids has_many :posts, embed: :ids
has_many :roles, embed: :ids has_many :roles, embed: :ids