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

[jOOQ/jOOQ#14166] Render correct SQL when COALESCE have empty contents

Browse Source
This commit is contained in:
Lukas Eder 2022-11-04 12:18:16 +01:00
parent 607bde4fa3
commit ec5b7a1060
1 changed files with 20 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
jOOQ/src/main/java/org/jooq/impl/Coalesce.java
View File

@ -37,6 +37,7 @@
*/
package org.jooq.impl;
import static org.jooq.impl.DSL.NULL;
import static org.jooq.impl.Names.N_COALESCE;
import static org.jooq.impl.SQLDataType.OTHER;
import static org.jooq.impl.Tools.EMPTY_FIELD;
@ -70,7 +71,14 @@ final class Coalesce<T> extends AbstractField<T> implements QOM.Coalesce<T> {
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {
if (fields.length == 0) {
ctx.visit(NULL(getDataType()));
}
else if (fields.length == 1) {
ctx.visit(fields[0]);
}
else {
switch (ctx.family()) {
@ -84,17 +92,18 @@ final class Coalesce<T> extends AbstractField<T> implements QOM.Coalesce<T> {
case DERBY: {
// [#13601] Workaround for https://issues.apache.org/jira/browse/DERBY-7139
ctx.visit(DSL.function(N_COALESCE, getDataType(),
Tools.map(fields, f -> f.getDataType().isBoolean() ? new ParenthesisedField<>(f) : f, Field[]::new)
));
break;
}
case DERBY: {
// [#13601] Workaround for https://issues.apache.org/jira/browse/DERBY-7139
ctx.visit(DSL.function(N_COALESCE, getDataType(),
Tools.map(fields, f -> f.getDataType().isBoolean() ? new ParenthesisedField<>(f) : f, Field[]::new)
));
break;
}
default: {
ctx.visit(DSL.function(N_COALESCE, getDataType(), fields));
break;
default: {
ctx.visit(DSL.function(N_COALESCE, getDataType(), fields));
break;
}
}
}
}