Fix getSetMethodNormalizer to correctly ignore the attributes specifi…

…ed in IGNORED_ATTRIBUTES
symfony · Mar 6, 2019 · 0d28a1d · 0d28a1d
1 parent e74e0f9
commit 0d28a1d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/UPGRADE-4.2.md b/UPGRADE-4.2.md
@@ -179,11 +179,12 @@ 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 `AbstractNormalizer::IGNORED_ATTRIBUTES`.
 
 HttpFoundation
 --------------
 
- * The default value of the `$secure` and `$samesite` arguments of Cookie's constructor
+ * The default value of the `$secure` and `$samesite` arguments o`f Cookie's constructor
    will respectively change from `false` to `null` and from `null` to `lax` in Symfony
    5.0, you should define their values explicitly or use `Cookie::create()` instead.
 

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;
             }
         }