[Serializer] Support canners in object normalizer

Rokas Mikalkėnas · Rokas Mikalkėnas · commit 60ee9ed68f09 · 2022-02-11T11:54:31.000+02:00
diff --git a/src/Symfony/Component/Serializer/CHANGELOG.md b/src/Symfony/Component/Serializer/CHANGELOG.md
@@ -10,6 +10,7 @@ CHANGELOG
  * Deprecate `ContextAwareDenormalizerInterface`, use `DenormalizerInterface` instead
  * Deprecate `ContextAwareEncoderInterface`, use `EncoderInterface` instead
  * Deprecate `ContextAwareDecoderInterface`, use `DecoderInterface` instead
+ * Add support for `can*()` methods to `ObjectNormalizer`
 
 6.0
 ---
diff --git a/src/Symfony/Component/Serializer/Normalizer/ObjectNormalizer.php b/src/Symfony/Component/Serializer/Normalizer/ObjectNormalizer.php
@@ -86,8 +86,8 @@ protected function extractAttributes(object $object, string $format = null, arra
             $name = $reflMethod->name;
             $attributeName = null;
 
-            if (str_starts_with($name, 'get') || str_starts_with($name, 'has')) {
-                // getters and hassers
+            if (str_starts_with($name, 'get') || str_starts_with($name, 'has') || str_starts_with($name, 'can')) {
+                // getters, hassers and canners
                 $attributeName = substr($name, 3);
 
                 if (!$reflClass->hasProperty($attributeName)) {
diff --git a/src/Symfony/Component/Serializer/Tests/Normalizer/Features/ObjectDummy.php b/src/Symfony/Component/Serializer/Tests/Normalizer/Features/ObjectDummy.php
@@ -12,6 +12,7 @@ class ObjectDummy
     private $baz;
     protected $camelCase;
     protected $object;
+    private $go;
 
     public function getFoo()
     {
@@ -62,4 +63,14 @@ public function getObject()
     {
         return $this->object;
     }
+
+    public function setGo($go)
+    {
+        $this->go = $go;
+    }
+
+    public function canGo()
+    {
+        return $this->go;
+    }
 }
diff --git a/src/Symfony/Component/Serializer/Tests/Normalizer/ObjectNormalizerTest.php b/src/Symfony/Component/Serializer/Tests/Normalizer/ObjectNormalizerTest.php
@@ -107,6 +107,7 @@ public function testNormalize()
         $obj->setBaz(true);
         $obj->setCamelCase('camelcase');
         $obj->setObject($object);
+        $obj->setGo(true);
 
         $this->serializer
             ->expects($this->once())
@@ -123,6 +124,7 @@ public function testNormalize()
                 'fooBar' => 'foobar',
                 'camelCase' => 'camelcase',
                 'object' => 'string_object',
+                'go' => true,
             ],
             $this->normalizer->normalize($obj, 'any')
         );
@@ -664,6 +666,7 @@ public function testNormalizeNotSerializableContext()
             'camelCase' => null,
             'object' => null,
             'bar' => null,
+            'go' => null,
         ];
 
         $this->assertEquals($expected, $this->normalizer->normalize($objectDummy, null, ['not_serializable' => function () {
@@ -800,6 +803,7 @@ public function testDefaultObjectClassResolver()
         $obj->setBaz(true);
         $obj->setCamelCase('camelcase');
         $obj->unwantedProperty = 'notwanted';
+        $obj->setGo(false);
 
         $this->assertEquals(
             [
@@ -809,6 +813,7 @@ public function testDefaultObjectClassResolver()
                 'fooBar' => 'foobar',
                 'camelCase' => 'camelcase',
                 'object' => null,
+                'go' => false,
             ],
             $normalizer->normalize($obj, 'any')
         );
@@ -837,6 +842,7 @@ public function testObjectClassResolver()
                 'fooBar' => 'foobar',
                 'camelCase' => 'camelcase',
                 'object' => null,
+                'go' => null,
             ],
             $normalizer->normalize($obj, 'any')
         );

Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,7 @@ class ObjectDummy`
`12`	`12`	`private $baz;`
`13`	`13`	`protected $camelCase;`
`14`	`14`	`protected $object;`
	`15`	`+ private $go;`
`15`	`16`
`16`	`17`	`public function getFoo()`
`17`	`18`	`{`
`@@ -62,4 +63,14 @@ public function getObject()`
`62`	`63`	`{`
`63`	`64`	`return $this->object;`
`64`	`65`	`}`
	`66`	`+`
	`67`	`+ public function setGo($go)`
	`68`	`+ {`
	`69`	`+ $this->go = $go;`
	`70`	`+ }`
	`71`	`+`
	`72`	`+ public function canGo()`
	`73`	`+ {`
	`74`	`+ return $this->go;`
	`75`	`+ }`
`65`	`76`	`}`