Почему не работает пагинация?

Question

[Ram Ce]

пытаюсь настроить пагинацию, но почему-то не получается, подскажите что делаю не так? строго не судите, я еще новичок
views.py

class ProductList(ListView):
    model = Product
    template_name = 'products/list.html'
    paginate_by = 2
    context_object_name = "product_list"

def product_list(request, category_slug=None):
    category = None
    categories = Category.objects.all()
    products = Product.objects.filter(available=True)
    if category_slug:
        category = get_object_or_404(Category, slug=category_slug)
        products = products.filter(category=category)
    
    def get_context_data(self, **kwargs):
        context = super(ProductList(ListView), self).get_context_data(**kwargs)
        paginator = Paginator(products, self.paginate_by)

        page = self.request.GET.get('page')

        try:
            prods = paginator.page(page)
        except PageNotAnInteger:
            prods = paginator.page(1)
        except EmptyPage:
            prods = paginator.page(paginator.num_pages)

        context['prodsCatalog'] = prods
        return context

    return render(request,
                  'products/list.html',
                  {'category': category,
                   'categories': categories,
                   'products': products})

urls.py

urlpatterns = [
    url(r'^$', views.product_list, name='product_list'),
    url(r'^(?P<category_slug>[-\w]+)/$',
        views.product_list,
        name='product_list_by_category'),
    url(r'^(?P<id>\d+)/(?P<slug>[-\w]+)/$',
        views.product_detail,
        name='product_detail'),
]

paginator.html

<div class="pagination">
            <span class="page-links">
                {% if page_obj.has_previous %}
                    <a href="/products?page={{ page_obj.previous_page_number }}">previous</a>
                {% endif %}
                <span class="page-current">
                    Page {{ page_obj.number }} of {{ page_obj.paginator.num_pages }}.
                </span>
                {% if page_obj.has_next %}
                    <a href="/products?page={{ page_obj.next_page_number }}">next</a>
                {% endif %}
            </span>
        </div>

Answer 1

[Виталий Дмтириев]

Попробуй так

def news(request):
    # 'Новости'
    all_news = News.objects.filter(type='news').filter(visible=True).order_by('-publish')
    paginator = Paginator(all_news, 10)
    page = request.GET.get('page')
    try:
        all_news = paginator.page(page)
    except PageNotAnInteger:
        # Если страница не является целым числом,возвращаем первую страницу.
        all_news = paginator.page(1)
    except EmptyPage:
        # Если номер страницы больше, чем общее количество страниц,
        # возвращаем последнюю.
        all_news = paginator.page(paginator.num_pages)
    return render(request, 'news.html',
                  {'page': page,
                   'all_news': all_news,})

Answer 2

[Владимир]

Присмотритесь к своему коду, это просто какая-тол дикая мешанина из CBV и процедурного подходов: функция product_list не входит в класс ProductList.

Comments

[Ram Ce]

поправил, но теперь управление пагинатором отображается и работает(?), но список товаров не отображается

[Ram Ce]

если вас не затруднит, прокомментировать ошибки, или выложить исправленный код, чтобы понял в чем ошибки и как их исправить

Answer 3

[neqrotek]

Тут написано, как правильно собрать пагинацию. Смысл в том, что сам список "Product" должен участвовать в переходе между страницами. А тут связи нет.
Как у меня получилось:

views.py

def product_list(request)
    category = None
    categories = Category.objects.all()
    products = Product.objects.filter(available=True)
    paginator = Paginator(products, 3)  # 12 posts in each page
    page = request.GET.get('page')

    if category_slug:
        category = get_object_or_404(Category, slug=category_slug)
        products = products.filter(category=category)
   
    try:
        products = paginator.page(page)
    except PageNotAnInteger:
        products = paginator.page(1)
    except EmptyPage:
        products = paginator.page(paginator.num_pages)

    return render(request,
                  'products/list.html',
                  {'category': category,
                   'categories': categories,
                   'products': products
                   'page', page})

В месте где будет выдаваться блок:

main.html

{% for product in products %}
{% endfor %}
{% include "shop/pagination.html" with page=products %}

И в шаблоне pagination.html:

pagination.html

<div class="pagination">
    <span class="step-links">
        {% if page.has_previous %}
            <a href="?page={{ products.previous_page_number }}">Назад</a>
        {% endif %}
        <span class="current">
            Страница {{ products.number }} из {{ products.paginator.num_pages }}.
        </span>
        {% if page.has_next %}
            <a href="?page={{ products.next_page_number }}">Вперед</a>
        {% endif %}
    </span>
</div>

Мое первое объяснение))
Виталий выше правильно написал, только чтобы понять, пришлось перелопатить документацию, по ссылке выше.

Чтобы работали страницы в категориях, надо все запихать в if

views.py

if category_slug:
        category = get_object_or_404(Category, slug=category_slug)
        products = products.filter(category=category)
        paginator = Paginator(products, 3)  # 12 posts in each page
        page = request.GET.get('page')

        try:
            products = paginator.page(page)
        except PageNotAnInteger:
            # If page is not an integer deliver the first page
            products = paginator.page(1)
        except EmptyPage:
            # If page is out of range deliver last page of results
            products = paginator.page(paginator.num_pages)

Comments

[Ram Ce]

в таком случае работают и категории и пагинация, но когда выбрана какая-то категория, все равно показываются все товары, даже не из этой категории

пока что код у меня выглядит так:

def product_list(request, category_slug=None):
    category = None
    categories = Category.objects.all()
    products = Product.objects.filter(available=True)
    page = request.GET.get('page')
    paginator = Paginator(products, 3)

    if category_slug:
        category = get_object_or_404(Category, slug=category_slug)
        products = products.filter(category=category)

    try:
        products = paginator.page(page)
    except PageNotAnInteger:
        products = paginator.page(1)
    except EmptyPage:
        products = paginator.page(paginator.num_pages)

    return render(request,
                'products/list.html',
                {'category': category,
                'categories': categories,
                'products': products,
                'page': page})

[neqrotek]

Ram Ce, проблема не в этом коде, а в представлении pagination.html, main.html

и тем более, эта часть

try:
        products = paginator.page(page)
    except PageNotAnInteger:
        products = paginator.page(1)
    except EmptyPage:
        products = paginator.page(paginator.num_pages)

    return render(request,
                'products/list.html',
                {'category': category,
                'categories': categories,
                'products': products,
                'page': page})

должна находится в if
Если не будет работать, можно запихать в if только

spoiler

page = request.GET.get('page')
    paginator = Paginator(products, 3)

[Ram Ce]

neqrotek, в обоих случаях The view products.views.product_list didn't return an HttpResponse object. It returned None instead.

[neqrotek]

а в urls:

urls.py

from django.urls import path
from . import views

urlpatterns = [
    path('', views.product_list, name='product_list'),
    path('category/<category_slug>/', views.product_list, name='product_list_by_category')
    path('product/<slug:slug>/<int:id>/', views.product_detail, name='product_detail')
]

'page': page должен идти до products.

А вообще ошибка могла из любой части view вылезти, откуда там вообще httpResponse?

[Ram Ce]

neqrotek, пагинация работает только если выбрана категория и все равно показываются все товары, даже не из выбранной категории. измучался уже, все перепробовал. ошибок вроде нигде как таковых нет (хотя скорее всего я упускаю что-то)