Fix phpstan errors because of changes in Symfony

Author: teohhanhui

phpstan.neon.dist

@@ -21,7 +21,7 @@ parameters:
 		# Real problems, hard to fix
 		- '#Parameter \#2 \$dqlPart of method Doctrine\\ORM\\QueryBuilder::add\(\) expects array\|object, string given\.#'
 		-
-			message: '#Return type \(int\) of method ApiPlatform\\Core\\Identifier\\Normalizer\\IntegerDenormalizer::denormalize\(\) should be compatible with return type \(object\) of method Symfony\\Component\\Serializer\\Normalizer\\DenormalizerInterface::denormalize\(\)#'
+			message: '#Return type \(int\) of method ApiPlatform\\Core\\Identifier\\Normalizer\\IntegerDenormalizer::denormalize\(\) should be compatible with return type \(array\|object\) of method Symfony\\Component\\Serializer\\Normalizer\\DenormalizerInterface::denormalize\(\)#'
 			path: %currentWorkingDirectory%/src/Identifier/Normalizer/IntegerDenormalizer.php
 
 		# False positives

src/Bridge/Elasticsearch/DataProvider/ItemDataProvider.php

@@ -101,6 +101,11 @@ public function getItem(string $resourceClass, $id, ?string $operationName = nul
             return null;
         }
 
-        return $this->denormalizer->denormalize($document, $resourceClass, ItemNormalizer::FORMAT, [AbstractNormalizer::ALLOW_EXTRA_ATTRIBUTES => true]);
+        $item = $this->denormalizer->denormalize($document, $resourceClass, ItemNormalizer::FORMAT, [AbstractNormalizer::ALLOW_EXTRA_ATTRIBUTES => true]);
+        if (!\is_object($item) && null !== $item) {
+            throw new \UnexpectedValueException('Expected item to be an object or null.');
+        }
+
+        return $item;
     }
 }

src/GraphQl/Resolver/Factory/ItemMutationResolverFactory.php

@@ -123,6 +123,9 @@ public function __invoke(string $resourceClass = null, string $rootClass = null,
 
                 $context += $resourceMetadata->getGraphqlAttribute($operationName, 'denormalization_context', [], true);
                 $item = $this->normalizer->denormalize($args['input'], $inputClass ?: $resourceClass, ItemNormalizer::FORMAT, $context);
+                if (!\is_object($item)) {
+                    throw new \UnexpectedValueException('Expected item to be an object.');
+                }
                 $this->canAccess($this->resourceAccessChecker, $resourceMetadata, $resourceClass, $info, [
                     'object' => $item,
                     'previous_object' => $previousItem,

src/HttpCache/EventListener/AddHeadersListener.php

@@ -63,7 +63,7 @@ public function onKernelResponse(FilterResponseEvent $event): void
         }
 
         if ($this->etag && !$response->getEtag()) {
-            $response->setEtag(md5($response->getContent()));
+            $response->setEtag(md5((string) $response->getContent()));
         }
 
         if (null !== ($maxAge = $resourceCacheHeaders['max_age'] ?? $this->maxAge) && !$response->headers->hasCacheControlDirective('max-age')) {

src/Serializer/AbstractItemNormalizer.php

@@ -183,6 +183,9 @@ public function denormalize($data, $class, $format = null, array $context = [])
                 throw new LogicException('Cannot denormalize the input because the injected serializer is not a denormalizer');
             }
             $denormalizedInput = $this->serializer->denormalize($data, $inputClass, $format, $context);
+            if (!\is_object($denormalizedInput)) {
+                throw new \UnexpectedValueException('Expected denormalized input to be an object.');
+            }
 
             return $dataTransformer->transform($denormalizedInput, $resourceClass, $dataTransformerContext);
         }
@@ -437,7 +440,12 @@ protected function denormalizeRelation(string $attributeName, PropertyMetadata $
             }
 
             try {
-                return $this->serializer->denormalize($value, $className, $format, $context);
+                $item = $this->serializer->denormalize($value, $className, $format, $context);
+                if (!\is_object($item) && null !== $item) {
+                    throw new \UnexpectedValueException('Expected item to be an object or null.');
+                }
+
+                return $item;
             } catch (InvalidValueException $e) {
                 if (!$supportsPlainIdentifiers) {
                     throw $e;