Opened 19 years ago
Closed 18 years ago
#720 closed enhancement (fixed)
Added methods to ObjectPaginator for getting the numbers of first and last object on page
Reported by: | Owned by: | Adrian Holovaty | |
---|---|---|---|
Component: | Core (Other) | Version: | |
Severity: | minor | Keywords: | |
Cc: | Triage Stage: | Unreviewed | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Hi,
I added two convenience methods to django/core/paginator.py for getting the numbers of the first and last object on the current page, relative to the total number of objects found (paginator.hits):
- first_on_page(page_number)
- last_on_page(page_number)
The purpose of having these methods would be to make it easy for the view author to make these values available to the template author who would commonly need to output stuff like:
Now showing x through y of z matching records
I have tried to implement these methods in the style of the other "public" methods of the ObjectPaginator class, so that they take the page number as their only argument when called externally. I am still fairly new to Python and to submitting patches to Django, so please feel free to rename/correct at will.
If these methods are incorporated into the default distribution, I think the generic views should be modified accordingly to provide these values. I would be happy to make those changes, if needed.
Thanks!
/Morten
Attachments (1)
Change History (5)
by , 19 years ago
Attachment: | paginator_diff.py added |
---|
comment:1 by , 18 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:2 by , 18 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
A small correction is needed here.
If you call last_on_page before a call to _get_pages you get a TypeError since self._pages has not been set.
unsupported operand type(s) for -: 'NoneType' and 'int'
This is a small thing since usually you will have called this method but it should be easily corrected I suppose.
br
Vidar
comment:4 by , 18 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
svn diff output with added methods