Search code examples
pythondjangodjango-pagination

Pagination in Django not working as expected


I wanted pagination for all the courses available and that was easy to achieve. But now I'm stuck because I wanted pagination for faculties also, which will show specific courses of the accessed faculty. I have 4 models: faculties, departments, studies, and courses. The pagination will show for faculties as well, but the problem is that if I try to go to the second page, it will redirect me to the second page of all courses list. Or, if I change page on all courses and then try to access a faculty, no course will show at all in the faculty.

def index(request):
    course_list = Course.objects.all()
    page = request.GET.get('page', 1)
    paginator = Paginator(course_list, 1)
    try:
        courses = paginator.page(page)
    except PageNotAnInteger:
        courses = paginator.page(1)
    except EmptyPage:
        courses = paginator.page(paginator.num_pages)
    faculty_list = Faculty.objects.all()
    page = request.GET.get('page2', 1)
    paginator = Paginator(faculty_list, 1)
    try:
        faculties = paginator.page(page)
    except PageNotAnInteger:
        faculties = paginator.page(1)
    except EmptyPage:
        faculties = paginator.page(paginator.num_pages)
    context = {'courses': courses,
               'faculties': faculties,
               'departments': Department.objects.all(),
               'studies': StudyProgramme.objects.all(),
               'teachers': Teacher.objects.all()
               }
    return render(request, 'courses/index.html', context)

<div id="crs">
    <h3>All courses</h3>
    <ul>
        {% for course in courses %}
            <li><a href={{ course.slug }}>{{ course.name }}</a></li>
        {% endfor %}
    </ul>
    <div class="pagination">
        <span class="step-links">
            {% if courses.has_previous %}
                <a href="?page=1">&laquo; first</a>
                <a href="?page={{ courses.previous_page_number }}">{{ courses.previous_page_number }}</a>
            {% endif %}
            <span class="current">
                Page {{ courses.number }} of {{ courses.paginator.num_pages }}
            </span>
            {% if courses.has_next %}
                <a href="?page={{ courses.next_page_number }}">{{ courses.next_page_number }}</a>
                <a href="?page={{ courses.paginator.num_pages }}">last &raquo;</a>

            {% endif %}
        </span>
    </div>
</div>
{% for faculty in faculties %}
    <div id="fac_{{ faculty.pk }}_tab" style="display:none;">
        <h3> {{ faculty.name }} Courses</h3>
        <ul>
            {% for department in faculty.department_set.all %}
                {% for study in studies %}
                    {% if study.department == department %}
                        {% for course in courses %}
                            {% if course.study_programme == study %}
                                <li>
                                    <a class="first" href={{ course.slug }}>{{ course.name }}</a>
                                </li>
                            {% endif %}
                        {% endfor %}
                    {% endif %}
                {% endfor %}
            {% endfor %}
        </ul>
        <div class="pagination">
            <span class="step-links">
                {% if faculties.has_previous %}
                    <a href="?page2=1">&laquo; first</a>
                    <a href="?page2={{ faculties.previous_page_number }}">{{ faculties.previous_page_number }}</a>
                {% endif %}
                <span class="current">
                    Page {{ faculties.number }} of {{ faculties.paginator.num_pages }}
                </span>
                {% if faculties.has_next %}
                    <a href="?page2={{ faculties.next_page_number }}">{{ faculties.next_page_number }}</a>
                    <a href="?page2={{ faculties.paginator.num_pages }}">last &raquo;</a>
                {% endif %}
            </span>
        </div>
    </div>
{% endfor %}


Solution

  • paginator = Paginator(course_list, 1)

    paginator = Paginator(faculty_list, 1)

    Changing your second instance name to paginator_two would solve your problem