Fixes ThatRendle#301

richardhopton · richardhopton · commit 356e37037b18 · 2013-08-07T12:13:03.000+01:00
diff --git a/Simple.Data.SqlServer/SqlQueryPager.cs b/Simple.Data.SqlServer/SqlQueryPager.cs
@@ -44,11 +44,12 @@ public IEnumerable<string> ApplyPaging(string sql, string[] keys, int skip, int
                 builder.AppendFormat(string.Join(" AND ", keys.Select(MakeDataJoin)));
             else
                 builder.AppendFormat(MakeDataJoin(keys[0]));
+            var groupBy = ExtractGroupBy(ref fromEtc);
             var rest = Regex.Replace(fromEtc, @"^from (\[.*?\]\.\[.*?\])", @"");
             builder.Append(rest);
             
-            builder.AppendFormat(" AND [_#_] BETWEEN {0} AND {1}", skip + 1, skip + take);
-
+            builder.AppendFormat(" AND [_#_] BETWEEN {0} AND {1} ", skip + 1, skip + take);
+            builder.Append(groupBy);
             yield return builder.ToString();
         }
 
@@ -79,5 +80,17 @@ private static string ExtractOrderBy(string columns, string[] keys, ref string f
             }
             return orderBy;
         }
+
+        private static string ExtractGroupBy(ref string fromEtc)
+        {
+            string groupBy = string.Empty;
+            int index = fromEtc.IndexOf("GROUP BY", StringComparison.InvariantCultureIgnoreCase);
+            if (index > -1)
+            {
+                groupBy = fromEtc.Substring(index).Trim();
+                fromEtc = fromEtc.Remove(index).Trim();
+            }
+            return groupBy;
+        }
     }
 }
