CodeCow/querydsl
1
0
Fork 0
mirror of https://github.com/querydsl/querydsl.git synced 2026-06-13 21:01:01 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix count with empty resultsets

Browse Source
This commit is contained in:
Ruben Dijkstra 2014-11-26 21:28:18 +01:00
parent dccc48d4a4
commit fb505d0495
2 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
querydsl-sql/src/main/java/com/mysema/query/sql/AbstractSQLQuery.java
View File

@ -482,10 +482,14 @@ public abstract class AbstractSQLQuery<Q extends AbstractSQLQuery<Q>> extends Pr
listeners.preExecute(context);
rs = stmt.executeQuery();
rs.next();
boolean hasResult = rs.next();
listeners.executed(context);
return rs.getLong(1);
if (hasResult) {
return rs.getLong(1);
} else {
return 0;
}
} catch (SQLException e) {
onException(context, e);
throw configuration.translate(queryString, constants, e);

10
querydsl-sql/src/test/java/com/mysema/query/SelectBase.java
View File

@ -610,6 +610,16 @@ public class SelectBase extends AbstractBaseTest {
employee.salary.avg());
}
@Test
public void Having_Count() {
//Produces empty resultset https://github.com/querydsl/querydsl/issues/1055
query().from(employee)
.innerJoin(employee2)
.groupBy(employee.id)
.having(Wildcard.count.eq(4L))
.listResults(employee.id, employee.firstname);
}
@SuppressWarnings("unchecked")
@Test(expected=IllegalArgumentException.class)
public void IllegalUnion() throws SQLException {