diff options
| author | Sebastian Spaeth <Sebastian@SSpaeth.de> | 2012-12-17 12:56:29 +0100 |
|---|---|---|
| committer | Sebastian Spaeth <Sebastian@SSpaeth.de> | 2012-12-21 11:30:29 +0100 |
| commit | 10f1f2f56b9ff1153b58cb938427d74c8211194b (patch) | |
| tree | 2da4e05cda0edb30ef9bdce5d96b4578e0914dc8 /mediagoblin/tools/pagination.py | |
| parent | af008743ca89a3f3634efa31718a65ea8adbc2fd (diff) | |
| download | mediagoblin-10f1f2f56b9ff1153b58cb938427d74c8211194b.tar.lz mediagoblin-10f1f2f56b9ff1153b58cb938427d74c8211194b.tar.xz mediagoblin-10f1f2f56b9ff1153b58cb938427d74c8211194b.zip | |
Remove Mongoism query.skip()
sqlalchemy supports slice() or [n:m] just fine.
Right now, it seems we cannot distinguish beween "empty" results
and out-of bound slices. It would be nice if we could distinguish
these somehow.
Signed-off-by: Sebastian Spaeth <Sebastian@SSpaeth.de>
Diffstat (limited to 'mediagoblin/tools/pagination.py')
| -rw-r--r-- | mediagoblin/tools/pagination.py | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/mediagoblin/tools/pagination.py b/mediagoblin/tools/pagination.py index 141d91cc..af889abd 100644 --- a/mediagoblin/tools/pagination.py +++ b/mediagoblin/tools/pagination.py @@ -63,8 +63,11 @@ class Pagination(object): """ Returns slice of objects for the requested page """ - return self.cursor.skip( - (self.page - 1) * self.per_page).limit(self.per_page) + # TODO, return None for out of index so templates can + # distinguish between empty galleries and out-of-bound pages??? + return self.cursor.slice( + (self.page - 1) * self.per_page, + self.page * self.per_page) @property def pages(self): |