From 90b04e68b85e3e6b941baa34bffe5d67cf53980d Mon Sep 17 00:00:00 2001
From: Michael Bayne <mdb@samskivert.com>
Date: Mon, 13 Dec 2010 17:50:50 +0000
Subject: [PATCH] Prevent confusion if someone uses selectCount() (which
 returns int) on a grouped query. They should be using
 select(Funcs.count(Exps.literal("*"))) (which returns List<Integer>) in such
 cases.

---
 src/main/java/com/samskivert/depot/Query.java | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/main/java/com/samskivert/depot/Query.java b/src/main/java/com/samskivert/depot/Query.java
index 63882be..9a2f721 100644
--- a/src/main/java/com/samskivert/depot/Query.java
+++ b/src/main/java/com/samskivert/depot/Query.java
@@ -359,6 +359,7 @@ public class Query<T extends PersistentRecord>
      */
     public int selectCount ()
     {
+        checkState(_groupBy == null, "You surely mean to select(Funcs.count(Exps.literal(\"*\"))).");
         _fromOverride = new FromOverride(_pclass);
         return _repo.load(CountRecord.class, _cache, getClauseArray()).count;
     }