[Serializer] Document Normalizers #10515
Conversation
0ec283d to
e13e9e1
Compare
| .. index:: | ||
| single: Serializer; Custom normalizers | ||
|
|
||
| How to Create your Custom Normalizer |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
There was a problem hiding this comment.
I copied the schema of https://symfony.com/doc/current/serializer/custom_encoders.html ; So are you sure I should be not consistant with existing documentation?
There was a problem hiding this comment.
Fair enough, lets keep it then 👍
serializer/custom_normalizer.rst
Outdated
| private $router; | ||
| private $normaliser; | ||
|
|
||
| public function __construct(UrlGeneratorInterface $router, ObjectNormalizer $normaliser) |
There was a problem hiding this comment.
s/$normaliser/$normalizer/
serializer/custom_normalizer.rst
Outdated
|
|
||
| namespace AppBundle\Serializer; | ||
|
|
||
| use App\Entity\Topic; |
There was a problem hiding this comment.
you use App here, but 2 lines above you are in AppBundle namespace, maybe a mistake?
There was a problem hiding this comment.
yes, good catch :) thanks
e13e9e1 to
0837b4f
Compare
0837b4f to
aa4a747
Compare
| } | ||
| } | ||
|
|
||
| Registering it in your app |
There was a problem hiding this comment.
A question for when we merge this into upper branches. Does Symfony autoconfiguration support Normalizers? Thanks!
There was a problem hiding this comment.
Yes it does ❤️
I already planned to open a new PR to remove theses for new Symfony versions
|
Thanks Grégoire! This has been merged now. |
|
The change it's now merged in all branches. Next steps would be:
Thanks! |
refs #10505