search

pirave / django-pagination

Automatically exported from code.google.com/p/django-pagination
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Remove support for depreciated QuerySetPaginator #15

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Paginator and QuerySetPaginator are now the same thing, so there is no need
for QuerySet detection. This line:

value = list(value)

also kills any lazy loading objects that are not QuerySets (djangosearch,
django-sphinx etc), making it impractical to use them with django-pagination.

I have attached a patch which should fix both issues.

Original issue reported on code.google.com by ben.firs...@gmail.com on 29 Aug 2008 at 12:29

Attachments:

GoogleCodeExporter commented 8 years ago 
Please do accept this patch! I'm using django-pagination with djangosearch,and 
it's painful before this patch, 
and great after it. The lack of lazy loading on non-QuerySets is killing me -- 
but I love django-pagination and 
want to use it! :)

Original comment by jeffrey....@gmail.com on 30 Aug 2008 at 2:43

GoogleCodeExporter commented 8 years ago 
Fixed in r27.  Thanks, Ben Firshman!

Original comment by flo...@gmail.com on 30 Aug 2008 at 6:09