temp

api-platform · Oct 7, 2024 · b173e20 · b173e20
1 parent 78cdfe0
commit b173e20
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/src/Metadata/ApiProperty.php b/src/Metadata/ApiProperty.php
@@ -47,7 +47,7 @@ final class ApiProperty
      * @param Type[]                                                                   $builtinTypes
      * @param string|null                                                              $uriTemplate             (experimental) whether to return the subRessource collection IRI instead of an iterable of IRI
      * @param string|null                                                              $property                The property name
-     * @param array<int, Groups|SerializedName|SerializedPath|MaxDepth|Ignore|Context> $serialize
+     * @param array<int, Context|Groups|Ignore|SerializedName|SerializedPath|MaxDepth> $serialize
      */
     public function __construct(
         private ?string $description = null,
@@ -615,7 +615,7 @@ public function getSerialize(): ?array
     }
 
     /**
-     * @param array<int, Groups|SerializedName|SerializedPath|MaxDepth|Ignore|Context> $serialize
+     * @param array<int, Context|Groups|Ignore|SerializedName|SerializedPath|MaxDepth> $serialize
      */
     public function withSerialize(array $serialize): static
     {

diff --git a/src/Metadata/Tests/Extractor/Adapter/XmlPropertyAdapter.php b/src/Metadata/Tests/Extractor/Adapter/XmlPropertyAdapter.php
@@ -47,7 +47,7 @@ final class XmlPropertyAdapter implements PropertyAdapterInterface
         'property',
     ];
 
-    # Laravel only, we don't need xml support
+    // Laravel only, we don't need XML support
     private const EXCLUDE = ['policy', 'serialize'];
 
     /**

diff --git a/src/Serializer/Mapping/Loader/PropertyMetadataLoader.php b/src/Serializer/Mapping/Loader/PropertyMetadataLoader.php
@@ -46,7 +46,7 @@ public function loadClassMetadata(ClassMetadataInterface $classMetadata): bool
         $classGroups = [];
         $classContextAnnotation = null;
 
-        // It's very weird to grab an eloquent's properties in that case as they're never serialized
+        // It's very weird to grab Eloquent's properties in that case as they're never serialized
         // the Serializer makes a call on the abstract class, let's save some unneeded work with a condition
         if (Model::class === $classMetadata->getName()) {
             return false;
@@ -109,6 +109,7 @@ public function loadClassMetadata(ClassMetadataInterface $classMetadata): bool
                 continue;
             }
 
+            // This code is adapted from Symfony\Component\Serializer\Mapping\Loader\AttributeLoader
             foreach ($attributes[$propertyName] as $attr) {
                 if ($attr instanceof Groups) {
                     foreach ($attr->getGroups() as $group) {