database/migrations/structure/20230421_2234_currencies.sql

@@ -1,10 +0,0 @@
-CREATE TABLE `currencies` (
-  `id` int(10) unsigned NOT NULL AUTO_INCREMENT,
-  `community_id` int(10) unsigned NOT NULL,
-  `code` varchar(3) NOT NULL,
-  `round_digits` tinyint(1) unsigned NOT NULL,
-  PRIMARY KEY (`id`),
-  UNIQUE KEY `unique_currency_for_community` (`community_id`, `code`),
-  KEY `community_id` (`community_id`),
-  CONSTRAINT `currencies_community_id` FOREIGN KEY (`community_id`) REFERENCES `communities` (`id`)
-) ENGINE = InnoDB DEFAULT CHARSET = utf8mb4;

public/static/css/rvr.css

@@ -459,14 +459,6 @@ table th, table td {
     vertical-align: middle;
 }
 
-table th:not(:first-child), table td:not(:first-child) {
-    padding-left: 3px;
-}
-
-table th:not(:last-child), table td:not(:last-child) {
-    padding-right: 3px;
-}
-
 .choices__inner {
     box-sizing: border-box;
 }

scripts/gitHooks/prepare-commit-msg

@@ -7,7 +7,7 @@ if [[ "${BRANCH_NAME}" =~ $BRANCH_PATTERN ]]; then
     TICKET_ID=$(echo $BRANCH_NAME | sed -E "s@$BRANCH_PATTERN@\\2@")
 
     COMMIT_MESSAGE=$(head -n 1 $1)
-    COMMIT_MESSAGE_REGEX="^(fixup! )?$TICKET_ID .*"
+    COMMIT_MESSAGE_REGEX="^$TICKET_ID .*"
 
     if [[ ! "${COMMIT_MESSAGE}" =~ $COMMIT_MESSAGE_REGEX ]]; then
         sed -i.bak -e "1s/^/$TICKET_ID /" $1

src/Controller/CommunityController.php

@@ -3,11 +3,9 @@
 use DateTime;
 use RVR\PersistentData\Model\Community;
 use RVR\PersistentData\Model\CommunityMember;
-use RVR\PersistentData\Model\Currency;
 use RVR\PersistentData\Model\User;
 use RVR\Repository\CommunityRepository;
 use RVR\Repository\CommunityMemberRepository;
-use RVR\Repository\CurrencyRepository;
 use RVR\Repository\UserRepository;
 use SokoWeb\Interfaces\Authentication\IAuthenticationRequired;
 use SokoWeb\Interfaces\Response\IContent;
@@ -22,14 +20,11 @@ class CommunityController implements IAuthenticationRequired
 
     private CommunityMemberRepository $communityMemberRepository;
 
-    private CurrencyRepository $currencyRepository;
-
     public function __construct()
     {
         $this->userRepository = new UserRepository();
         $this->communityRepository = new CommunityRepository();
         $this->communityMemberRepository = new CommunityMemberRepository();
-        $this->currencyRepository = new CurrencyRepository();
     }
 
     public function isAuthenticationRequired(): bool
@@ -43,15 +38,10 @@ class CommunityController implements IAuthenticationRequired
             return null;
         }
 
-        $currencyCodes = [];
-        foreach ($this->getCurrencies($community) as $currency) {
-            $currencyCodes[] = $currency->getCode();
-        }
-
         return new HtmlContent('communities/community', [
             'community' => $community,
             'members' => $this->getMembers($community),
-            'currencyCodes' => $currencyCodes,
+            'currencyNames' => [],
             'upcomingEvents' => [],
             'editPermission' => $ownCommunityMember->getOwner()
         ]);
@@ -94,6 +84,7 @@ class CommunityController implements IAuthenticationRequired
         return $members;
     }
 
+
     public function newMember(): ?IContent
     {
         if (!$this->checkPermission(\Container::$request->query('communityId'), true, $community, $ownCommunityMember)) {
@@ -148,82 +139,6 @@ class CommunityController implements IAuthenticationRequired
         return new JsonContent(['success' => true]);
     }
 
-    public function getCurrenciesEdit(): ?IContent
-    {
-        if (!$this->checkPermission(\Container::$request->query('communityId'), true, $community, $ownCommunityMember)) {
-            return null;
-        }
-
-        return new HtmlContent('communities/community_currencies', [
-            'community' => $community,
-            'currencies' => $this->getCurrencies($community)
-        ]);
-    }
-
-    private function getCurrencies(Community $community): array
-    {
-        $currencies = iterator_to_array($this->currencyRepository->getAllByCommunity($community));
-        usort($currencies, function($a, $b) {
-            return strnatcmp($a->getCode(), $b->getCode());
-        });
-        return $currencies;
-    }
-
-    public function newCurrency(): ?IContent
-    {
-        if (!$this->checkPermission(\Container::$request->query('communityId'), true, $community, $ownCommunityMember)) {
-            return null;
-        }
-
-        $code = \Container::$request->post('code');
-        if (strlen($code) === 0 || strlen($code) > 3) {
-            return new JsonContent([
-                'error' => ['errorText' => 'Please fill all required fields!']
-            ]);
-        }
-
-        $existingCurrency = $this->currencyRepository->getByCommunityAndCurrencyCode($community, $code);
-        if ($existingCurrency !== null) {
-            return new JsonContent([
-                'error' => ['errorText' => 'A currency with the same code exists for this community.']
-            ]);
-        }
-
-        $currency = new Currency();
-        $currency->setCommunity($community);
-        $currency->setCode($code);
-        $currency->setRoundDigits((int)\Container::$request->post('round_digits'));
-        \Container::$persistentDataManager->saveToDb($currency);
-
-        return new JsonContent(['success' => true]);
-    }
-
-    public function editCurrency(): ?IContent
-    {
-        if (!$this->checkPermission(\Container::$request->query('communityId'), true, $community, $ownCommunityMember)) {
-            return null;
-        }
-
-        $currency = $this->currencyRepository->getById(\Container::$request->query('currency_id'));
-        $currency->setCode(\Container::$request->post('code'));
-        $currency->setRoundDigits((int)\Container::$request->post('round_digits'));
-        \Container::$persistentDataManager->saveToDb($currency);
-
-        return new JsonContent(['success' => true]);
-    }
-
-    public function deleteCurrency(): ?IContent
-    {
-        if (!$this->checkPermission(\Container::$request->query('communityId'), true, $community, $ownCommunityMember)) {
-            return null;
-        }
-
-        $currency = $this->currencyRepository->getById(\Container::$request->query('currency_id'));
-        \Container::$persistentDataManager->deleteFromDb($currency);
-
-        return new JsonContent(['success' => true]);
-    }
-
     public function saveCommunity(): ?IContent
     {
         $communityId = \Container::$request->query('communityId');

src/PersistentData/Model/Currency.php

@@ -1,60 +0,0 @@
-<?php namespace RVR\PersistentData\Model;
-
-use SokoWeb\PersistentData\Model\Model;
-
-class Currency extends Model
-{
-    protected static string $table = 'currencies';
-
-    protected static array $fields = ['community_id', 'code', 'round_digits'];
-
-    protected static array $relations = ['community' => Community::class];
-
-    private ?Community $community = null;
-
-    private ?int $communityId = null;
-
-    private string $code = '';
-
-    private int $roundDigits = 0;
-
-    public function setCommunity(Community $community): void
-    {
-        $this->community = $community;
-    }
-
-    public function setCommunityId(int $communityId): void
-    {
-        $this->communityId = $communityId;
-    }
-
-    public function setCode(string $code): void
-    {
-        $this->code = $code;
-    }
-
-    public function setRoundDigits(int $roundDigits): void
-    {
-        $this->roundDigits = $roundDigits;
-    }
-
-    public function getCommunity(): ?Community
-    {
-        return $this->community;
-    }
-
-    public function getCommunityId(): ?int
-    {
-        return $this->communityId;
-    }
-
-    public function getCode(): string
-    {
-        return $this->code;
-    }
-
-    public function getRoundDigits(): int
-    {
-        return $this->roundDigits;
-    }
-}

src/Repository/CurrencyRepository.php

@@ -1,32 +0,0 @@
-<?php namespace RVR\Repository;
-
-use Generator;
-use Container;
-use RVR\PersistentData\Model\Community;
-use RVR\PersistentData\Model\Currency;
-use SokoWeb\Database\Query\Select;
-
-class CurrencyRepository
-{
-    public function getById(int $id): ?Currency
-    {
-        return \Container::$persistentDataManager->selectFromDbById($id, Currency::class);
-    }
-
-    public function getAllByCommunity(Community $community, bool $useRelations = false): Generator
-    {
-        $select = new Select(Container::$dbConnection);
-        $select->where('community_id', '=', $community->getId());
-
-        yield from Container::$persistentDataManager->selectMultipleFromDb($select, Currency::class, $useRelations);
-    }
-
-    public function getByCommunityAndCurrencyCode(Community $community, string $code): ?Currency
-    {
-        $select = new Select(Container::$dbConnection);
-        $select->where('community_id', '=', $community->getId());
-        $select->where('code', '=', $code);
-
-        return Container::$persistentDataManager->selectFromDb($select, Currency::class);
-    }
-}

views/communities/community.php

@@ -17,13 +17,7 @@
         <div>
             <h3 class="marginBottom">Currencies</h3>
             <p>Main currency: <b><?= $community->getCurrency() ?></b></p>
-            <?php if (count($currencyCodes) > 0): ?>
+            <p>Further currencies: <b><?= implode(', ', $currencyNames) ?></b></p>
-                <p>Further currencies: <b><?= implode(', ', $currencyCodes) ?></b></p>
-            <?php endif; ?>
-            <?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>

views/communities/community_currencies.php

@@ -1,44 +0,0 @@
-@extends(templates/layout_normal)
-
-@section(main)
-    <h2><?= $community->getName() ?> - Edit currencies</h2>
-    <div class="box">
-        <table class="fullWidth">
-            <thead>
-                <tr>
-                    <th style="width: calc(100% / 3); text-align: left;">Currency</th>
-                    <th style="width: calc(100% / 3); text-align: left;">Round digits</th>
-                    <th style="width: calc(100% / 3);"></th>
-                </tr>
-            </thead>
-            <?php foreach ($currencies as $currency): ?>
-                <tr>
-                    <td>
-                        <form id="editCurrency_<?= $currency->getId() ?>" action="<?= Container::$routeCollection->getRoute('community-currencies-edit')->generateLink(['communityId' => $community->getId(), 'currency_id' => $currency->getId()]) ?>" method="post" data-reload-on-success="true" data-observe-inputs="currency,round_digits"></form>
-                        <form id="deleteCurrency_<?= $currency->getId() ?>" action="<?= Container::$routeCollection->getRoute('community-currencies-delete')->generateLink(['communityId' => $community->getId(), 'currency_id' => $currency->getId()]) ?>" method="post" data-reload-on-success="true"></form>
-                        <input type="text" form="editCurrency_<?= $currency->getId() ?>" class="text fullWidth" name="code" value="<?= $currency->getCode() ?>" maxlength="3" required>
-                    </td>
-                    <td>
-                        <input type="number" form="editCurrency_<?= $currency->getId() ?>" class="text fullWidth" name="round_digits" value="<?= $currency->getRoundDigits() ?>" min="0" max="9" required>
-                    </td>
-                    <td style="text-align: right;">
-                        <button type="submit" form="editCurrency_<?= $currency->getId() ?>" name="submit" class="small marginRight" disabled>Save</button><!--
-                     --><button type="submit" form="deleteCurrency_<?= $currency->getId() ?>" class="small red">Delete</button>
-                    </td>
-                </tr>
-            <?php endforeach; ?>
-            <tr>
-                <td>
-                    <form id="newCurrency" action="<?= Container::$routeCollection->getRoute('community-currencies-new')->generateLink(['communityId' => $community->getId()]) ?>" method="post" data-reload-on-success="true" data-observe-inputs="currency,round_digits"></form>
-                    <input type="text" form="newCurrency" class="text fullWidth" name="code" placeholder="Currency" maxlength="3" required>
-                </td>
-                <td>
-                    <input type="number" form="newCurrency" class="text fullWidth" name="round_digits" placeholder="Round digits" min="0" max="9" required>
-                </td>
-                <td style="text-align: right;">
-                    <button type="submit" form="newCurrency" name="submit" class="small" disabled>Add</button>
-                </td>
-            </tr>
-        </table>
-    </div>
-@endsection

views/communities/community_edit.php

@@ -11,7 +11,7 @@
         <form id="communityForm" action="<?= $formAction ?>" method="post" data-redirect-on-success="true">
             <input type="text" class="text big fullWidth" name="name" placeholder="Name" value="<?= isset($community) ? $community->getName() : '' ?>" required>
             <input type="text" class="text big fullWidth marginTop" name="currency" value="<?= isset($community) ? $community->getCurrency() : '' ?>" placeholder="Default currency" maxlength="3" required>
-            <p id="communityFormError" class="formError justify marginTop"></p>
+            <p id="accountFormError" class="formError justify marginTop"></p>
             <div class="right marginTop">
                 <button type="submit" name="submit"><?= isset($community) ? 'Save' : 'Create' ?></button>
             </div>

views/communities/community_members.php

@@ -29,18 +29,18 @@
                     <td style="text-align: right;">
                         <?php if ($editable): ?>
                             <button type="submit" form="editMember_<?= $member->getId() ?>" name="submit" class="small marginRight" disabled>Save</button><!--
-                         --><button type="submit" form="deleteMember_<?= $member->getId() ?>" class="small red">Delete</button>
+                         --><button type="submit" form="deleteMember_<?= $member->getId() ?>" class="small red delete_member" data-action="<?= Container::$routeCollection->getRoute('community-members-delete')->generateLink(['communityId' => $community->getId(), 'community_member_id' => $member->getId()]) ?>">Delete</button>
                         <?php endif; ?>
                     </td>
                 </tr>
             <?php endforeach; ?>
             <tr>
                 <td>
-                    <form id="newMember" action="<?= Container::$routeCollection->getRoute('community-members-new')->generateLink(['communityId' => $community->getId()]) ?>" method="post" data-reload-on-success="true" data-observe-inputs="user_id"></form>
+                <form id="newMember" action="<?= Container::$routeCollection->getRoute('community-members-new')->generateLink(['communityId' => $community->getId()]) ?>" method="post" data-reload-on-success="true" data-observe-inputs="user_id"></form>
                     <select type="text" form="newMember" name="user_id">
                 </td>
                 <td style="text-align: center;">
-                    <input type="checkbox" form="newMember" name="owner" />
+                <input type="checkbox" form="newMember" name="owner" />
                 </td>
                 <td style="text-align: right;">
                     <button type="submit" form="newMember" name="submit" class="small" disabled>Add</button>

web.php

@@ -66,10 +66,6 @@ Container::$routeCollection->group('communities', function (RouteCollection $rou
         $routeCollection->post('community-members-new', 'newMember', [CommunityController::class, 'newMember']);
         $routeCollection->post('community-members-edit', 'editMember', [CommunityController::class, 'editMember']);
         $routeCollection->post('community-members-delete', 'deleteMember', [CommunityController::class, 'deleteMember']);
-        $routeCollection->get('community-currencies', 'currencies', [CommunityController::class, 'getCurrenciesEdit']);
-        $routeCollection->post('community-currencies-new', 'newCurrency', [CommunityController::class, 'newCurrency']);
-        $routeCollection->post('community-currencies-edit', 'editCurrency', [CommunityController::class, 'editCurrency']);
-        $routeCollection->post('community-currencies-delete', 'deleteCurrency', [CommunityController::class, 'deleteCurrency']);
     });
 });