src/Controller/CommunityController.php

@@ -53,44 +53,32 @@ class CommunityController implements IAuthenticationRequired
 
         $balanceCalculator = new BalanceCalculator($community);
         $debts = $balanceCalculator->calculate();
-        $debtItems = [];
+        $debtUsers = [];
         $debtBalance = 0.0;
-        $outstandingItems = [];
+        $outstandingUsers = [];
         $outstandingBalance = 0.0;
         foreach ($debts as $debt) {
             if ($debt['payer']->getId() === \Container::$request->user()->getUniqueId()) {
                 $debtBalance += $debt['amount'];
-                $debtItems[] = $debt;
+                $debtUsers[] = $debt;
             }
             if ($debt['payee']->getId() === \Container::$request->user()->getUniqueId()) {
                 $outstandingBalance += $debt['amount'];
-                $outstandingItems[] = $debt;
+                $outstandingUsers[] = $debt;
             }
         }
         $balance = $outstandingBalance - $debtBalance;
 
         return new HtmlContent('communities/community', [
-            'community' => $community,
-            'upcomingEvents' => [],
-            'debtItems' => $debtItems,
-            'debtBalance' => $debtBalance,
-            'outstandingItems' => $outstandingItems,
-            'outstandingBalance' => $outstandingBalance,
-            'balance' => $balance,
-            'editPermission' => $ownCommunityMember->getOwner()
-        ]);
-    }
-
-    public function getCommunitySettings(): ?IContent
-    {
-        if (!$this->checkPermission(\Container::$request->query('communityId'), false, $community, $ownCommunityMember)) {
-            return null;
-        }
-
-        return new HtmlContent('communities/community_settings', [
             'community' => $community,
             'members' => $this->getMembers($community),
             'currencies' => $this->getCurrencies($community),
+            'upcomingEvents' => [],
+            'debtUsers' => $debtUsers,
+            'debtBalance' => $debtBalance,
+            'outstandingUsers' => $outstandingUsers,
+            'outstandingBalance' => $outstandingBalance,
+            'balance' => $balance,
             'editPermission' => $ownCommunityMember->getOwner()
         ]);
     }

src/Controller/TransactionController.php

@@ -86,7 +86,6 @@ class TransactionController implements IAuthenticationRequired, ISecured
             'exchangeRateCalculator' => $exchangeRateCalculator,
             'pages' => $pages,
             'currentPage' => $currentPage,
-            'numberOfTransactions' => $numberOfTransactions,
             'transactions' => $transactions
         ]);
     }

views/communities/community.php

@@ -1,16 +1,35 @@
 @extends(templates/layout_normal)
 
 @section(main)
-    <h2>
+    <h2><?= $community->getName() ?> <span class="small">[<a href="<?= Container::$routeCollection->getRoute('community-edit')->generateLink(['communityId' => $community->getId()]) ?>">edit</a>]</span></h2>
-        <?= $community->getName() ?>
-        <?php if ($editPermission): ?>
-            <span class="small">[<a href="<?= Container::$routeCollection->getRoute('community-edit')->generateLink(['communityId' => $community->getId()]) ?>">edit</a>]</span>
-        <?php endif; ?>
-    </h2>
-
-    <p class="marginTop"><a href="<?= Container::$routeCollection->getRoute('community.settings')->generateLink(['communityId' => $community->getId()]) ?>">Settings</a></p>
 
     <div class="gridContainer marginTop">
+        <div>
+            <h3 class="marginBottom">Members</h3>
+            <?php foreach ($members as $member): ?>
+                <p><?= $member->getUser()->getDisplayName() ?></p>
+            <?php endforeach; ?>
+            <?php if ($editPermission): ?>
+                <hr>
+                <p><a href="<?= Container::$routeCollection->getRoute('community-members')->generateLink(['communityId' => $community->getId()]) ?>">Edit members</a></p>
+            <?php endif; ?>
+        </div>
+        <div>
+            <h3 class="marginBottom">Currencies</h3>
+            <?php foreach ($currencies as $currency): ?>
+                <p>
+                    <?php if ($currency->getId() === $community->getMainCurrencyId()): ?>
+                        <b><?= $currency->getCode() ?></b>
+                    <?php else: ?>
+                        <a href="<?= Container::$routeCollection->getRoute('community-currency-exchange-rates')->generateLink(['communityId' => $community->getId(), 'code' => $currency->getCode()]) ?>"><?= $currency->getCode() ?></a>
+                    <?php endif; ?>
+                </p>
+            <?php endforeach; ?>
+            <?php if ($editPermission): ?>
+                <hr>
+                <p><a href="<?= Container::$routeCollection->getRoute('community-currencies')->generateLink(['communityId' => $community->getId()]) ?>">Edit currencies</a></p>
+            <?php endif; ?>
+        </div>
         <div>
             <h3 class="marginBottom">Upcoming events</h3>
             <?php if (count($upcomingEvents) > 0): ?>
@@ -33,20 +52,20 @@
                     <td class="bold">You owe</td>
                     <td class="mono red" style="text-align: right;"><?= number_format($debtBalance, $mainCurrencyRoundDigits) ?> <?= $mainCurrencyCode ?></td>
                 </tr>
-                <?php foreach ($debtItems as $item): ?>
+                <?php foreach ($debtUsers as $owe): ?>
                     <tr>
-                        <td class="small"><?= $item['payee']->getUser()->getDisplayName() ?></td>
+                        <td class="small"><?= $owe['payee']->getUser()->getDisplayName() ?></td>
-                        <td class="small mono red" style="text-align: right;"><?= number_format($item['amount'], $mainCurrencyRoundDigits) ?> <?= $mainCurrencyCode ?></td>
+                        <td class="small mono red" style="text-align: right;"><?= number_format($owe['amount'], $mainCurrencyRoundDigits) ?> <?= $mainCurrencyCode ?></td>
                     </tr>
                 <?php endforeach; ?>
                 <tr>
                     <td class="bold">You're owed</td>
                     <td class="mono green" style="text-align: right;"><?= number_format($outstandingBalance, $mainCurrencyRoundDigits) ?> <?= $mainCurrencyCode ?></td>
                 </tr>
-                <?php foreach ($outstandingItems as $item): ?>
+                <?php foreach ($outstandingUsers as $owe): ?>
                     <tr>
-                        <td class="small"><?= $item['payer']->getUser()->getDisplayName() ?></td>
+                        <td class="small"><?= $owe['payer']->getUser()->getDisplayName() ?></td>
-                        <td class="small mono green" style="text-align: right;"><?= number_format($item['amount'], $mainCurrencyRoundDigits) ?> <?= $mainCurrencyCode ?></td>
+                        <td class="small mono green" style="text-align: right;"><?= number_format($owe['amount'], $mainCurrencyRoundDigits) ?> <?= $mainCurrencyCode ?></td>
                     </tr>
                 <?php endforeach; ?>
                 <tr>

views/communities/community_currencies.php

@@ -3,7 +3,6 @@
 @section(main)
     <h2>
         <a href="<?= Container::$routeCollection->getRoute('community')->generateLink(['communityId' => $community->getId()]) ?>"><?= $community->getName() ?></a> »
-        <a href="<?= Container::$routeCollection->getRoute('community.settings')->generateLink(['communityId' => $community->getId()]) ?>">Settings</a> »
         Edit currencies
     </h2>
     <div class="box compactBox">

views/communities/community_members.php

@@ -7,7 +7,6 @@
 @section(main)
     <h2>
         <a href="<?= Container::$routeCollection->getRoute('community')->generateLink(['communityId' => $community->getId()]) ?>"><?= $community->getName() ?></a> »
-        <a href="<?= Container::$routeCollection->getRoute('community.settings')->generateLink(['communityId' => $community->getId()]) ?>">Settings</a> »
         Edit members
     </h2>
     <div class="box compactBox">

views/communities/community_settings.php

@@ -1,39 +0,0 @@
-@extends(templates/layout_normal)
-
-@section(main)
-    <h2>
-        <a href="<?= Container::$routeCollection->getRoute('community')->generateLink(['communityId' => $community->getId()]) ?>"><?= $community->getName() ?></a> »
-        Settings
-    </h2>
-
-    <div class="gridContainer marginTop">
-        <div>
-            <h3 class="marginBottom">
-                Members
-                <?php if ($editPermission): ?>
-                    <span class="small">[<a href="<?= Container::$routeCollection->getRoute('community-members')->generateLink(['communityId' => $community->getId()]) ?>">edit</a>]</span>
-                <?php endif; ?>
-            </h3>
-            <?php foreach ($members as $member): ?>
-                <p><?= $member->getUser()->getDisplayName() ?></p>
-            <?php endforeach; ?>
-        </div>
-        <div>
-            <h3 class="marginBottom">
-                Currencies
-                <?php if ($editPermission): ?>
-                    <span class="small">[<a href="<?= Container::$routeCollection->getRoute('community-currencies')->generateLink(['communityId' => $community->getId()]) ?>">edit</a>]</span>
-                <?php endif; ?>
-            </h3>
-            <?php foreach ($currencies as $currency): ?>
-                <p>
-                    <?php if ($currency->getId() === $community->getMainCurrencyId()): ?>
-                        <b><?= $currency->getCode() ?></b>
-                    <?php else: ?>
-                        <a href="<?= Container::$routeCollection->getRoute('community-currency-exchange-rates')->generateLink(['communityId' => $community->getId(), 'code' => $currency->getCode()]) ?>"><?= $currency->getCode() ?></a>
-                    <?php endif; ?>
-                </p>
-            <?php endforeach; ?>
-        </div>
-    </div>
-@endsection

views/communities/currency_exchange_rates.php

@@ -3,7 +3,6 @@
 @section(main)
     <h2>
         <a href="<?= Container::$routeCollection->getRoute('community')->generateLink(['communityId' => $community->getId()]) ?>"><?= $community->getName() ?></a> »
-        <a href="<?= Container::$routeCollection->getRoute('community.settings')->generateLink(['communityId' => $community->getId()]) ?>">Settings</a> »
         Exchange rates for <?= $currency->getCode() ?>
     </h2>
     <div class="box compactBox">

views/communities/transactions.php

@@ -8,61 +8,55 @@
 
     <p class="marginTop"><a href="<?= Container::$routeCollection->getRoute('community.transactions.new')->generateLink(['communityId' => $community->getId()]) ?>">New transaction</a></p>
 
-    <?php if ($numberOfTransactions > 0): ?>
+    <?php if ($pages > 1): ?>
-        <?php if ($pages > 1): ?>
+        <p class="paginateContainer marginTop">
-            <p class="paginateContainer marginTop">
+            <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => 0]) ?>">«</a>
-                <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => 0]) ?>">«</a>
+            <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => max(0, $currentPage - 1)]) ?>">‹</a>
-                <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => max(0, $currentPage - 1)]) ?>">‹</a>
+            <?php for ($i = 0; $i < $pages; $i++): ?>
-                <?php for ($i = 0; $i < $pages; $i++): ?>
+                <?php if ($currentPage == $i): ?>
-                    <?php if ($currentPage == $i): ?>
+                    <span class="selected"><?= $i + 1 ?></span>
-                        <span class="selected"><?= $i + 1 ?></span>
+                <?php else: ?>
-                    <?php else: ?>
+                    <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => $i]) ?>"><?= $i + 1 ?></a>
-                        <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => $i]) ?>"><?= $i + 1 ?></a>
+                <?php endif; ?>
-                    <?php endif; ?>
+            <?php endfor; ?>
-                <?php endfor; ?>
+            <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => min($pages - 1, $currentPage + 1)]) ?>">›</a>
-                <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => min($pages - 1, $currentPage + 1)]) ?>">›</a>
+            <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => $pages - 1]) ?>">»</a>
-                <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => $pages - 1]) ?>">»</a>
+        </p>
-            </p>
-        <?php endif; ?>
-
-        <?php foreach ($transactions as $transaction): ?>
-            <a class="block" href="<?= Container::$routeCollection->getRoute('community.transactions.edit')->generateLink(['communityId' => $community->getId(), 'transactionId' => $transaction->getId()]) ?>">
-            <div class="box transaction">
-                <div>
-                    <p style="font-weight: bold;"><?= $transaction->getDescription() ?></p>
-                    <p class="small"><?= $transaction->getPayerUser()->getDisplayName() ?> ► <?= $transaction->getPayeeUser() ? $transaction->getPayeeUser()->getDisplayName() : '[common]' ?></p>
-                    <p class="small"><?= $transaction->getTimeDate()->format('Y-m-d H:i') ?></p>
-                </div>
-                <div style="text-align: right;">
-                    <h3><?= number_format($exchangeRateCalculator->calculate($transaction->getSum(), $transaction->getCurrency(), $transaction->getTimeDate()), $community->getMainCurrency()->getRoundDigits()) ?> <?= $community->getMainCurrency()->getCode() ?></h3>
-                    <?php if ($community->getMainCurrencyId() !== $transaction->getCurrencyId()): ?>
-                        <p style="color: #8e8e8e; font-weight: bold;"><?= number_format($transaction->getSum(), $transaction->getCurrency()->getRoundDigits()) ?> <?= $transaction->getCurrency()->getCode() ?></p>
-                    <?php endif; ?>
-                </div>
-            </div>
-            </a>
-        <?php endforeach; ?>
-
-        <?php if ($pages > 1): ?>
-            <p class="paginateContainer marginTop">
-                <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => 0]) ?>">«</a>
-                <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => max(0, $currentPage - 1)]) ?>">‹</a>
-                <?php for ($i = 0; $i < $pages; $i++): ?>
-                    <?php if ($currentPage == $i): ?>
-                        <span class="selected"><?= $i + 1 ?></span>
-                    <?php else: ?>
-                        <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => $i]) ?>"><?= $i + 1 ?></a>
-                    <?php endif; ?>
-                <?php endfor; ?>
-                <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => min($pages - 1, $currentPage + 1)]) ?>">›</a>
-                <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => $pages - 1]) ?>">»</a>
-            </p>
-        <?php endif; ?>
-
-        <p class="marginTop"><a href="<?= Container::$routeCollection->getRoute('community.transactions.new')->generateLink(['communityId' => $community->getId()]) ?>">New transaction</a></p>
-    <?php else: ?>
-        <div class="box">
-            <p>There are no transactions yet.</p>
-        </div>
     <?php endif; ?>
+
+    <?php foreach ($transactions as $transaction): ?>
+        <a class="block" href="<?= Container::$routeCollection->getRoute('community.transactions.edit')->generateLink(['communityId' => $community->getId(), 'transactionId' => $transaction->getId()]) ?>">
+        <div class="box transaction">
+            <div>
+                <p style="font-weight: bold;"><?= $transaction->getDescription() ?></p>
+                <p class="small"><?= $transaction->getPayerUser()->getDisplayName() ?> ► <?= $transaction->getPayeeUser() ? $transaction->getPayeeUser()->getDisplayName() : '[common]' ?></p>
+                <p class="small"><?= $transaction->getTimeDate()->format('Y-m-d H:i') ?></p>
+            </div>
+            <div style="text-align: right;">
+                <h3><?= number_format($exchangeRateCalculator->calculate($transaction->getSum(), $transaction->getCurrency(), $transaction->getTimeDate()), $community->getMainCurrency()->getRoundDigits()) ?> <?= $community->getMainCurrency()->getCode() ?></h3>
+                <?php if ($community->getMainCurrencyId() !== $transaction->getCurrencyId()): ?>
+                    <p style="color: #8e8e8e; font-weight: bold;"><?= number_format($transaction->getSum(), $transaction->getCurrency()->getRoundDigits()) ?> <?= $transaction->getCurrency()->getCode() ?></p>
+                <?php endif; ?>
+            </div>
+        </div>
+        </a>
+    <?php endforeach; ?>
+
+    <?php if ($pages > 1): ?>
+        <p class="paginateContainer marginTop">
+            <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => 0]) ?>">«</a>
+            <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => max(0, $currentPage - 1)]) ?>">‹</a>
+            <?php for ($i = 0; $i < $pages; $i++): ?>
+                <?php if ($currentPage == $i): ?>
+                    <span class="selected"><?= $i + 1 ?></span>
+                <?php else: ?>
+                    <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => $i]) ?>"><?= $i + 1 ?></a>
+                <?php endif; ?>
+            <?php endfor; ?>
+            <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => min($pages - 1, $currentPage + 1)]) ?>">›</a>
+            <a href="<?= Container::$routeCollection->getRoute('community.transactions')->generateLink(['communityId' => $community->getId(), 'page' => $pages - 1]) ?>">»</a>
+        </p>
+    <?php endif; ?>
+
+    <p class="marginTop"><a href="<?= Container::$routeCollection->getRoute('community.transactions.new')->generateLink(['communityId' => $community->getId()]) ?>">New transaction</a></p>
 @endsection

views/home.php

@@ -3,10 +3,7 @@
 @section(main)
     <div class="gridContainer">
         <div>
-            <h3 class="marginBottom">
+            <h3 class="marginBottom">Communities</h3>
-                Communities
-                <span class="small">[<a href="<?= Container::$routeCollection->getRoute('community-new')->generateLink() ?>">new</a>]</span>
-            </h3>
             <?php if (count($communities) > 0): ?>
                 <?php foreach ($communities as $community): ?>
                     <p><a href="<?= Container::$routeCollection->getRoute('community')->generateLink(['communityId' => $community->getId()]) ?>"><?= $community->getName() ?></a></p>
@@ -14,6 +11,8 @@
             <?php else: ?>
                 <p>You have no community.</p>
             <?php endif; ?>
+            <hr>
+            <p><a href="<?= Container::$routeCollection->getRoute('community-new')->generateLink() ?>">New community</a></p>
         </div>
         <div>
             <h3 class="marginBottom">Upcoming events</h3>

web.php

@@ -61,7 +61,6 @@ Container::$routeCollection->group('communities', function (RouteCollection $rou
     $routeCollection->post('community-new-action', 'new', [CommunityController::class, 'saveCommunity']);
     $routeCollection->group('{communityId}', function (RouteCollection $routeCollection) {
         $routeCollection->get('community', '', [CommunityController::class, 'getCommunityHome']);
-        $routeCollection->get('community.settings', 'settings', [CommunityController::class, 'getCommunitySettings']);
         $routeCollection->get('community-edit', 'edit', [CommunityController::class, 'getCommunityEdit']);
         $routeCollection->post('community-edit-action', 'edit', [CommunityController::class, 'saveCommunity']);
         $routeCollection->get('community-members', 'members', [CommunityController::class, 'getMembersEdit']);