From ddf1fecc90a429e29fc27c0615913fad34b508b1 Mon Sep 17 00:00:00 2001
From: Simon Vieille <simon.vieille@trinaps.com>
Date: Wed, 26 Jul 2023 11:28:55 +0200
Subject: [PATCH] fix collection widget: allow_add/allow_delete and prototype

---
 src/core/Form/Type/CollectionType.php         |   2 +
 .../form/bootstrap_4_form_theme.html.twig     | 108 ++++++++++--------
 2 files changed, 62 insertions(+), 48 deletions(-)

diff --git a/src/core/Form/Type/CollectionType.php b/src/core/Form/Type/CollectionType.php
index 166d5d5..c64eb7d 100644
--- a/src/core/Form/Type/CollectionType.php
+++ b/src/core/Form/Type/CollectionType.php
@@ -20,6 +20,8 @@ class CollectionType extends BaseCollectionType
             'collection_name' => $options['collection_name'],
             'label_add' => $options['label_add'],
             'label_delete' => $options['label_delete'],
+            'allow_add' => $options['allow_add'],
+            'allow_delete' => $options['allow_delete'],
         ]);
     }
 
diff --git a/src/core/Resources/views/form/bootstrap_4_form_theme.html.twig b/src/core/Resources/views/form/bootstrap_4_form_theme.html.twig
index 6d13df6..1091e6b 100644
--- a/src/core/Resources/views/form/bootstrap_4_form_theme.html.twig
+++ b/src/core/Resources/views/form/bootstrap_4_form_theme.html.twig
@@ -58,36 +58,42 @@
                     {{ form_row(child) }}
                 {% endfor %}
 
-                <div class="text-right">
-                    <span data-collection-delete-container class="btn btn-sm btn-danger">
-                        <span data-collection-delete="{{ loop.index }}">
-                            <span data-collection-delete="{{ loop.index }}" class="fa fa-trash"></span>
-                            {{ label_delete|trans }}
+                {% if allow_delete %}
+                    <div class="text-right">
+                        <span data-collection-delete-container class="btn btn-sm btn-danger">
+                            <span data-collection-delete="{{ loop.index }}">
+                                <span data-collection-delete="{{ loop.index }}" class="fa fa-trash"></span>
+                                {{ label_delete|trans }}
+                            </span>
                         </span>
-                    </span>
-                </div>
+                    </div>
+                {% endif %}
             </div>
         {% endfor %}
     </div>
 
-    <div data-collection-add="collection-{{ collection_name }}" class="collection-add">
-        <span class="btn btn-sm btn-primary" data-collection-add="collection-{{ collection_name }}">
-            <span data-collection-add="collection-{{ collection_name }}" class="fa fa-plus"></span>
-            {{ label_add|trans }}
-        </span>
-    </div>
-
-    <template type="text/template" id="collection-{{ collection_name }}">
-        <div data-collection-item="__name__">
-            {{ form_rest(form.value.vars.prototype) }}
-
-            <div class="text-right">
-                <span data-collection-delete-container class="btn btn-sm btn-danger">
-                    {{ label_delete|trans }}
-                </span>
-            </div>
+    {% if allow_add %}
+        <div data-collection-add="collection-{{ collection_name }}" class="collection-add">
+            <span class="btn btn-sm btn-primary" data-collection-add="collection-{{ collection_name }}">
+                <span data-collection-add="collection-{{ collection_name }}" class="fa fa-plus"></span>
+                {{ label_add|trans }}
+            </span>
         </div>
-    </template>
+
+        <template type="text/template" id="collection-{{ collection_name }}">
+            <div data-collection-item="__name__">
+                {{ form_rest(form.value.vars.prototype) }}
+
+                {% if allow_delete %}
+                    <div class="text-right">
+                        <span data-collection-delete-container class="btn btn-sm btn-danger">
+                            {{ label_delete|trans }}
+                        </span>
+                    </div>
+                {% endif %}
+            </div>
+        </template>
+    {% endif %}
 {% endblock %}
 
 {% block murph_collection_widget %}
@@ -100,36 +106,42 @@
                     {{ form_row(child) }}
                 {% endfor %}
 
-                <div class="text-right">
-                    <span data-collection-delete-container class="btn btn-sm btn-danger">
-                        <span data-collection-delete="{{ loop.index }}">
-                            <span data-collection-delete="{{ loop.index }}" class="fa fa-trash"></span>
-                            {{ label_delete|trans }}
+                {% if allow_delete %}
+                    <div class="text-right">
+                        <span data-collection-delete-container class="btn btn-sm btn-danger">
+                            <span data-collection-delete="{{ loop.index }}">
+                                <span data-collection-delete="{{ loop.index }}" class="fa fa-trash"></span>
+                                {{ label_delete|trans }}
+                            </span>
                         </span>
-                    </span>
-                </div>
+                    </div>
+                {% endif %}
             </div>
         {% endfor %}
     </div>
 
-    <div data-collection-add="collection-{{ collection_name }}" class="collection-add">
-        <span class="btn btn-sm btn-primary" data-collection-add="collection-{{ collection_name }}">
-            <span data-collection-add="collection-{{ collection_name }}" class="fa fa-plus"></span>
-            {{ label_add|trans }}
-        </span>
-    </div>
-
-    <template type="text/template" id="collection-{{ collection_name }}">
-        <div data-collection-item="__name__" {% for attr, value in attrs %}{{ attr }}="{{ value }}"{% endfor %}>
-            {{ form_rest(form.vars.prototype) }}
-
-            <div class="text-right">
-                <span data-collection-delete-container class="btn btn-sm btn-danger">
-                    {{ label_delete|trans }}
-                </span>
-            </div>
+    {% if allow_add %}
+        <div data-collection-add="collection-{{ collection_name }}" class="collection-add">
+            <span class="btn btn-sm btn-primary" data-collection-add="collection-{{ collection_name }}">
+                <span data-collection-add="collection-{{ collection_name }}" class="fa fa-plus"></span>
+                {{ label_add|trans }}
+            </span>
         </div>
-    </template>
+
+        <template type="text/template" id="collection-{{ collection_name }}">
+            <div data-collection-item="__name__" {% for attr, value in attrs %}{{ attr }}="{{ value }}"{% endfor %}>
+                {{ form_rest(form.vars.prototype) }}
+
+                {% if allow_delete %}
+                    <div class="text-right">
+                        <span data-collection-delete-container class="btn btn-sm btn-danger">
+                            {{ label_delete|trans }}
+                        </span>
+                    </div>
+                {% endif %}
+            </div>
+        </template>
+    {% endif %}
 {% endblock %}
 
 {% block file_picker_widget %}