From 391c4219a282372325af87944d8b78b8b19bee59 Mon Sep 17 00:00:00 2001
From: MilouH <m.m.l.s.henzen@student.tue.nl>
Date: Thu, 3 Apr 2025 11:13:44 +0200
Subject: [PATCH] More whiteline reversions and re-adding old comment

---
 ihatemoney/templates/list_bills.html | 18 +++++++++---------
 ihatemoney/web.py                    |  3 +++
 2 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/ihatemoney/templates/list_bills.html b/ihatemoney/templates/list_bills.html
index d570f1a3..be98b445 100644
--- a/ihatemoney/templates/list_bills.html
+++ b/ihatemoney/templates/list_bills.html
@@ -94,15 +94,15 @@
 <div class="d-flex flex-wrap w-100 pt-2 mt-2" id="bill-toolbar">
 {% if bills.pages > 1 %}
 <ul class="pagination mr-2 mb-0 pb-2 flex-wrap" id="pagination-top">
-    <li class="page-item {% if bills.page == 1 %}disabled{% endif %}"><a class="page-link" href="{{ url_for('main.list_bills', page=bills.prev_num) }}">&laquo; {{ _("Newer bills") }}</a></li>
-    {%- for page in bills.iter_pages() %}
-    {% if page %}
-    <li class="page-item {% if page == bills.page %}active{% endif %}"><a class="page-link" href="{{ url_for('main.list_bills', page=page) }}">{{ page }}</a></li>
-    {% else %}
-    <li class="page-item disabled"><span class="ellipsis page-link">…</span></li>
-    {% endif %}
-    {%- endfor %}
-    <li class="page-item {% if bills.page == bills.pages %}disabled{% endif %}"><a class="page-link" href="{{ url_for('main.list_bills', page=bills.next_num) }}">{{ _("Older bills") }} &raquo;</a></li>
+  <li class="page-item {% if bills.page == 1 %}disabled{% endif %}"><a class="page-link" href="{{ url_for('main.list_bills', page=bills.prev_num) }}">&laquo; {{ _("Newer bills") }}</a></li>
+  {%- for page in bills.iter_pages() %}
+  {% if page %}
+  <li class="page-item {% if page == bills.page %}active{% endif %}"><a class="page-link" href="{{ url_for('main.list_bills', page=page) }}">{{ page }}</a></li>
+  {% else %}
+  <li class="page-item disabled"><span class="ellipsis page-link">…</span></li>
+  {% endif %}
+  {%- endfor %}
+  <li class="page-item {% if bills.page == bills.pages %}disabled{% endif %}"><a class="page-link" href="{{ url_for('main.list_bills', page=bills.next_num) }}">{{ _("Older bills") }} &raquo;</a></li>
 </ul>
 {% endif %}
 
diff --git a/ihatemoney/web.py b/ihatemoney/web.py
index b7f780d9..6f404a8f 100644
--- a/ihatemoney/web.py
+++ b/ihatemoney/web.py
@@ -685,6 +685,9 @@ def list_bills():
 
     # standard query if no filters are active
     if not filters_active:
+        # Each item will be a (weight_sum, Bill) tuple.
+        # TODO: improve this awkward result using column_property:
+        # https://docs.sqlalchemy.org/en/14/orm/mapped_sql_expr.html.
         weighted_bills = g.project.get_bill_weights_ordered().paginate(per_page=100, error_out=True)
     else:
         query = Bill.query.join(Person).filter(Person.project_id == g.project.id)