Skip to content

Commit 8092f2d

Browse files
committed
Merge pull request #732 from togglepro/serialize-polymorphic-has-one-include
serialize included polymorphic has one relationship correctly
2 parents b9ad711 + b3ac924 commit 8092f2d

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

lib/jsonapi/resource_serializer.rb

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,8 @@ def relationships_hash(source, include_directives)
219219
resources.each do |resource|
220220
next if self_referential_and_already_in_source(resource)
221221
id = resource.id
222-
relationships_only = already_serialized?(relationship.type, id)
222+
type = resource.class.resource_for_model(resource._model)
223+
relationships_only = already_serialized?(type, id)
223224
if include_linkage && !relationships_only
224225
add_included_object(id, object_hash(resource, ia))
225226
elsif include_linked_children || relationships_only

0 commit comments

Comments
 (0)