RVRNEXT-5 formError should be searched in the whole document

public/static/js/rvr.js

@@ -49,7 +49,7 @@ var RVR = {
             document.getElementById('loading').style.visibility = 'visible';
 
             var formData = new FormData(form);
-            var formError = form.getElementsByClassName('formError')[0];
+            var formError = document.getElementsByClassName('formError')[0];
             var pageLeaveOnSuccess = form.dataset.redirectOnSuccess || form.dataset.reloadOnSuccess;
 
             RVR.httpRequest('POST', form.action, function () {

src/Controller/CommunityController.php

@@ -85,15 +85,6 @@ class CommunityController implements IAuthenticationRequired
         ]);
     }
 
-    private function getMembers(Community $community): array
-    {
-        $members = iterator_to_array($this->communityMemberRepository->getAllByCommunity($community, true));
-        usort($members, function($a, $b) {
-            return strnatcmp($a->getUser()->getDisplayName(), $b->getUser()->getDisplayName());
-        });
-        return $members;
-    }
-
     public function newMember(): ?IContent
     {
         if (!$this->checkPermission(\Container::$request->query('communityId'), true, $community, $ownCommunityMember)) {
@@ -160,15 +151,6 @@ class CommunityController implements IAuthenticationRequired
         ]);
     }
 
-    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)) {
@@ -176,7 +158,8 @@ class CommunityController implements IAuthenticationRequired
         }
 
         $code = \Container::$request->post('code');
-        if (strlen($code) === 0 || strlen($code) > 3) {
+        $roundDigits = (int)\Container::$request->post('round_digits');
+        if (strlen($code) === 0 || strlen($code) > 3 || $roundDigits < 0 || $roundDigits > 9) {
             return new JsonContent([
                 'error' => ['errorText' => 'Please fill all required fields!']
             ]);
@@ -192,7 +175,7 @@ class CommunityController implements IAuthenticationRequired
         $currency = new Currency();
         $currency->setCommunity($community);
         $currency->setCode($code);
-        $currency->setRoundDigits((int)\Container::$request->post('round_digits'));
+        $currency->setRoundDigits($roundDigits);
         \Container::$persistentDataManager->saveToDb($currency);
 
         return new JsonContent(['success' => true]);
@@ -268,6 +251,24 @@ class CommunityController implements IAuthenticationRequired
         ]);
     }
 
+    private function getMembers(Community $community): array
+    {
+        $members = iterator_to_array($this->communityMemberRepository->getAllByCommunity($community, true));
+        usort($members, function($a, $b) {
+            return strnatcmp($a->getUser()->getDisplayName(), $b->getUser()->getDisplayName());
+        });
+        return $members;
+    }
+
+    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;
+    }
+
     private function checkPermission(
         int $communityId,
         bool $needToBeOwner,

views/communities/community_currencies.php

@@ -40,5 +40,6 @@
                 </td>
             </tr>
         </table>
+        <p class="formError justify marginTop"></p>
     </div>
 @endsection

views/communities/community_members.php

@@ -47,6 +47,7 @@
                 </td>
             </tr>
         </table>
+        <p class="formError justify marginTop"></p>
     </div>
 @endsection