Opened 18 years ago
Closed 18 years ago
#6852 closed (fixed)
DeprecationWarning in admin, caused by recent changes to pagination
| Reported by: | Nick Efford | Owned by: | nobody |
|---|---|---|---|
| Component: | contrib.admin | Version: | dev |
| Severity: | Keywords: | admin, pagination | |
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
The admin app uses the old ObjectPaginator class, which triggers DeprecationWarning as of [7306]. The attached patch, against trunk, tweaks admin to use the new API. It seems to work OK as far I as can see, but wiser heads than mine will need to verify :)
Note that this is against trunk, not newforms-admin, as the latter hasn't merged in the new pagination code yet.
Feel free to wontfix if you think we can live with warnings until newforms-admin arrives. In any case, the patch might be useful to newforms-admin devs as the basis of a proper, long-term solution.
Attachments (1)
Change History (2)
by , 18 years ago
| Attachment: | admin_pagination.diff added |
|---|
comment:1 by , 18 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Fixes to pagination code of admin app