When using @operation, treat deprecated false as NULL (reduce openapi.json noise) (#575)

The prior change adding reading @operation, introduced an issue in that all non-deprecated methods then explicitly had in openapi.json a deprecated = false attribute.

This "fixes" that issue by only setting deprecated attribute when it is true.

Author: rbygrave

http-generator-core/src/main/java/io/avaje/http/generator/core/MethodReader.java

@@ -273,9 +273,12 @@ private List<OpenAPIResponsePrism> buildApiResponses() {
   public void readOperation(Operation operation, Javadoc javadoc) {
     OperationPrism.getOptionalOn(element).ifPresent(an -> {
       operation.setOperationId(emptyToNull(an.operationId()));
-      operation.setDeprecated(an.deprecated());
       operation.setSummary(emptyToNull(an.summary()));
       operation.setDescription(emptyToNull(an.description()));
+      if (Boolean.TRUE.equals(an.deprecated())) {
+        // leave deprecated false as NULL to reduce openapi noise
+        operation.setDeprecated(true);
+      }
     });
     if (operation.getDescription() == null) {
       operation.setDescription(javadoc.getDescription());

tests/test-javalin/src/main/java/org/example/myapp/web/HelloController.java

@@ -76,6 +76,7 @@ HelloDto hello(int id, LocalDate date, String otherParam) {
    * @return The Hellos that we found.
    */
   @Roles(AppRoles.ADMIN)
+  @Operation(operationId = "findByName")
   @Get("/findbyname/{name}")
   List<HelloDto> findByName(String name, @QueryParam("my-param") @Default("one") String myParam) {
     return new ArrayList<>();

tests/test-javalin/src/main/resources/public/openapi.json

@@ -351,6 +351,7 @@
 				],
 				"summary" : "Find Hellos by name",
 				"description" : "",
+				"operationId" : "findByName",
 				"parameters" : [
 					{
 						"name" : "name",