From 1bf71e0f5322f158f454fa941901d15d68eb5db9 Mon Sep 17 00:00:00 2001
From: Phyks <webmaster@phyks.me>
Date: Fri, 4 Oct 2013 13:38:24 +0200
Subject: [PATCH] Simplified matrix for non admin users on index

The balance table on the index page is smaller for non admin users and
only displays necessary lines.
---
 TODO                      |  1 -
 tpl/default_en/index.html | 63 +++++++++++++++++++++++++++------------
 tpl/default_fr/index.html | 63 +++++++++++++++++++++++++++------------
 3 files changed, 88 insertions(+), 39 deletions(-)

diff --git a/TODO b/TODO
index b79437c..fb159af 100755
--- a/TODO
+++ b/TODO
@@ -1,4 +1,3 @@
-* Don't display the whole balance table if not admin
 * Notifications by e-mail for users
 
 Improvements :
diff --git a/tpl/default_en/index.html b/tpl/default_en/index.html
index 6a463a4..59f9d17 100755
--- a/tpl/default_en/index.html
+++ b/tpl/default_en/index.html
@@ -6,29 +6,54 @@
 
 <div id="quick_summary">
     <h2>Balance</h2>
-    <p class="center">Read <em>line</em> owes <em>case</em> {$currency} to <em>column</em>. You can click on links to confirm the payback.
-    <table id="balance_table"> 
-        <tr>
-            <th>Owes\To</th>
-            {loop="users"}
-            <th>{$value->getDisplayName()}</th>
-            {/loop}
-        </tr>
-        {loop="users"}
+    {if condition="$current_user->getAdmin()"}
+        <p class="center">Read <em>line</em> owes <em>case</em> {$currency} to <em>column</em>. You can click on links to confirm the payback.
+        <table id="balance_table"> 
             <tr>
-                <th>{$value->getDisplayName()}</th>
+                <th>Owes\To</th>
                 {loop="users"}
-                    {if condition="$balances[$value1->getId()][$value2->getId()] === 'X'"}
-                    <td class="cell-disabled"></td>
-                    {elseif condition="$balances[$value1->getId()][$value2->getId()] === '-' || ($value2->getId() != $current_user->getId() && !$current_user->getAdmin())"}
-                        <td>{$balances[$value1->getId()][$value2->getId()]}</td>
-                    {else}
-                        <td><a href="?do=payall&amp;from={$value1->getId()}&amp;to={$value2->getId()}&amp;token={$token}">{$balances[$value1->getId()][$value2->getId()]}</a></td>
-                    {/if}
+                <th>{$value->getDisplayName()}</th>
                 {/loop}
             </tr>
-        {/loop}
-    </table>
+            {loop="users"}
+                <tr>
+                    <th>{$value->getDisplayName()}</th>
+                    {loop="users"}
+                        {if condition="$balances[$value1->getId()][$value2->getId()] === 'X'"}
+                        <td class="cell-disabled"></td>
+                        {elseif condition="$balances[$value1->getId()][$value2->getId()] === '-' || ($value2->getId() != $current_user->getId() && !$current_user->getAdmin())"}
+                            <td>{$balances[$value1->getId()][$value2->getId()]}</td>
+                        {else}
+                            <td><a href="?do=payall&amp;from={$value1->getId()}&amp;to={$value2->getId()}&amp;token={$token}">{$balances[$value1->getId()][$value2->getId()]}</a></td>
+                        {/if}
+                    {/loop}
+                </tr>
+            {/loop}
+        </table>
+    {else}
+        <table>
+            <tr>
+                <th>Users</th>
+                <th>You owe him</th>
+                <th>He owes you</th>
+            </tr>
+            {loop="$users"}
+                {if condition="$balances[$value->getId()][$current_user->getId()] !== 'X' && $balances[$value->getId()][$current_user->getId()] !== '-'"}
+                    </tr>
+                        <td>{$value->getDisplayName()}</td>
+                        <td>-</td>
+                        <td>{$balances[$value->getId()][$current_user->getId()]}{$currency}</td>
+                    </tr>
+                {elseif condition="$balances[$current_user->getId()][$value->getId()] !== 'X' && $balances[$current_user->getId()][$value->getId()] !== '-'"}
+                    </tr>
+                        <td>{$value->getDisplayName()}</td>
+                        <td>{$balances[$current_user->getId()][$value->getId()]}{$currency}</td>
+                        <td>-</td>
+                    </tr>
+                {/if}
+            {/loop}
+        </table>
+    {/if}
 </div>
 <div id="detailed_summary">
     <h2>Detailed list of bills{if condition="$all == 0"} for last month{/if}</h2>
diff --git a/tpl/default_fr/index.html b/tpl/default_fr/index.html
index cce237f..17cc6da 100755
--- a/tpl/default_fr/index.html
+++ b/tpl/default_fr/index.html
@@ -6,29 +6,54 @@
 
 <div id="quick_summary">
     <h2>Bilan</h2>
-    <p class="center">Lire <em>ligne</em> doit <em>case</em> {$currency} à <em>colonne</em>. Vous pouvez cliquer sur les liens pour confirmer les remboursements.
-    <table id="balance_table"> 
-        <tr>
-            <th>Doit\À</th>
-            {loop="users"}
-            <th>{$value->getDisplayName()}</th>
-            {/loop}
-        </tr>
-        {loop="users"}
+    {if condition="$current_user->getAdmin()"}
+        <p class="center">Lire <em>ligne</em> doit <em>case</em> {$currency} à <em>colonne</em>. Vous pouvez cliquer sur les liens pour confirmer les remboursements.
+        <table id="balance_table"> 
             <tr>
-                <th>{$value->getDisplayName()}</th>
+                <th>Doit\À</th>
                 {loop="users"}
-                    {if condition="$balances[$value1->getId()][$value2->getId()] === 'X'"}
-                    <td class="cell-disabled"></td>
-                    {elseif condition="$balances[$value1->getId()][$value2->getId()] === '-' || ($value2->getId() != $current_user->getId() && !$current_user->getAdmin())"}
-                        <td>{$balances[$value1->getId()][$value2->getId()]}</td>
-                    {else}
-                        <td><a href="?do=payall&amp;from={$value1->getId()}&amp;to={$value2->getId()}&amp;token={$token}">{$balances[$value1->getId()][$value2->getId()]}</a></td>
-                    {/if}
+                <th>{$value->getDisplayName()}</th>
                 {/loop}
             </tr>
-        {/loop}
-    </table>
+            {loop="users"}
+                <tr>
+                    <th>{$value->getDisplayName()}</th>
+                    {loop="users"}
+                        {if condition="$balances[$value1->getId()][$value2->getId()] === 'X'"}
+                        <td class="cell-disabled"></td>
+                        {elseif condition="$balances[$value1->getId()][$value2->getId()] === '-' || ($value2->getId() != $current_user->getId() && !$current_user->getAdmin())"}
+                            <td>{$balances[$value1->getId()][$value2->getId()]}</td>
+                        {else}
+                            <td><a href="?do=payall&amp;from={$value1->getId()}&amp;to={$value2->getId()}&amp;token={$token}">{$balances[$value1->getId()][$value2->getId()]}</a></td>
+                        {/if}
+                    {/loop}
+                </tr>
+            {/loop}
+        </table>
+    {else}
+        <table>
+            <tr>
+                <th>Utilisateurs</th>
+                <th>Vous devez</th>
+                <th>Il vous doit</th>
+            </tr>
+            {loop="$users"}
+                {if condition="$balances[$value->getId()][$current_user->getId()] !== 'X' && $balances[$value->getId()][$current_user->getId()] !== '-'"}
+                    </tr>
+                        <td>{$value->getDisplayName()}</td>
+                        <td>-</td>
+                        <td>{$balances[$value->getId()][$current_user->getId()]}{$currency}</td>
+                    </tr>
+                {elseif condition="$balances[$current_user->getId()][$value->getId()] !== 'X' && $balances[$current_user->getId()][$value->getId()] !== '-'"}
+                    </tr>
+                        <td>{$value->getDisplayName()}</td>
+                        <td>{$balances[$current_user->getId()][$value->getId()]}{$currency}</td>
+                        <td>-</td>
+                    </tr>
+                {/if}
+            {/loop}
+        </table>
+    {/if}
 </div>
 <div id="detailed_summary">
     <h2>Liste détaillée des dépenses{if condition="$all == 0"} du dernier mois{/if}</h2>