diff options
| author | Jocelyn Delalande <jocelyn@crapouillou.net> | 2018-02-03 18:04:06 +0100 |
|---|---|---|
| committer | Jocelyn Delalande <jocelyn@crapouillou.net> | 2018-02-07 00:01:13 +0100 |
| commit | b1a4572e8c72e1d7f49b07aaeb5be0f3603bf0a7 (patch) | |
| tree | d97e0c6211ca1d408af3e346a60cad1a65ccdca0 /ihatemoney/templates/statistics.html | |
| parent | 389c7b8bcd2813d8549858265432859259942fd6 (diff) | |
| download | ihatemoney-mirror-b1a4572e8c72e1d7f49b07aaeb5be0f3603bf0a7.zip ihatemoney-mirror-b1a4572e8c72e1d7f49b07aaeb5be0f3603bf0a7.tar.gz ihatemoney-mirror-b1a4572e8c72e1d7f49b07aaeb5be0f3603bf0a7.tar.bz2 | |
Change statistics data structure
Clearer data structure, and simpler template
This commit has a side effect: sidebar now hides disabled members.
IMHO, the disabled members should either be hidden or shown consistently between
sidebar and central table. Previous status was: shown in sidebar (if balance ≠
0) and hidden in central table.
Diffstat (limited to 'ihatemoney/templates/statistics.html')
| -rw-r--r-- | ihatemoney/templates/statistics.html | 19 |
1 files changed, 9 insertions, 10 deletions
diff --git a/ihatemoney/templates/statistics.html b/ihatemoney/templates/statistics.html index ee59cb9..1b07a33 100644 --- a/ihatemoney/templates/statistics.html +++ b/ihatemoney/templates/statistics.html @@ -3,12 +3,11 @@ {% block sidebar %} <div id="table_overflow"> <table class="balance table"> - {% set balance = g.project.balance %} - {% for member in g.project.members | sort(attribute='name') if member.activated or balance[member.id]|round(2) != 0 %} + {% for stat in members_stats| sort(attribute='member.name') %} <tr> - <td class="balance-name">{{ member.name }}</td> - <td class="balance-value {% if balance[member.id]|round(2) > 0 %}positive{% elif balance[member.id]|round(2) < 0 %}negative{% endif %}"> - {% if balance[member.id]|round(2) > 0 %}+{% endif %}{{ "%.2f" | format(balance[member.id]) }} + <td class="balance-name">{{ stat.member.name }}</td> + <td class="balance-value {% if stat.balance|round(2) > 0 %}positive{% elif stat.balance|round(2) < 0 %}negative{% endif %}"> + {% if stat.balance|round(2) > 0 %}+{% endif %}{{ "%.2f" | format(stat.balance) }} </td> </tr> {% endfor %} @@ -21,12 +20,12 @@ <table id="bill_table" class="split_bills table table-striped"> <thead><tr><th>{{ _("Who?") }}</th><th>{{ _("Paid") }}</th><th>{{ _("Spent") }}</th><th>{{ _("Balance") }}</th></tr></thead> <tbody> - {% for member in members %} + {% for stat in members_stats %} <tr> - <td>{{ member.name }}</td> - <td>{{ "%0.2f"|format(paid[member.id]) }}</td> - <td>{{ "%0.2f"|format(spent[member.id]) }}</td> - <td>{{ "%0.2f"|format(balance[member.id]) }}</td> + <td>{{ stat.member.name }}</td> + <td>{{ "%0.2f"|format(stat.paid) }}</td> + <td>{{ "%0.2f"|format(stat.spent) }}</td> + <td>{{ "%0.2f"|format(stat.balance) }}</td> </tr> {% endfor %} </tbody> |
