Fix getSetMethodNormalizer to correctly ignore the attributes specified in "ignored_attributes"

Emmanuel BORGES · Emmanuel BORGES · commit 24f77a01be5a · 2019-03-06T14:00:04.000+01:00
diff --git a/UPGRADE-4.2.md b/UPGRADE-4.2.md
@@ -179,6 +179,7 @@ FrameworkBundle
  * The `Templating\Helper\TranslatorHelper::transChoice()` method has been deprecated, use the `trans()` one instead with a `%count%` parameter.
  * Deprecated support for legacy translations directories `src/Resources/translations/` and `src/Resources/<BundleName>/translations/`, use `translations/` instead.
  * Support for the legacy directory structure in `translation:update` and `debug:translation` commands has been deprecated.
+ * The `GetSetMethodNormalizer` class correctly ignores the attributes specified in "ignored_attributes".
 
 HttpFoundation
 --------------
diff --git a/src/Symfony/Component/Serializer/Normalizer/GetSetMethodNormalizer.php b/src/Symfony/Component/Serializer/Normalizer/GetSetMethodNormalizer.php
@@ -110,7 +110,7 @@ protected function extractAttributes($object, $format = null, array $context = [
 
             $attributeName = lcfirst(substr($method->name, 0 === strpos($method->name, 'is') ? 2 : 3));
 
-            if ($this->isAllowedAttribute($object, $attributeName)) {
+            if ($this->isAllowedAttribute($object, $attributeName, null, $context)) {
                 $attributes[] = $attributeName;
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,7 @@ protected function extractAttributes($object, $format = null, array $context = [`
`110`	`110`
`111`	`111`	`$attributeName = lcfirst(substr($method->name, 0 === strpos($method->name, 'is') ? 2 : 3));`
`112`	`112`
`113`		`- if ($this->isAllowedAttribute($object, $attributeName)) {`
	`113`	`+ if ($this->isAllowedAttribute($object, $attributeName, null, $context)) {`
`114`	`114`	`$attributes[] = $attributeName;`
`115`	`115`	`}`
`116`	`116`	`}`