diff --git a/lib/active_model/serializer.rb b/lib/active_model/serializer.rb index 030e47d2..61343378 100644 --- a/lib/active_model/serializer.rb +++ b/lib/active_model/serializer.rb @@ -62,7 +62,7 @@ module ActiveModel ActiveModelSerializers.silence_warnings do define_method key do object.read_attribute_for_serialization(attr) - end unless (key != :id && method_defined?(key)) || _fragmented.respond_to?(attr) + end unless method_defined?(key) || _fragmented.respond_to?(attr) end end diff --git a/test/serializers/attribute_test.rb b/test/serializers/attribute_test.rb index 5ca7f842..99452e53 100644 --- a/test/serializers/attribute_test.rb +++ b/test/serializers/attribute_test.rb @@ -57,6 +57,20 @@ module ActiveModel adapter = ActiveModel::Serializer::Adapter::Json.new(attributes_serializer.new(@blog)) assert_equal({ blog: { type: 'stuff' } }, adapter.serializable_hash) end + + def test_id_attribute_override_before + serializer = Class.new(ActiveModel::Serializer) do + def id + 'custom' + end + + attribute :id + end + + hash = ActiveModel::SerializableResource.new(@blog, adapter: :json, serializer: serializer).serializable_hash + + assert_equal('custom', hash[:blog][:id]) + end end end end