[PropertyInfo] [Serializer] Fixed extracting ignored properties for Serializer

javer · javer · commit 776462c75412 · 2020-12-03T13:55:23.000+02:00
diff --git a/src/Symfony/Component/PropertyInfo/Extractor/SerializerExtractor.php b/src/Symfony/Component/PropertyInfo/Extractor/SerializerExtractor.php
@@ -47,7 +47,7 @@ public function getProperties(string $class, array $context = []): ?array
         $serializerClassMetadata = $this->classMetadataFactory->getMetadataFor($class);
 
         foreach ($serializerClassMetadata->getAttributesMetadata() as $serializerAttributeMetadata) {
-            $ignored = method_exists($serializerClassMetadata, 'isIgnored') && $serializerAttributeMetadata->isIgnored();
+            $ignored = method_exists($serializerAttributeMetadata, 'isIgnored') && $serializerAttributeMetadata->isIgnored();
             if (!$ignored && (null === $context['serializer_groups'] || array_intersect($context['serializer_groups'], $serializerAttributeMetadata->getGroups()))) {
                 $properties[] = $serializerAttributeMetadata->getName();
             }
diff --git a/src/Symfony/Component/PropertyInfo/Tests/Extractor/SerializerExtractorTest.php b/src/Symfony/Component/PropertyInfo/Tests/Extractor/SerializerExtractorTest.php
@@ -45,4 +45,9 @@ public function testGetPropertiesWithAnyGroup()
     {
         $this->assertSame(['analyses', 'feet'], $this->extractor->getProperties('Symfony\Component\PropertyInfo\Tests\Fixtures\AdderRemoverDummy', ['serializer_groups' => null]));
     }
+
+    public function testGetPropertiesWithIgnoredProperties()
+    {
+        $this->assertSame(['visibleProperty'], $this->extractor->getProperties('Symfony\Component\PropertyInfo\Tests\Fixtures\IgnorePropertyDummy', ['serializer_groups' => null]));
+    }
 }
diff --git a/src/Symfony/Component/PropertyInfo/Tests/Fixtures/IgnorePropertyDummy.php b/src/Symfony/Component/PropertyInfo/Tests/Fixtures/IgnorePropertyDummy.php
@@ -0,0 +1,27 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\PropertyInfo\Tests\Fixtures;
+
+use Symfony\Component\Serializer\Annotation\Ignore;
+
+/**
+ * @author Vadim Borodavko <vadim.borodavko@gmail.com>
+ */
+class IgnorePropertyDummy
+{
+    public $visibleProperty;
+
+    /**
+     * @Ignore
+     */
+    private $ignoredProperty;
+}

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ public function getProperties(string $class, array $context = []): ?array`
`47`	`47`	`$serializerClassMetadata = $this->classMetadataFactory->getMetadataFor($class);`
`48`	`48`
`49`	`49`	`foreach ($serializerClassMetadata->getAttributesMetadata() as $serializerAttributeMetadata) {`
`50`		`- $ignored = method_exists($serializerClassMetadata, 'isIgnored') && $serializerAttributeMetadata->isIgnored();`
	`50`	`+ $ignored = method_exists($serializerAttributeMetadata, 'isIgnored') && $serializerAttributeMetadata->isIgnored();`
`51`	`51`	`if (!$ignored && (null === $context['serializer_groups'] \|\| array_intersect($context['serializer_groups'], $serializerAttributeMetadata->getGroups()))) {`
`52`	`52`	`$properties[] = $serializerAttributeMetadata->getName();`
`53`	`53`	`}`
Original file line number	Diff line number	Diff line change
`@@ -45,4 +45,9 @@ public function testGetPropertiesWithAnyGroup()`
`45`	`45`	`{`
`46`	`46`	`$this->assertSame(['analyses', 'feet'], $this->extractor->getProperties('Symfony\Component\PropertyInfo\Tests\Fixtures\AdderRemoverDummy', ['serializer_groups' => null]));`
`47`	`47`	`}`
	`48`	`+`
	`49`	`+ public function testGetPropertiesWithIgnoredProperties()`
	`50`	`+ {`
	`51`	`+ $this->assertSame(['visibleProperty'], $this->extractor->getProperties('Symfony\Component\PropertyInfo\Tests\Fixtures\IgnorePropertyDummy', ['serializer_groups' => null]));`
	`52`	`+ }`
`48`	`53`	`}`