From 8f6244092d84505cd4002c8579fca57f782a3305 Mon Sep 17 00:00:00 2001 From: Lukas Eder Date: Tue, 11 Jan 2022 10:32:20 +0100 Subject: [PATCH] [jOOQ/jOOQ#12801] GroupConcat renders string_agg separator argument within wrong parentheses --- jOOQ/src/main/java/org/jooq/impl/ListAgg.java | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/jOOQ/src/main/java/org/jooq/impl/ListAgg.java b/jOOQ/src/main/java/org/jooq/impl/ListAgg.java index 37dc051fea..d5170fa689 100644 --- a/jOOQ/src/main/java/org/jooq/impl/ListAgg.java +++ b/jOOQ/src/main/java/org/jooq/impl/ListAgg.java @@ -202,13 +202,14 @@ final class ListAgg extends AbstractAggregateFunction implements UNotYet ctx.sql('('); // The explicit cast is needed in Postgres - QueryPartListView> args = wrap( - castIfNeeded((Field) arguments.get(0), String.class), - arguments.size() > 1 ? arguments.get(1) : inline("") - ); - + QueryPartListView> args = wrap(castIfNeeded((Field) arguments.get(0), String.class)); acceptArguments1(ctx, args); - + + if (arguments.size() > 1) + ctx.sql(", ").visit(arguments.get(1)); + else + ctx.sql(", ").visit(inline("")); +