Merge pull request #45 from Nirhoshan/add_xml_support

Address inline schema for xml
ballerina-platform · Feb 7, 2024 · 937367e · 937367e
2 parents ba779f9 + 491600f
commit 937367e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/ballerina/Ballerina.toml b/ballerina/Ballerina.toml
@@ -2,7 +2,7 @@
 distribution = "2201.8.4"
 org = "ballerinax"
 name = "ai.agent"
-version = "0.7.4"
+version = "0.7.5"
 license = ["Apache-2.0"]
 authors = ["Ballerina"]
 keywords = ["AI/Agent", "Cost/Freemium"]

diff --git a/ballerina/openapi_utils.bal b/ballerina/openapi_utils.bal
@@ -213,7 +213,7 @@ class OpenApiSpecVisitor {
         return error UnsupportedMediaTypeError("Only json, xml or text content is supported.", availableContentTypes = content.keys());
     }
 
-    private isolated function visitRequestBody(RequestBody requestBody) returns RequestBodySchema|error {
+    private isolated function visitRequestBody(RequestBody requestBody) returns RequestBodySchema|OpenApiParsingError|error {
         map<MediaType> content = requestBody.content;
         string mediaType;
         Schema schema;
@@ -224,10 +224,16 @@ class OpenApiSpecVisitor {
                 schema: check self.visitSchema(schema)
             };
         }
-        if schema is Reference {
-            string refName = regexp:split(re `/`, schema.\$ref).pop();
-            schema = {'type: OBJECT, properties: {[refName] : schema}};
+        string? xmlName = schema.'xml?.name;
+        string outerBlockName;
+        if xmlName is string {
+            outerBlockName = xmlName;
+        } else if schema is Reference {
+            outerBlockName = regexp:split(re `/`, schema.\$ref).pop();
+        } else {
+            return error OpenApiParsingError("Error while parsing the OpenAPI specification. The schema should have a name for xml content type.", cause = schema);
         }
+        schema = {'type: OBJECT, properties: {[outerBlockName] : schema}};
         return {
             mediaType,
             schema: check self.visitSchema(schema, true)