Skip to content

Commit 830f826

Browse files
committed
code review
1 parent 740827a commit 830f826

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

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

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,7 @@ public OpenAPI build(Locale locale) {
247247
else {
248248
try {
249249
ObjectMapper objectMapper = new ObjectMapper();
250-
calculatedOpenAPI = objectMapper.readValue(objectMapper.writeValueAsString(openAPI), OpenAPI.class );
250+
calculatedOpenAPI = objectMapper.readValue(objectMapper.writeValueAsString(openAPI), OpenAPI.class);
251251
}
252252
catch (JsonProcessingException e) {
253253
LOGGER.warn("Json Processing Exception occurred: {}", e.getMessage());
@@ -258,7 +258,7 @@ public OpenAPI build(Locale locale) {
258258
buildOpenAPIWithOpenAPIDefinition(calculatedOpenAPI, apiDef.get(), locale);
259259
}
260260
// Set default info
261-
else if (calculatedOpenAPI.getInfo() == null) {
261+
else if (calculatedOpenAPI != null && calculatedOpenAPI.getInfo() == null) {
262262
Info infos = new Info().title(DEFAULT_TITLE).version(DEFAULT_VERSION);
263263
calculatedOpenAPI.setInfo(infos);
264264
}
@@ -270,7 +270,8 @@ else if (calculatedOpenAPI.getInfo() == null) {
270270
initializeHiddenRestController();
271271

272272
// add security schemes
273-
this.calculateSecuritySchemes(calculatedOpenAPI.getComponents(), locale);
273+
if (calculatedOpenAPI != null)
274+
this.calculateSecuritySchemes(calculatedOpenAPI.getComponents(), locale);
274275
openApiBuilderCustomisers.ifPresent(customisers -> customisers.forEach(customiser -> customiser.customise(this)));
275276

276277
return calculatedOpenAPI;

0 commit comments

Comments
 (0)