fix

Author: Maxcastel

src/Hydra/Serializer/CollectionNormalizer.php

@@ -73,24 +73,11 @@ protected function getPaginationData(iterable $object, array $context = []): arr
         }
 
         if (null !== $this->resourceMetadataCollectionFactory && ($context['hydra_operations'] ?? $this->defaultContext['hydra_operations'] ?? false)) {
-            $allHydraOperations = [];
-            $operationNames = [];
-            foreach ($this->resourceMetadataCollectionFactory->create($resourceClass) as $resourceMetadata) {
-                $hydraOperations = $this->getHydraOperations(
-                    true,
-                    $resourceMetadata,
-                    $hydraPrefix
-                );
-                if (!empty($hydraOperations)) {
-                    foreach ($hydraOperations as $operation) {
-                        $operationName = $operation['method'];
-                        if (!\in_array($operationName, $operationNames, true)) {
-                            $operationNames[] = $operationName;
-                            $allHydraOperations[] = $operation;
-                        }
-                    }
-                }
-            }
+            $allHydraOperations = $this->getHydraOperationsFromResourceMetadatas(
+                $resourceClass,
+                true,
+                $hydraPrefix
+            );
 
             if (!empty($allHydraOperations)) {
                 $data[$hydraPrefix.'operation'] = $allHydraOperations;

src/Hydra/Serializer/HydraOperationsTrait.php

@@ -25,6 +25,53 @@
  */
 trait HydraOperationsTrait
 {
+    /**
+     * Gets Hydra operations from all resource metadata.
+     */
+    private function getHydraOperationsFromResourceMetadatas(string $resourceClass, bool $collection, string $hydraPrefix): array
+    {
+        $allHydraOperations = [];
+        $operationNames = [];
+
+        foreach ($this->resourceMetadataCollectionFactory->create($resourceClass) as $resourceMetadata) {
+            $hydraOperations = $this->getHydraOperationsFromResourceMetadata(
+                $collection,
+                $resourceMetadata,
+                $hydraPrefix,
+                $operationNames
+            );
+
+            $allHydraOperations = array_merge($allHydraOperations, $hydraOperations);
+        }
+
+        return $allHydraOperations;
+    }
+
+    /**
+     * Gets Hydra operations from a single resource metadata.
+     */
+    private function getHydraOperationsFromResourceMetadata(bool $collection, ApiResource $resourceMetadata, string $hydraPrefix, array &$operationNames): array
+    {
+        $operations = [];
+        $hydraOperations = $this->getHydraOperations(
+            $collection,
+            $resourceMetadata,
+            $hydraPrefix
+        );
+
+        if (!empty($hydraOperations)) {
+            foreach ($hydraOperations as $operation) {
+                $operationName = $operation['method'];
+                if (!\in_array($operationName, $operationNames, true)) {
+                    $operationNames[] = $operationName;
+                    $operations[] = $operation;
+                }
+            }
+        }
+
+        return $operations;
+    }
+
     /**
      * Gets Hydra operations.
      */

src/JsonLd/Serializer/ItemNormalizer.php

@@ -191,24 +191,11 @@ public function normalize(mixed $data, ?string $format = null, array $context =
         }
 
         if ($isResourceClass && ($context['hydra_operations'] ?? $this->itemNormalizerDefaultContext['hydra_operations'] ?? false)) {
-            $allHydraOperations = [];
-            $operationNames = [];
-            foreach ($this->resourceMetadataCollectionFactory->create($resourceClass) as $resourceMetadata) {
-                $hydraOperations = $this->getHydraOperations(
-                    false,
-                    $resourceMetadata,
-                    $this->getHydraPrefix($context + $this->itemNormalizerDefaultContext)
-                );
-                if (!empty($hydraOperations)) {
-                    foreach ($hydraOperations as $operation) {
-                        $operationName = $operation['method'];
-                        if (!\in_array($operationName, $operationNames, true)) {
-                            $operationNames[] = $operationName;
-                            $allHydraOperations[] = $operation;
-                        }
-                    }
-                }
-            }
+            $allHydraOperations = $this->getHydraOperationsFromResourceMetadatas(
+                $resourceClass,
+                false,
+                $this->getHydraPrefix($context + $this->itemNormalizerDefaultContext)
+            );
 
             if (!empty($allHydraOperations)) {
                 $metadata['operation'] = $allHydraOperations;