fix: prevent duplicate submit buttons in forms

Author: earthcomfy

djangocms_form_builder/cms_plugins/__init__.py

@@ -10,6 +10,7 @@
     EmailFieldPlugin,
     IntegerFieldPlugin,
     SelectPlugin,
+    SubmitPlugin,
     TextareaPlugin,
     TimeFieldPlugin,
     URLFieldPlugin,
@@ -26,6 +27,7 @@
     "EmailFieldPlugin",
     "IntegerFieldPlugin",
     "SelectPlugin",
+    "SubmitPlugin",
     "TextareaPlugin",
     "TimeFieldPlugin",
     "URLFieldPlugin",

djangocms_form_builder/cms_plugins/ajax_plugins.py

@@ -240,11 +240,17 @@ def render(self, context, instance, placeholder):
         form = self.get_ajax_form()
         context.update(self.set_context(context, instance, placeholder))
         context["form_counter"] = context.get("form_counter", 0) + 1
-        context.update({
-            "instance": instance,
-            "form": form,
-            "uid": f"{instance.id}{getattr(form, 'slug', '')}-{context['form_counter']}",
-        })
+        has_submit_button = any(
+            child.plugin_type == "SubmitPlugin" for child in instance.get_children()
+        )
+        context.update(
+            {
+                "instance": instance,
+                "form": form,
+                "uid": f"{instance.id}{getattr(form, 'slug', '')}-{context['form_counter']}",
+                "auto_submit": not has_submit_button,
+            }
+        )
         return context
 
 
@@ -290,7 +296,9 @@ def get_parent_classes(cls, slot, page, instance=None):
 
     def get_fieldsets(self, request, obj=None):
         fieldsets = super().get_fieldsets(request, obj)
-        if obj is None or not obj.form_selection:  # No Actions if a Django form has been selected
+        if (
+            obj is None or not obj.form_selection
+        ):  # No Actions if a Django form has been selected
             fieldsets = insert_fields(
                 fieldsets,
                 ("form_actions",),
@@ -362,22 +370,28 @@ def traverse(instance):
         meta_options = dict(form_name=self.instance.form_name)
         if self.instance.form_floating_labels:
             meta_options["floating_labels"] = True
-        meta_options[
-            "field_sep"
-        ] = f'{self.instance.form_spacing}'
-        meta_options[
-            "redirect"
-        ] = SAME_PAGE_REDIRECT  # Default behavior: redirect to same page
+        meta_options["field_sep"] = f"{self.instance.form_spacing}"
+        meta_options["redirect"] = (
+            SAME_PAGE_REDIRECT  # Default behavior: redirect to same page
+        )
         meta_options["login_required"] = self.instance.form_login_required
         meta_options["unique"] = self.instance.form_unique
         form_actions = self.instance.form_actions or "[]"
         meta_options["form_actions"] = json.loads(form_actions.replace("'", '"'))
-        meta_options["form_parameters"] = getattr(self.instance, "action_parameters", {})
+        meta_options["form_parameters"] = getattr(
+            self.instance, "action_parameters", {}
+        )
 
-        fields["Meta"] = type("Meta", (), dict(
-            options=meta_options,
-            verbose_name=self.instance.form_name.replace("-", " ").replace("_", " ").capitalize(),
-        ))  # Meta class with options and verbose name
+        fields["Meta"] = type(
+            "Meta",
+            (),
+            dict(
+                options=meta_options,
+                verbose_name=self.instance.form_name.replace("-", " ")
+                .replace("_", " ")
+                .capitalize(),
+            ),
+        )  # Meta class with options and verbose name
 
         return type(
             "FrontendAutoForm",

djangocms_form_builder/templates/djangocms_form_builder/bootstrap5/widgets/submit.html

@@ -1 +1,3 @@
-<input type="submit" value="{{ instance.form_submit_message|default:_("Submit") }}" class="btn btn-{{ instance.form_submit_context|default:"primary" }}
+{% if auto_submit %}
+<input type="submit" value="{{ instance.form_submit_message|default:_("Submit") }}" class="btn btn-{{ instance.form_submit_context|default:"primary" }}">
+{% endif %}

tests/test_formeditor.py

@@ -24,6 +24,7 @@ def test_form_editor(self):
                 inspect.isclass(cls)
                 and issubclass(cls, FormElementPlugin)
                 and not issubclass(cls, cms_plugins.ChoicePlugin)
+                and cls is not cms_plugins.SubmitPlugin
             ):
                 field = add_plugin(
                     placeholder=self.placeholder,
@@ -48,5 +49,63 @@ def test_form_editor(self):
                 inspect.isclass(cls)
                 and issubclass(cls, FormElementPlugin)
                 and not issubclass(cls, cms_plugins.ChoicePlugin)
+                and cls is not cms_plugins.SubmitPlugin
             ):
                 self.assertContains(response, f'name="field_{item}"')
+
+    def test_auto_submit_button_appears_when_no_button(self):
+        form = add_plugin(
+            placeholder=self.placeholder,
+            plugin_type=cms_plugins.FormPlugin.__name__,
+            language=self.language,
+            form_selection="",
+            form_name="test-form",
+        )
+        add_plugin(
+            placeholder=self.placeholder,
+            plugin_type=cms_plugins.CharFieldPlugin.__name__,
+            target=form,
+            language=self.language,
+            config=dict(
+                field_name="text_field",
+            ),
+        )
+        self.publish(self.page, self.language)
+        with self.login_user_context(self.superuser):
+            response = self.client.get(self.request_url)
+
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(response.content.decode().count('type="submit"'), 1)
+
+    def test_auto_submit_button_does_not_appear_when_button_exists(self):
+        form = add_plugin(
+            placeholder=self.placeholder,
+            plugin_type=cms_plugins.FormPlugin.__name__,
+            language=self.language,
+            form_selection="",
+            form_name="test-form",
+        )
+        add_plugin(
+            placeholder=self.placeholder,
+            plugin_type=cms_plugins.CharFieldPlugin.__name__,
+            target=form,
+            language=self.language,
+            config=dict(
+                field_name="text_field",
+            ),
+        )
+        add_plugin(
+            placeholder=self.placeholder,
+            plugin_type=cms_plugins.SubmitPlugin.__name__,
+            target=form,
+            language=self.language,
+            config=dict(
+                submit_cta="Submit Form",
+            ),
+        )
+        self.publish(self.page, self.language)
+        with self.login_user_context(self.superuser):
+            response = self.client.get(self.request_url)
+
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(response.content.decode().count('type="submit"'), 1)