Fix filtered pagination

This commit is contained in:
2022-07-18 22:48:04 +02:00
parent 503a214a4d
commit ad33731913
5 changed files with 8 additions and 14 deletions

View File

@@ -125,7 +125,7 @@
<td>{{ rolling_stock.sku }}</td>
</tr>
<tr>
<th scope="row">ERA</th>
<th scope="row">Era</th>
<td>{{ rolling_stock.era }}</td>
</tr>
<tr>

View File

@@ -10,7 +10,7 @@
<ul class="pagination justify-content-center mt-4">
{% if rolling_stock.has_previous %}
<li class="page-item">
<a class="page-link" href="{% url 'search_pagination' search=search page=rolling_stock.previous_page_number %}#rolling-stock" tabindex="-1">Previous</a>
<a class="page-link" href="{% url 'filtered_pagination' _filter=filter search=search page=rolling_stock.previous_page_number %}#rolling-stock" tabindex="-1">Previous</a>
</li>
{% else %}
<li class="page-item disabled">
@@ -26,13 +26,13 @@
{% if i == rolling_stock.paginator.ELLIPSIS %}
<li class="page-item"><span class="page-link">{{ i }}</span></li>
{% else %}
<li class="page-item"><a class="page-link" href="{% url 'search_pagination' search=search page=i %}#rolling-stock">{{ i }}</a></li>
<li class="page-item"><a class="page-link" href="{% url 'filtered_pagination' _filter=filter search=search page=i %}#rolling-stock">{{ i }}</a></li>
{% endif %}
{% endif %}
{% endfor %}
{% if rolling_stock.has_next %}
<li class="page-item">
<a class="page-link" href="{% url 'search_pagination' search=search page=rolling_stock.next_page_number %}#rolling-stock" tabindex="-1">Next</a>
<a class="page-link" href="{% url 'filtered_pagination' _filter=filter search=search page=rolling_stock.next_page_number %}#rolling-stock" tabindex="-1">Next</a>
</li>
{% else %}
<li class="page-item disabled">

View File

@@ -18,12 +18,6 @@ urlpatterns = [
GetHomeFiltered.as_view(http_method_names=["post"]),
name="search",
),
path("search/<str:search>", GetHomeFiltered.as_view(), name="search"),
path(
"search/<str:search>/<int:page>",
GetHomeFiltered.as_view(),
name="search_pagination",
),
path("consists", Consists.as_view(), name="consists"),
path(
"consists/<int:page>", Consists.as_view(), name="consists_pagination"

View File

@@ -50,7 +50,7 @@ class GetHome(View):
class GetHomeFiltered(View):
def run_search(self, request, search, _filter, page=1):
site_conf = get_site_conf()
if _filter is None:
if _filter == "search":
query = reduce(
operator.or_,
(
@@ -90,7 +90,7 @@ class GetHomeFiltered(View):
return rolling_stock, matches, page_range
def get(self, request, search, _filter=None, page=1):
def get(self, request, search, _filter="search", page=1):
rolling_stock, matches, page_range = self.run_search(
request, search, _filter, page
)
@@ -107,7 +107,7 @@ class GetHomeFiltered(View):
},
)
def post(self, request, _filter=None, page=1):
def post(self, request, _filter="search", page=1):
search = request.POST.get("search")
if not search:
raise Http404