Postponed
Project:
JSON:API
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
5 Feb 2019 at 19:46 UTC
Updated:
13 Feb 2019 at 11:16 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
effulgentsia commentedComment #3
gabesulliceThis patch makes use of the existing behavior: #3024896-50: Remove Relationship and RelationshipItem classes
See line 79 of the green additions under
EntityReferenceFieldNormalizer.I realize now that I didn't quite understand what you were doing in 9.3 of #3029897-9: [PP-1] Do not overload Symfony's normalize() method to return objects; define a new interface for that. but I think I "get it" now.
In #15 of #3029897-15: [PP-1] Do not overload Symfony's normalize() method to return objects; define a new interface for that., you said that this issue could:
Could we try the latter once #3029897 lands? I'd be happy to write that patch.
IMO, the less differently our serializer behaves relative to the core serializer the better. I'm still dreaming of the day that we can just drop all our custom normalizer stuff because it's no longer necessary.
Comment #4
gabesulliceComment #5
wim leersPer #3, waiting for #3029897: [PP-1] Do not overload Symfony's normalize() method to return objects; define a new interface for that..
Comment #6
wim leersNothing is broken, so I don't think this is a bug?