bugfix: required properties are now listed correctly

Author: eceltov

app/helpers/Swagger/AnnotationData.php

@@ -13,8 +13,17 @@ class AnnotationData
 
     public string $className;
     public string $methodName;
+    /**
+     * @var AnnotationParameterData[]
+     */
     public array $pathParams;
+    /**
+     * @var AnnotationParameterData[]
+     */
     public array $queryParams;
+    /**
+     * @var AnnotationParameterData[]
+     */
     public array $bodyParams;
     public ?string $endpointDescription;
 
@@ -68,9 +77,26 @@ private function getBodyAnnotation(): string | null
         // only json is supported due to the media type
         $head = '@OA\RequestBody(@OA\MediaType(mediaType="application/json",@OA\Schema';
         $body = new ParenthesesBuilder();
+        // list of all required properties
+        $required = [];
 
         foreach ($this->bodyParams as $bodyParam) {
             $body->addValue($bodyParam->toPropertyAnnotation());
+            if ($bodyParam->required) {
+                $required[] = $bodyParam->name;
+            }
+        }
+
+        // add a list of required properties
+        if (count($required) > 0) {
+            // stringify the list (it has to be in '{"name1","name1",...}' format)
+            $requiredString = "{";
+            foreach ($required as $name) {
+                $requiredString .= "\"$name\",";
+            }
+            $requiredString .= "}";
+    
+            $body->addValue("required=" . $requiredString);
         }
 
         return $head . $body->toString() . "))";

app/helpers/Swagger/AnnotationParameterData.php

@@ -128,7 +128,6 @@ public function toPropertyAnnotation(): string
         $body->addKeyValue("property", $this->name);
         $body->addKeyValue("type", $this->swaggerType);
         $body->addKeyValue("nullable", $this->nullable);
-        $body->addKeyValue("required", $this->required);
 
         if ($this->description !== null) {
             $body->addKeyValue("description", $this->description);