Fix method parameter type mapping (#1775)

Author: krissss

src/Processors/AugmentParameters.php

@@ -92,13 +92,15 @@ protected function augmentOperationParameters(Analysis $analysis): void
                                     $parameter->description = $details['description'];
                                 }
                             }
-
-                            if (!Generator::isDefault($parameter->schema)) {
-                                $this->mapNativeType($parameter->schema, $parameter->schema->type);
-                            }
                         }
                     }
                 }
+
+                foreach ($operation->parameters as $parameter) {
+                    if (!Generator::isDefault($parameter->schema)) {
+                        $this->mapNativeType($parameter->schema, $parameter->schema->type);
+                    }
+                }
             }
         }
     }

tests/Fixtures/RequestUsingAttribute.php

@@ -0,0 +1,21 @@
+<?php declare(strict_types=1);
+
+/**
+ * @license Apache 2.0
+ */
+
+namespace OpenApi\Tests\Fixtures;
+
+use OpenApi\Attributes as OAT;
+
+class RequestUsingAttribute
+{
+    #[OAT\Get(
+        path: '/get/{id}',
+        summary: 'get desc'
+    )]
+    public function get(#[OAT\PathParameter] int $id)
+    {
+
+    }
+}

tests/Processors/AugmentParametersTest.php

@@ -6,8 +6,15 @@
 
 namespace OpenApi\Tests\Processors;
 
+use OpenApi\Annotations\Operation;
+use OpenApi\Annotations\Parameter;
+use OpenApi\Annotations\PathItem;
 use OpenApi\Generator;
+use OpenApi\Processors\AugmentParameters;
+use OpenApi\Processors\BuildPaths;
 use OpenApi\Processors\Concerns\DocblockTrait;
+use OpenApi\Processors\MergeIntoComponents;
+use OpenApi\Processors\MergeIntoOpenApi;
 use OpenApi\Tests\OpenApiTestCase;
 
 class AugmentParametersTest extends OpenApiTestCase
@@ -56,4 +63,40 @@ public function testExtractTags(string $params, array $expected): void
         $this->extractContent($mixed->comment, $tags);
         $this->assertEquals($expected, $tags);
     }
+
+    /**
+     * @requires PHP 8.1
+     */
+    public function testParameterNativeType(): void
+    {
+        $analysis = $this->analysisFromFixtures(['RequestUsingAttribute.php']);
+        $analysis->process([
+            new MergeIntoOpenApi(),
+            new MergeIntoComponents(),
+            new BuildPaths(),
+            new AugmentParameters(),
+        ]);
+
+        $findPathItemByPath = function (string $path) use ($analysis): PathItem {
+            foreach ($analysis->openapi->paths as $pathItem) {
+                if ($pathItem->path === $path) {
+                    return $pathItem;
+                }
+            }
+            throw new \InvalidArgumentException('Not found');
+        };
+        $findParameterByName = function (string $name, Operation $operation): Parameter {
+            foreach ($operation->parameters as $parameter) {
+                if ($parameter->name === $name) {
+                    return $parameter;
+                }
+            }
+            throw new \InvalidArgumentException('Not found');
+        };
+
+        $pathItem = $findPathItemByPath('/get/{id}');
+        $parameter = $findParameterByName('id', $pathItem->get);
+
+        $this->assertEquals('integer', $parameter->schema->type);
+    }
 }