project review

Author: bnasslahsen

springdoc-openapi-common/src/main/java/org/springdoc/core/OpenAPIBuilder.java

@@ -200,8 +200,9 @@ private void buildOpenAPIWithOpenAPIDefinition(OpenAPI openAPI, OpenAPIDefinitio
         // OpenApiDefinition tags
         AnnotationsUtils.getTags(apiDef.tags(), false).ifPresent(tags -> openAPI.setTags(new ArrayList<>(tags)));
         // OpenApiDefinition servers
-        if(AnnotationsUtils.getServers(apiDef.servers()).isPresent()){
-            openAPI.setServers(AnnotationsUtils.getServers(apiDef.servers()).get());
+        Optional<List<Server>> optionalServers = AnnotationsUtils.getServers(apiDef.servers());
+        if(optionalServers.isPresent()){
+            openAPI.setServers(optionalServers.get());
             this.isServersPresent=true;
         }
         // OpenApiDefinition extensions

springdoc-openapi-common/src/main/java/org/springdoc/core/SwaggerUiConfigProperties.java

@@ -132,10 +132,7 @@ public static void setSwaggerUrls(List<SwaggerUrl> swaggerUrls) {
     }
 
     public static void addUrl(String url) {
-        swaggerUrls.forEach(elt ->
-        {
-            elt.setUrl(url + DEFAULT_PATH_SEPARATOR + elt.getName());
-        });
+        swaggerUrls.forEach(elt -> elt.setUrl(url + DEFAULT_PATH_SEPARATOR + elt.getName()));
     }
 
     public Map<String, Object> getConfigParameters() {

springdoc-openapi-webmvc-core/src/test/java/test/org/springdoc/core/BlockingAutoConfigurationTest.java renamed to springdoc-openapi-webmvc-core/src/test/java/test/org/springdoc/api/app72/BlockingAutoConfigurationTest.java

@@ -1,4 +1,4 @@
-package test.org.springdoc.core;
+package test.org.springdoc.api.app72;
 
 import org.junit.Test;
 import org.springdoc.core.GroupedOpenApi;