Merge pull request #21 from sitegeist/bugfix/documentFactoryIgnoresUn…

…knownHttpMethods BUGFIX: DocumentFactory will ignore unknown http methods instead of throwing errors
sitegeist · Jun 19, 2024 · cafa8fb · cafa8fb
2 parents 68df7c8 + 4c87ff3
commit cafa8fb
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/Classes/Domain/OpenApiDocumentFactory.php b/Classes/Domain/OpenApiDocumentFactory.php
@@ -222,13 +222,16 @@ private function createPathsFromPathAndMethodReflection(ClassReflection $classRe
                     $route->getUriPattern()
                 );
                 foreach ($route->getHttpMethods() as $httpMethod) {
-                    $paths[] = new OpenApiPathItem(
-                        new PathDefinition('/' . $path),
-                        HttpMethod::from(strtolower($httpMethod)),
-                        new OpenApiParameterCollection(...$parameters),
-                        $requestBody,
-                        OpenApiResponses::fromReflectionMethod($methodReflection)
-                    );
+                    $httpMethod = HttpMethod::tryFrom(strtolower($httpMethod));
+                    if ($httpMethod instanceof HttpMethod) {
+                        $paths[] = new OpenApiPathItem(
+                            new PathDefinition('/' . $path),
+                            $httpMethod,
+                            new OpenApiParameterCollection(...$parameters),
+                            $requestBody,
+                            OpenApiResponses::fromReflectionMethod($methodReflection)
+                        );
+                    }
                 }
                 // only the first matching route is needed so we can break the loop here
                 break;