Django

Code

Changeset 7363

Show
Ignore:
Timestamp:
03/25/08 00:20:55 (7 months ago)
Author:
adrian
Message:

Fixed #6852 -- Converted the admin to use the new paginator, to remove DeprecationWarnings?. Thanks for the patch, nickefford

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • django/trunk/django/contrib/admin/templatetags/admin_list.py

    r6888 r7363  
    2323        return mark_safe(u'<span class="this-page">%d</span> ' % (i+1)) 
    2424    else: 
    25         return mark_safe(u'<a href="%s"%s>%d</a> ' % (cl.get_query_string({PAGE_VAR: i}), (i == cl.paginator.pages-1 and ' class="end"' or ''), i+1)) 
     25        return mark_safe(u'<a href="%s"%s>%d</a> ' % (cl.get_query_string({PAGE_VAR: i}), (i == cl.paginator.num_pages-1 and ' class="end"' or ''), i+1)) 
    2626paginator_number = register.simple_tag(paginator_number) 
    2727 
     
    3838        # If there are 10 or fewer pages, display links to every page. 
    3939        # Otherwise, do some fancy 
    40         if paginator.pages <= 10: 
    41             page_range = range(paginator.pages) 
     40        if paginator.num_pages <= 10: 
     41            page_range = range(paginator.num_pages) 
    4242        else: 
    4343            # Insert "smart" pagination links, so that there are always ON_ENDS 
     
    5151            else: 
    5252                page_range.extend(range(0, page_num + 1)) 
    53             if page_num < (paginator.pages - ON_EACH_SIDE - ON_ENDS - 1): 
     53            if page_num < (paginator.num_pages - ON_EACH_SIDE - ON_ENDS - 1): 
    5454                page_range.extend(range(page_num + 1, page_num + ON_EACH_SIDE + 1)) 
    5555                page_range.append(DOT) 
    56                 page_range.extend(range(paginator.pages - ON_ENDS, paginator.pages)) 
    57             else: 
    58                 page_range.extend(range(page_num + 1, paginator.pages)) 
     56                page_range.extend(range(paginator.num_pages - ON_ENDS, paginator.num_pages)) 
     57            else: 
     58                page_range.extend(range(page_num + 1, paginator.num_pages)) 
    5959 
    6060    need_show_all_link = cl.can_show_all and not cl.show_all and cl.multi_page 
  • django/trunk/django/contrib/admin/views/main.py

    r6691 r7363  
    66from django.contrib.contenttypes.models import ContentType 
    77from django.core.exceptions import ImproperlyConfigured, ObjectDoesNotExist, PermissionDenied 
    8 from django.core.paginator import ObjectPaginator, InvalidPage 
     8from django.core.paginator import QuerySetPaginator, InvalidPage 
    99from django.shortcuts import get_object_or_404, render_to_response 
    1010from django.db import models 
     
    612612 
    613613    def get_results(self, request): 
    614         paginator = ObjectPaginator(self.query_set, self.lookup_opts.admin.list_per_page) 
     614        paginator = QuerySetPaginator(self.query_set, self.lookup_opts.admin.list_per_page) 
    615615 
    616616        # Get the number of objects, with admin filters applied. 
    617617        try: 
    618             result_count = paginator.hits 
     618            result_count = paginator.count 
    619619        # Naked except! Because we don't have any other way of validating 
    620620        # "params". They might be invalid if the keyword arguments are 
     
    641641        else: 
    642642            try: 
    643                 result_list = paginator.get_page(self.page_num) 
     643                result_list = paginator.page(self.page_num+1).object_list 
    644644            except InvalidPage: 
    645645                result_list = ()