From 061c26322289118275d535aaf45c9e2cf5697ad0 Mon Sep 17 00:00:00 2001
From: Hugo Peek <hugo@fractal-farming.com>
Date: Thu, 21 Oct 2021 15:19:48 +0800
Subject: [PATCH] Only load front-end form validation if enabled

---
 _build/config.json                                         | 7 +++++++
 _build/src/00_electrons/e_formblocks/settings.json         | 6 ++++++
 core/components/romanesco/docs/changelog.md                | 1 +
 .../06_formulas/f_formblocks/fbloadassets.snippet.php      | 6 ++++++
 .../templates/04_templates/t_global/fbform.template.tpl    | 6 +-----
 5 files changed, 21 insertions(+), 5 deletions(-)

diff --git a/_build/config.json b/_build/config.json
index ced7f911..1f2d6763 100644
--- a/_build/config.json
+++ b/_build/config.json
@@ -12567,6 +12567,13 @@
                 "area": "FormBlocks - Settings",
                 "value": ""
             },
+            {
+                "key": "frontend_validation",
+                "namespace": "formblocks",
+                "type": "combo-boolean",
+                "area": "FormBlocks - Settings",
+                "value": "1"
+            },
             {
                 "key": "formsource",
                 "namespace": "formblocks",
diff --git a/_build/src/00_electrons/e_formblocks/settings.json b/_build/src/00_electrons/e_formblocks/settings.json
index c2d76e08..c8fab22e 100644
--- a/_build/src/00_electrons/e_formblocks/settings.json
+++ b/_build/src/00_electrons/e_formblocks/settings.json
@@ -24,6 +24,12 @@
                 "type": "textfield",
                 "area": "FormBlocks - Settings",
                 "value": ""
+            },{
+                "key": "frontend_validation",
+                "namespace": "formblocks",
+                "type": "combo-boolean",
+                "area": "FormBlocks - Settings",
+                "value": "1"
             },{
                 "key": "formsource",
                 "namespace": "formblocks",
diff --git a/core/components/romanesco/docs/changelog.md b/core/components/romanesco/docs/changelog.md
index 4fa704d9..878b0f13 100644
--- a/core/components/romanesco/docs/changelog.md
+++ b/core/components/romanesco/docs/changelog.md
@@ -4,6 +4,7 @@
 Released on ...
 
 New features:
+- Add front-end validation to FormBlocks
 - Add option to use Recaptcha v3 in forms
 - Add ContentBlock with Table of Contents menu
 - Add output modifier for pThumb to optimize image and generate WebP version
diff --git a/core/components/romanesco/elements/snippets/06_formulas/f_formblocks/fbloadassets.snippet.php b/core/components/romanesco/elements/snippets/06_formulas/f_formblocks/fbloadassets.snippet.php
index 4d16c3f4..b4d6a689 100644
--- a/core/components/romanesco/elements/snippets/06_formulas/f_formblocks/fbloadassets.snippet.php
+++ b/core/components/romanesco/elements/snippets/06_formulas/f_formblocks/fbloadassets.snippet.php
@@ -20,6 +20,7 @@
 $assetsPathVendor = $modx->getOption('romanesco.semantic_vendor_path', $scriptProperties, '');
 $assetsPathDist = $modx->getOption('romanesco.semantic_dist_path', $scriptProperties, '');
 $uploadFile = $modx->getOption('uploadFile', $scriptProperties, 0);
+$validationTpl = $modx->getOption('validationTpl', $scriptProperties, 'fbValidation');
 
 // Load strings to insert in asset paths when cache busting is enabled
 $cacheBusterCSS = $romanesco->getCacheBustingString('CSS');
@@ -45,6 +46,11 @@
     $modx->regClientHTMLBlock('<script defer src="' . $assetsPathJS . '/fileupload.min' . $cacheBusterJS . '.js"></script>');
 }
 
+// Load front-end validation, if enabled
+if ($modx->getOption('formblocks.frontend_validation', $scriptProperties, '')) {
+    $modx->regClientHTMLBlock($modx->getChunk($validationTpl));
+}
+
 // Load custom assets, if present
 // @todo: make this more dynamic
 if (is_file('assets/js/formblocks.min.js')) {
diff --git a/core/components/romanesco/elements/templates/04_templates/t_global/fbform.template.tpl b/core/components/romanesco/elements/templates/04_templates/t_global/fbform.template.tpl
index 07741080..a572f1ea 100644
--- a/core/components/romanesco/elements/templates/04_templates/t_global/fbform.template.tpl
+++ b/core/components/romanesco/elements/templates/04_templates/t_global/fbform.template.tpl
@@ -148,8 +148,4 @@
     </div>
 </form>
 
-[[fbLoadAssets? &uploadFile=`[[+upload_multiple]]`]]
-[[loadAssets?
-    &component=`custom`
-    &inlineJS=`[[$fbValidation]]`
-]]
\ No newline at end of file
+[[!fbLoadAssets? &uploadFile=`[[+upload_multiple]]`]]
\ No newline at end of file