github/jooq
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[jOOQ/jOOQ#9001] DefaultRecordMapper now uses RecordMapperProvider

Browse Source 
Having the `DefaultRecordMapper` internally also use
`Configuration#recordMapperProvider()` is important when the user needs
control over how nested POJOs get mapped.
This commit is contained in:
Knut Wannheden 2019-07-31 14:54:31 +02:00
parent acc47ecb37
commit 2825bbdf2a
1 changed files with 9 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
jOOQ/src/main/java/org/jooq/impl/DefaultRecordMapper.java
View File

@ -325,7 +325,7 @@ public class DefaultRecordMapper<R extends Record, E> implements RecordMapper<R,
if (Stream.class.isAssignableFrom(type)) {
DefaultRecordMapper<R, Object[]> local = new DefaultRecordMapper<>(rowType, Object[].class, configuration);
RecordMapper<R, Object[]> local = configuration.recordMapperProvider().provide(rowType, Object[].class);
delegate = r -> (E) Stream.of(local.map(r));
return;
}
@ -731,21 +731,17 @@ public class DefaultRecordMapper<R extends Record, E> implements RecordMapper<R,
List<RecordMapper<R, Object>> list = new ArrayList<>();
for (java.lang.reflect.Field member : getMatchingMembers(configuration, type, prefix)) {
list.add(new DefaultRecordMapper<>(
new Fields<>(entry.getValue()),
member.getType(),
null,
configuration
));
list.add(configuration
.recordMapperProvider()
.provide(new Fields<>(entry.getValue()), member.getType())
);
}
for (Method method : getMatchingSetters(configuration, type, prefix)) {
list.add(new DefaultRecordMapper<>(
new Fields<>(entry.getValue()),
method.getParameterTypes()[0],
null,
configuration
));
list.add(configuration
.recordMapperProvider()
.provide(new Fields<>(entry.getValue()), method.getParameterTypes()[0])
);
}
nested.put(prefix, list);