]> git.agnieray.net Git - galette.git/commitdiff
Fix member form when members list is empty
authorGuillaume AGNIERAY <dev@agnieray.net>
Sun, 1 Oct 2023 18:47:06 +0000 (20:47 +0200)
committerJohan Cwiklinski <johan@x-tnd.be>
Thu, 5 Oct 2023 04:50:20 +0000 (06:50 +0200)
galette/templates/default/pages/member_form.html.twig

index c42feb744b453787fd3b519d21aed5067d04d3b3..018e90eb14eeb28382c4b5d6ed0d43ccd0ce0e42 100644 (file)
@@ -2,7 +2,7 @@
 
 {% block content %}
 <form action="{% if self_adh %}{{ url_for("storeselfmembers") }}{% elseif not member.id %}{{ url_for("doAddMember") }}{% else %}{{ url_for("doEditMember", {"id": member.id}) }}{% endif %}" method="post" enctype="multipart/form-data" id="form" class="ui form{% if member.id %} edit-member{% endif %}">
-    {% if member.id or not self_adh %}
+    {% if member.id or not self_adh and members.list is defined and members.list|length > 0 %}
 <div class="ui stackable grid">
     <div class="row">
         <div class="sixteen wide tablet six wide computer five wide widescreen column order-2">
@@ -35,7 +35,7 @@
         {% endif %}
     {% endif %}
 
-    {% if not self_adh %}
+    {% if not self_adh and members.list is defined and members.list|length > 0 %}
                 <div class="ui basic horizontally fitted segment">
                     <div class="ui segment">
         {% if member.hasParent() and not member.isDuplicate() %}
@@ -59,7 +59,7 @@
                             </div>
                         </div>
             {% endif %}
-        {% elseif (login.isAdmin() or login.isStaff()) and not member.hasChildren() and members.list is defined %}
+        {% elseif (login.isAdmin() or login.isStaff()) and not member.hasChildren() %}
                         <div class="field">
                             <div id="attach-checkbox" class="ui{% if member.isDuplicate() %} checked{% endif %} toggle checkbox">
                                 <input type="checkbox" name="attach" id="attach" value="1"{% if member.isDuplicate() %} checked="checked"{% endif %}/>
                 </div>
     {% endif %}
 
-    {% if member.id or not self_adh %}
+    {% if member.id or not self_adh and members.list is defined and members.list|length > 0 %}
             </div>
         </div>
         <div class="sixteen wide tablet ten wide computer eleven wide widescreen column">
             {% include "components/forms/csrf.html.twig" %}
             </div>
 
-    {% if member.id or not self_adh %}
+    {% if member.id or not self_adh and members.list is defined and members.list|length > 0 %}
         </div>
     </div>
 </div>