Take care of reserved keywoards and empty webhook event classes

Author: WyriHaximus

src/Generator.php

@@ -37,7 +37,7 @@ public function generate(string $namespace, string $destinationPath)
 
     private function className(string $className): string
     {
-        return str_replace(['{', '}', '-', '$', '_'], ['Cb', 'Rcb', 'Dash', '_', '\\'], (new Convert($className))->toPascal());
+        return str_replace(['{', '}', '-', '$', '_'], ['Cb', 'Rcb', 'Dash', '_', '\\'], (new Convert($className))->toPascal()) . ($this->isKeyword($className) ? '_' : '');
     }
 
     private function cleanUpNamespace(string $namespace): string
@@ -164,4 +164,9 @@ private function basename(string $fqcn): string
 
         return $this->cleanUpNamespace(basename($fqcn));
     }
+
+    private function isKeyword(string $name): bool
+    {
+        return in_array($name, array('__halt_compiler', 'abstract', 'and', 'array', 'as', 'break', 'callable', 'case', 'catch', 'class', 'clone', 'const', 'continue', 'declare', 'default', 'die', 'do', 'echo', 'else', 'elseif', 'empty', 'enddeclare', 'endfor', 'endforeach', 'endif', 'endswitch', 'endwhile', 'eval', 'exit', 'extends', 'final', 'for', 'foreach', 'function', 'global', 'goto', 'if', 'implements', 'include', 'include_once', 'instanceof', 'insteadof', 'interface', 'isset', 'list', 'namespace', 'new', 'or', 'print', 'private', 'protected', 'public', 'require', 'require_once', 'return', 'static', 'switch', 'throw', 'trait', 'try', 'unset', 'use', 'var', 'while', 'xor'), false);
+    }
 }

src/Generator/WebHook.php

@@ -38,21 +38,22 @@ public static function generate(string $path, string $namespace, string $baseNam
         );
         if ($pathItem->post->requestBody->content !== null) {
             $content = current($pathItem->post->requestBody->content);
+            $tmts = [];
             if ($content->schema->oneOf !== null && count($content->schema->oneOf) > 0) {
-                $method->addStmt(new Node\Expr\Assign(new Node\Expr\Variable('schemaValidator'), new Node\Expr\New_(
+                $tmts[] = new Node\Expr\Assign(new Node\Expr\Variable('schemaValidator'), new Node\Expr\New_(
                     new Node\Name('\League\OpenAPIValidation\Schema\SchemaValidator'),
                     [
                         new Node\Arg(new Node\Expr\ClassConstFetch(
                             new Node\Name('\League\OpenAPIValidation\Schema\SchemaValidator'),
                             new Node\Name('VALIDATE_AS_REQUEST'),
                         ))
                     ]
-                )));
+                ));
                 $gotoLabels = 'a';
                 foreach ($content->schema->oneOf as $oneOfSchema) {
-                    $method->addStmt(new Node\Stmt\Label($gotoLabels));
+                    $tmts[] = new Node\Stmt\Label($gotoLabels);
                     $gotoLabels++;
-                    $method->addStmt(new Node\Stmt\TryCatch([
+                    $tmts[] = new Node\Stmt\TryCatch([
                         new Node\Stmt\Expression(new Node\Expr\MethodCall(
                             new Node\Expr\Variable('schemaValidator'),
                             new Node\Name('validate'),
@@ -70,11 +71,17 @@ public static function generate(string $path, string $namespace, string $baseNam
                                 new Node\Stmt\Goto_($gotoLabels),
                             ]
                         ),
-                    ]));
+                    ]);
                 }
-                $method->addStmt(new Node\Stmt\Label($gotoLabels));
-                $method->addStmt(new Node\Stmt\Throw_(new Node\Expr\Variable($gotoLabels)));
+                $tmts[] = new Node\Stmt\Label($gotoLabels);
+                $tmts[] = new Node\Stmt\Throw_(new Node\Expr\Variable($gotoLabels));
             }
+
+            if (count($tmts) === 0) {
+                $tmts[] = new Node\Stmt\Return_(new Node\Scalar\String_('TODO: Implement this'));
+            }
+
+            $method->addStmts($tmts);
         }
         $class->addStmt($method);