Changeset 3845

Timestamp:

12/24/07 10:00:12 (1 year ago)

Author:

roger

Message:

Fix #1641.

Files:

• branches/1.0/CHANGELOG.txt (modified) (1 diff)
• branches/1.0/turbogears/paginate.py (modified) (1 diff)
• branches/1.0/turbogears/tests/test_paginate.py (modified) (2 diffs)

branches/1.0/CHANGELOG.txt

@@ -20 +20 @@
-  exception (#1629).
-* Added missing logger configuration to quickstarted projects (#1630).
+* Fixed pagination of empty data when a SQL with "group by" is issued (#1641).
-
-Contributors
-~~~~~~~~~~~~
-
-
+, ondrejj
-
-

branches/1.0/turbogears/paginate.py

@@ -172 +172 @@
-               (SASelectResults and isinstance(var_data, SASelectResults)) or \
-               (Query and isinstance(var_data, Query)):
-
+ or 0
-                if ordering:
-                    # Build order_by list.

branches/1.0/turbogears/tests/test_paginate.py

@@ -619 +619 @@
-        zero_limit = expose("turbogears.tests.paginate")(zero_limit)
-
+        def empty_with_groupby(self):
+            data = session.query(Address).filter(Address.c.id<0).group_by(Address.c.user_id)
+
+            spy = Spy(var_name='data', pages=[], limit=10, page_count=0,
+                      order=None, row_count=0)
+            return dict(data=data, spy=spy)
+
+        empty_with_groupby = paginate("data", default_order="id")(empty_with_groupby)
+        empty_with_groupby = expose("turbogears.tests.paginate")(empty_with_groupby)
+
-    # end of MyRoot ------------------------------------------------------------
-
@@ -895 +905 @@
-            self.request("/zero_limit/?method=%s" % method)
-            self.assert_order(1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16)
+
+    def test_ticket_1641(self):
+        self.request("/empty_with_groupby")
+        self.assert_order()
-
-