Skip to content

Commit

Permalink
Merge pull request #534 from cj848/fix/bugfix-function-serializer
Browse files Browse the repository at this point in the history
fix the missing single quotation surrounding the function name of JpqlFunctionSerializer.
shouwn authored Nov 22, 2023
2 parents b7a6bf5 + 8233f0c commit 0919332
Showing 2 changed files with 6 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -21,7 +21,9 @@ class JpqlFunctionSerializer : JpqlSerializer<JpqlFunction<*>> {
writer.write("FUNCTION")

writer.writeParentheses {
writer.write("'")
writer.write(part.name)
writer.write("'")

if (IterableUtils.isNotEmpty(part.args)) {
writer.write(",")
Original file line number Diff line number Diff line change
@@ -59,7 +59,9 @@ class JpqlFunctionSerializerTest : WithAssertions {
verifySequence {
writer.write("FUNCTION")
writer.writeParentheses(any())
writer.write("'")
writer.write(functionName1)
writer.write("'")
writer.write(",")
writer.write(" ")
writer.writeEach(expressions, ", ", any())
@@ -86,7 +88,9 @@ class JpqlFunctionSerializerTest : WithAssertions {
verifySequence {
writer.write("FUNCTION")
writer.writeParentheses(any())
writer.write("'")
writer.write(functionName1)
writer.write("'")
}
}
}

0 comments on commit 0919332

Please sign in to comment.