fixed phone check + fixed showing "participant has not connected acco…

…unt" for troop leaders + minor wording for obrok
SkautDevs · Dec 4, 2023 · 95c12d3 · 95c12d3
1 parent 8d8206a
commit 95c12d3
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/src/Event/EventType/Obrok/cs_obrok.yaml b/src/Event/EventType/Obrok/cs_obrok.yaml
@@ -11,6 +11,7 @@ userCustomHelp:
 flash:
     warning:
         fullRegistration: "Už máme plno a ty jsi pod čarou, takže tě zatím nemůžeme registrovat. Zkus to později, třeba někdo nezaplatí registrační poplatek v termínu."
+        noLock: "Registraci nelze uzamknout - nějaké informace jsou špatně vyplněné nebo chybí (možná chybí jméno skupiny?)"
         plTooFewParticipantsTroop: "Nemůžeme uzamknout registraci - ve skupině je příliš málo účastníků*účastnic. Pro funkční skupinu jich je potřeba nejméně %minimalTroopParticipantsCount%"
         plTooManyParticipantsTroop: "Nemůžeme uzamknout registraci - ve skupině je příliš účastníků*účastnic. Může jich být nejvíce %maximalTroopParticipantsCount%, nějaké odstraň prosím"
         notLogged: "Omlouváme se, ale nejsi přihlášen*a. Přihlaš se prosím přes skautIS"

diff --git a/src/Participant/ParticipantService.php b/src/Participant/ParticipantService.php
@@ -283,7 +283,7 @@ public function isParticipantValidForClose(Participant $p, AbstractContentArbite
         }
 
         // numbers and plus sight up front only
-        if ($ca->phone && !empty($p->telephoneNumber) && preg_match('/^\+?[0-9 ]+$/', $p->telephoneNumber) === 0) {
+        if ($ca->phone && (empty($p->telephoneNumber) || preg_match('/^\+?[0-9 ]+$/', $p->telephoneNumber) === 0)) {
             return false;
         }
 

diff --git a/src/Templates/translatable/participant/dashboard.twig b/src/Templates/translatable/participant/dashboard.twig
@@ -52,7 +52,7 @@
                             {% elseif person is TroopLeader %}
                                 {% set participantUserStatus = p.user.status.value %}
                                 <span class="{% if participantUserStatus == 'open' %}hightlight{% endif %}">
-                                    {% if p.user.loginType.value == 'skautis' and p.user.skautisHasMembership == true %}
+                                    {% if p.user.loginType.value == 'skautis' and p.user.skautisHasMembership == false %}
                                         <img src="https://is.skaut.cz/Login/Favicons/favicon-16x16.png" alt="skautisLogo">
                                         {% trans %}_layout.skautisAccountNotConnected{% endtrans %} ❌
                                     {% endif %}