From 34289fbcd5c858d8d74a71f242a3f047148f2c4b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E5=BC=80=E6=BA=90=E6=B5=B7=E5=93=A5?= Date: Wed, 11 Oct 2023 09:51:22 +0800 Subject: [PATCH] fix: fix and close #184 --- .../com/mybatisflex/core/query/QueryWrapper.java | 2 +- .../mybatisflex/coretest/AccountSqlTester.java | 15 +++++++++++++++ 2 files changed, 16 insertions(+), 1 deletion(-) diff --git a/mybatis-flex-core/src/main/java/com/mybatisflex/core/query/QueryWrapper.java b/mybatis-flex-core/src/main/java/com/mybatisflex/core/query/QueryWrapper.java index bd4e3b5d..5f9d4545 100644 --- a/mybatis-flex-core/src/main/java/com/mybatisflex/core/query/QueryWrapper.java +++ b/mybatis-flex-core/src/main/java/com/mybatisflex/core/query/QueryWrapper.java @@ -611,7 +611,7 @@ public class QueryWrapper extends BaseQueryWrapper { } public QueryWrapper groupBy(String name) { - addGroupByColumns(new QueryColumn(name)); + addGroupByColumns(new RawQueryColumn(name)); return this; } diff --git a/mybatis-flex-core/src/test/java/com/mybatisflex/coretest/AccountSqlTester.java b/mybatis-flex-core/src/test/java/com/mybatisflex/coretest/AccountSqlTester.java index 94d4c77c..f0c0ca6b 100644 --- a/mybatis-flex-core/src/test/java/com/mybatisflex/coretest/AccountSqlTester.java +++ b/mybatis-flex-core/src/test/java/com/mybatisflex/coretest/AccountSqlTester.java @@ -391,6 +391,21 @@ public class AccountSqlTester { System.out.println(query.toSQL()); } + //https://gitee.com/mybatis-flex/mybatis-flex/issues/I7EAY9 + @Test + public void testGroup184() { + QueryWrapper query = QueryWrapper .create() + .select(left(ACCOUNT.AGE, 6).as("regionCode")) + .from(ACCOUNT) + .groupBy("regionCode"); + + Assert.assertEquals("SELECT LEFT(`age`, 6) AS `regionCode` FROM `tb_account` " + + "GROUP BY regionCode" + ,query.toSQL()); + + System.out.println(query.toSQL()); + } + @Test public void testHavingSql() { QueryWrapper query = QueryWrapper.create()