Skip to content
This repository was archived by the owner on Dec 31, 2024. It is now read-only.

Commit 8f43685

Browse files
committed
Include final fields when converting a POJO to a map
1 parent 98e51d1 commit 8f43685

File tree

2 files changed

+2
-5
lines changed
  • feign-form/src/main/java/feign/form/util
  • feign-form-spring/src/test/java/feign/form/feign/spring

2 files changed

+2
-5
lines changed

feign-form-spring/src/test/java/feign/form/feign/spring/Dto.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,19 +22,17 @@
2222

2323
import lombok.AllArgsConstructor;
2424
import lombok.Data;
25-
import lombok.NoArgsConstructor;
2625
import lombok.experimental.FieldDefaults;
2726
import org.springframework.web.multipart.MultipartFile;
2827

2928
@Data
30-
@NoArgsConstructor
3129
@AllArgsConstructor
3230
@FieldDefaults(level = PRIVATE)
3331
public class Dto implements Serializable {
3432

3533
private static final long serialVersionUID = -4218390863359894943L;
3634

37-
String field1;
35+
final String field1;
3836

3937
int field2;
4038

feign-form/src/main/java/feign/form/util/PojoUtil.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616

1717
package feign.form.util;
1818

19-
import static java.lang.reflect.Modifier.isFinal;
2019
import static java.lang.reflect.Modifier.isStatic;
2120
import static lombok.AccessLevel.PRIVATE;
2221

@@ -63,7 +62,7 @@ public static Map<String, Object> toMap (@NonNull Object object) {
6362
val setAccessibleAction = new SetAccessibleAction();
6463
for (val field : type.getDeclaredFields()) {
6564
val modifiers = field.getModifiers();
66-
if (isFinal(modifiers) || isStatic(modifiers)) {
65+
if (isStatic(modifiers)) {
6766
continue;
6867
}
6968
setAccessibleAction.setField(field);

0 commit comments

Comments
 (0)