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

[jOOQ/jOOQ#15732] Avoid duplication when reading MULTISET content

Browse Source 
The previous implementation relied on RecordUnmapper, which works, but doesn't set the changed() flags to false when records are nested!
This commit is contained in:
Lukas Eder 2024-03-28 11:54:26 +01:00
parent 48ddbcc1ab
commit a2804f15bb
1 changed files with 3 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
jOOQ/src/main/java/org/jooq/impl/DefaultBinding.java
View File

@ -4055,14 +4055,7 @@ public class DefaultBinding<T, U> implements Binding<T, U> {
if (object == null)
return null;
return newRecord(true,
(Class<Record>) dataType.getRecordType(),
(AbstractRow<Record>) dataType.getRow(),
ctx.configuration()
).operate(r -> {
r.from(object);
return r;
});
return readMultiset(ctx, dataType);
}
default:
@ -4104,6 +4097,7 @@ public class DefaultBinding<T, U> implements Binding<T, U> {
DefaultBindingGetResultSetContext<?> c = new DefaultBindingGetResultSetContext<>(ctx.executeContext(), ctx.resultSet(), ctx.index());
r.field(0).getBinding().get((BindingGetResultSetContext) c);
r.fromArray(c.value());
r.changed(false);
return r;
}));
}
@ -4445,6 +4439,7 @@ public class DefaultBinding<T, U> implements Binding<T, U> {
result.add(newRecord(true, recordType, row, ctx.configuration()).operate(r -> {
r.from(asList(s));
r.changed(false);
return r;
}));