do not overwrite the cache key when it is false

Author: xabbuh

Normalizer/AbstractObjectNormalizer.php

@@ -783,7 +783,7 @@ protected function createChildContext(array $parentContext, string $attribute, ?
         $context = parent::createChildContext($parentContext, $attribute, $format);
         if ($context['cache_key'] ?? false) {
             $context['cache_key'] .= '-'.$attribute;
-        } else {
+        } elseif (false !== ($context['cache_key'] ?? null)) {
             $context['cache_key'] = $this->getCacheKey($format, $context);
         }
 

Tests/Normalizer/AbstractObjectNormalizerTest.php

@@ -555,11 +555,6 @@ public function testProvidingContextCacheKeyGeneratesSameChildContextCacheKey()
         $foobar->foo = new EmptyDummy();
         $foobar->bar = 'bar';
         $foobar->baz = 'baz';
-        $data = [
-            'foo' => [],
-            'bar' => 'bar',
-            'baz' => 'baz',
-        ];
 
         $normalizer = new class() extends AbstractObjectNormalizerDummy {
             public $childContextCacheKey;
@@ -600,11 +595,6 @@ public function testChildContextKeepsOriginalContextCacheKey()
         $foobar->foo = new EmptyDummy();
         $foobar->bar = 'bar';
         $foobar->baz = 'baz';
-        $data = [
-            'foo' => [],
-            'bar' => 'bar',
-            'baz' => 'baz',
-        ];
 
         $normalizer = new class() extends AbstractObjectNormalizerDummy {
             public $childContextCacheKey;
@@ -640,11 +630,6 @@ public function testChildContextCacheKeyStaysFalseWhenOriginalCacheKeyIsFalse()
         $foobar->foo = new EmptyDummy();
         $foobar->bar = 'bar';
         $foobar->baz = 'baz';
-        $data = [
-            'foo' => [],
-            'bar' => 'bar',
-            'baz' => 'baz',
-        ];
 
         $normalizer = new class() extends AbstractObjectNormalizerDummy {
             public $childContextCacheKey;