root
/
spring-framework
mirror of https://github.com/spring-projects/spring-framework.git
1
0
Fork 0
Code Issues Actions Packages Projects Releases Wiki Activity

Polishing contribution 

Closes gh-29868
This commit is contained in:
rstoyanchev 2023-01-30 16:48:57 +00:00
parent aa9f445422
commit 15931532cd
5 changed files with 9 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
spring-web/src/main/java/org/springframework/http/ProblemDetail.java
View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2022 the original author or authors. * Copyright 2002-2023 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -206,7 +206,8 @@ public class ProblemDetail {
* are rendered as top level key-value pairs in the output JSON. Otherwise, * are rendered as top level key-value pairs in the output JSON. Otherwise,
* they are rendered as a {@code "properties"} sub-map. * they are rendered as a {@code "properties"} sub-map.
* @param name the property name * @param name the property name
* @param value the property value * @param value the property value, possibly {@code null} if the intent is
* to include a property with its value set to "null"
* @see org.springframework.http.converter.json.ProblemDetailJacksonMixin * @see org.springframework.http.converter.json.ProblemDetailJacksonMixin
*/ */
public void setProperty(String name, @Nullable Object value) { public void setProperty(String name, @Nullable Object value) {

2
spring-web/src/main/java/org/springframework/http/converter/json/ProblemDetailJacksonMixin.java
View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2022 the original author or authors. * Copyright 2002-2023 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.

4
spring-web/src/main/java/org/springframework/web/DefaultErrorResponseBuilder.java
View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2022 the original author or authors. * Copyright 2002-2023 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -122,7 +122,7 @@ final class DefaultErrorResponseBuilder implements ErrorResponse.Builder {
} }
@Override @Override
public ErrorResponse.Builder property(String name, Object value) { public ErrorResponse.Builder property(String name, @Nullable Object value) {
this.problemDetail.setProperty(name, value); this.problemDetail.setProperty(name, value);
return this; return this;
} }

4
spring-web/src/main/java/org/springframework/web/ErrorResponse.java
View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2022 the original author or authors. * Copyright 2002-2023 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -265,7 +265,7 @@ public interface ErrorResponse {
* property} on the underlying {@code ProblemDetail}. * property} on the underlying {@code ProblemDetail}.
* @return the same builder instance * @return the same builder instance
*/ */
Builder property(String name, Object value); Builder property(String name, @Nullable Object value);
/** /**
* Build the {@code ErrorResponse} instance. * Build the {@code ErrorResponse} instance.

2
spring-web/src/test/java/org/springframework/http/converter/json/ProblemDetailJacksonMixinTests.java
View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2022 the original author or authors. * Copyright 2002-2023 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.