Skip to content

Commit c9d36fd

Browse files
committed
code review
1 parent 4495df6 commit c9d36fd

File tree

3 files changed

+2
-8
lines changed

3 files changed

+2
-8
lines changed

springdoc-openapi-starter-common/src/main/java/org/springdoc/core/customizers/DataRestDelegatingMethodParameterCustomizer.java

-2
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323
package org.springdoc.core.customizers;
2424

2525
import java.lang.annotation.Annotation;
26-
import java.lang.reflect.Field;
2726
import java.util.Arrays;
2827
import java.util.List;
2928
import java.util.Objects;
@@ -128,7 +127,6 @@ public void customize(MethodParameter originalParameter, MethodParameter methodP
128127
*/
129128
private Optional<Annotation> getNewParameterAnnotationForField(MethodParameter methodParameter, PageableDefault pageableDefault, SortDefault sortDefault) {
130129
String parameterName = methodParameter.getParameterName();
131-
Field field;
132130
Parameter parameterNew;
133131
try {
134132
Parameter parameter = methodParameter.getContainingClass().getDeclaredField(parameterName).getAnnotation(Parameter.class);

springdoc-openapi-starter-webflux-api/src/main/java/org/springdoc/webflux/api/MultipleOpenApiResource.java

+1-3
Original file line numberDiff line numberDiff line change
@@ -141,9 +141,7 @@ public void afterPropertiesSet() {
141141
springDocConfigProperties.addGroupConfig(groupConfig);
142142
return buildWebFluxOpenApiResource(item);
143143
},
144-
(existingValue, newValue) -> {
145-
return existingValue; // choice to keep the existing value
146-
}
144+
(existingValue, newValue) -> existingValue // choice to keep the existing value
147145
));
148146
}
149147

springdoc-openapi-starter-webmvc-api/src/main/java/org/springdoc/webmvc/api/MultipleOpenApiResource.java

+1-3
Original file line numberDiff line numberDiff line change
@@ -140,9 +140,7 @@ public void afterPropertiesSet() {
140140
springDocConfigProperties.addGroupConfig(groupConfig);
141141
return buildWebMvcOpenApiResource(item);
142142
},
143-
(existingValue, newValue) -> {
144-
return existingValue; // choice to keep the existing value
145-
}
143+
(existingValue, newValue) -> existingValue // choice to keep the existing value
146144
));
147145
}
148146

0 commit comments

Comments
 (0)