From 66b347508d0fae142fd173fc74fd820d30b0ea46 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Primo=C5=BE=20Delopst?= <primoz.delopst@better.care>
Date: Mon, 7 Oct 2024 17:53:48 +0200
Subject: [PATCH] Add missing search using POST HTTP method to OpenAPI
 specification

---
 .../fhir/rest/openapi/OpenApiInterceptor.java | 52 +++++++++++++++----
 .../ca/uhn/fhir/rest/openapi/index.html       | 17 ++++++
 2 files changed, 58 insertions(+), 11 deletions(-)

diff --git a/hapi-fhir-server-openapi/src/main/java/ca/uhn/fhir/rest/openapi/OpenApiInterceptor.java b/hapi-fhir-server-openapi/src/main/java/ca/uhn/fhir/rest/openapi/OpenApiInterceptor.java
index a495e074b92..9510e555667 100644
--- a/hapi-fhir-server-openapi/src/main/java/ca/uhn/fhir/rest/openapi/OpenApiInterceptor.java
+++ b/hapi-fhir-server-openapi/src/main/java/ca/uhn/fhir/rest/openapi/OpenApiInterceptor.java
@@ -110,6 +110,7 @@ import java.io.InputStream;
 import java.math.BigDecimal;
 import java.nio.charset.StandardCharsets;
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.LinkedHashMap;
@@ -619,13 +620,22 @@ public class OpenApiInterceptor {
 						getPathItem(paths, "/" + resourceType, PathItem.HttpMethod.GET),
 						ctx,
 						resourceType,
-						nextResource);
+						nextResource,
+					    PathItem.HttpMethod.GET);
 				addSearchOperation(
 						openApi,
 						getPathItem(paths, "/" + resourceType + "/_search", PathItem.HttpMethod.GET),
 						ctx,
 						resourceType,
-						nextResource);
+						nextResource,
+						PathItem.HttpMethod.GET);
+				addSearchOperation(
+					openApi,
+					getPathItem(paths, "/" + resourceType + "/_search", PathItem.HttpMethod.POST),
+					ctx,
+					resourceType,
+					nextResource,
+					PathItem.HttpMethod.POST);
 			}
 
 			// Resource-level Operations
@@ -674,22 +684,42 @@ public class OpenApiInterceptor {
 			final Operation operation,
 			final FhirContext ctx,
 			final String resourceType,
-			final CapabilityStatement.CapabilityStatementRestResourceComponent nextResource) {
+			final CapabilityStatement.CapabilityStatementRestResourceComponent nextResource,
+			final PathItem.HttpMethod httpMethod) {
 		operation.addTagsItem(resourceType);
 		operation.setDescription("This is a search type");
 		operation.setSummary("search-type: Search for " + resourceType + " instances");
 		addFhirResourceResponse(ctx, openApi, operation, null);
 
-		for (final CapabilityStatement.CapabilityStatementRestResourceSearchParamComponent nextSearchParam :
+		if (httpMethod == PathItem.HttpMethod.GET) {
+			for (final CapabilityStatement.CapabilityStatementRestResourceSearchParamComponent nextSearchParam :
 				nextResource.getSearchParam()) {
-			final Parameter parametersItem = new Parameter();
-			operation.addParametersItem(parametersItem);
+				final Parameter parametersItem = new Parameter();
+				operation.addParametersItem(parametersItem);
 
-			parametersItem.setName(nextSearchParam.getName());
-			parametersItem.setRequired(false);
-			parametersItem.setIn("query");
-			parametersItem.setDescription(nextSearchParam.getDocumentation());
-			parametersItem.setSchema(toSchema(nextSearchParam.getType()));
+				parametersItem.setName(nextSearchParam.getName());
+				parametersItem.setRequired(false);
+				parametersItem.setIn("query");
+				parametersItem.setDescription(nextSearchParam.getDocumentation());
+				parametersItem.setSchema(toSchema(nextSearchParam.getType()));
+			}
+		} else {
+			Schema<?> objectSchema = new Schema<>().type("object");
+			for (final CapabilityStatement.CapabilityStatementRestResourceSearchParamComponent nextSearchParam :
+				nextResource.getSearchParam()) {
+				objectSchema.addProperty(
+					nextSearchParam.getName(),
+					toSchema(nextSearchParam.getType())
+						.required(Collections.emptyList())
+						.nullable(true)
+						.description(nextSearchParam.getDocumentation()));
+			}
+			RequestBody requestBody = new RequestBody()
+				.content(new Content()
+					.addMediaType(Constants.CT_X_FORM_URLENCODED, new MediaType().schema(objectSchema)))
+				.required(false);
+
+			operation.setRequestBody(requestBody);
 		}
 	}
 
diff --git a/hapi-fhir-server-openapi/src/main/resources/ca/uhn/fhir/rest/openapi/index.html b/hapi-fhir-server-openapi/src/main/resources/ca/uhn/fhir/rest/openapi/index.html
index dca541e2603..ef4d6bce1b8 100644
--- a/hapi-fhir-server-openapi/src/main/resources/ca/uhn/fhir/rest/openapi/index.html
+++ b/hapi-fhir-server-openapi/src/main/resources/ca/uhn/fhir/rest/openapi/index.html
@@ -43,6 +43,22 @@
 <script src="./swagger-ui-standalone-preset.js" charset="UTF-8"> </script>
 <script>
     window.onload = function() {
+		 //Uncheck send empty value by default
+		 const UncheckSendEmptyValuePlugin = function() {
+			 return {
+				 wrapComponents: {
+					 ParameterIncludeEmpty: (Original, { React }) => (props) => {
+						 props.isIncludedOptions.defaultValue = false;
+						 return React.createElement(Original, props)
+					 },
+					 TryItOutButton: (Original, { React }) => (props) => {
+						 if (props.isOAS3 && props.hasUserEditedBody) { props.hasUserEditedBody = false; }
+						 return React.createElement(Original, props)
+					 }
+				 }
+			 }
+		 };
+
         // Begin Swagger UI call region
         const ui = SwaggerUIBundle({
             url: "[[${OPENAPI_DOCS} + '?page=' + ${PAGE}]]",
@@ -53,6 +69,7 @@
                 SwaggerUIStandalonePreset
             ],
             plugins: [
+					UncheckSendEmptyValuePlugin
                 // SwaggerUIBundle.plugins.DownloadUrl
             ],
             // layout: "StandaloneLayout",