8000 feature #12295 [FrameworkBundle] make GetSetMethodNormalizer availabl… · symfony/symfony@f47ade6 · GitHub
[go: up one dir, main page]

Skip to content

Commit f47ade6

Browse files
committed
feature #12295 [FrameworkBundle] make GetSetMethodNormalizer available by default (dunglas)
This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes #12295). Discussion ---------- [FrameworkBundle] make GetSetMethodNormalizer available by default | Q | A | ------------- | --- | Bug fix? | no | New feature? | yes | BC breaks? | very limited | Deprecations? | no | Tests pass? | yes | License | MIT | Doc PR | not yet The `GetSetMethodNormalizer` has not been enabled by default in #6815 because it was broken by design. Since #12098 has been merged, circular references are handled and that normalizer cannot breaks applications. This PR makes that normalizer automatically available when the serializer is enabled. To keep BC, I've set a "high" priority for this service (higher than the default `0`). Commits ------- 4f0aa61 [FrameworkBundle] make GetSetMethodNormalizer available by default
2 parents 480819f + 4f0aa61 commit f47ade6

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

src/Symfony/Bundle/FrameworkBundle/Resources/config/serializer.xml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,13 @@
1515
<argument type="collection" />
1616
<argument type="collection" />
1717
</service>
18+
19+
<!-- Normalizer -->
20+
<service id="serializer.normalizer.get_set_method" class="Symfony\Component\Serializer\Normalizer\GetSetMethodNormalizer" public="false">
21+
<!-- Run after all custom serializers -->
22+
<tag name="serializer.normalizer" priority="-1000" />
23+
</service>
24+
1825
<!-- Encoders -->
1926
<service id="serializer.encoder.xml" class="%serializer.encoder.xml.class%" public="false" >
2027
<tag name="serializer.encoder" />

0 commit comments

Comments
 (0)
0