From 1e633eaa8d3ef83f6bb60776b9b21463abbd8f38 Mon Sep 17 00:00:00 2001
From: Benny Born <benny.born@numero2.de>
Date: Mon, 24 Feb 2020 12:45:57 +0100
Subject: [PATCH 1/4] Improve handling of palettes in DCA (fixes #10)

---
 src/Resources/contao/dca/tl_settings.php | 22 ++++++++++++++--------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/src/Resources/contao/dca/tl_settings.php b/src/Resources/contao/dca/tl_settings.php
index 2654a8d..9846e1e 100755
--- a/src/Resources/contao/dca/tl_settings.php
+++ b/src/Resources/contao/dca/tl_settings.php
@@ -1,23 +1,29 @@
 <?php
 
-$palette = $GLOBALS['TL_DCA']['tl_settings']['palettes'];
-$palette = isset($palette['default']) ? $palette['default'] : $palette;
+$GLOBALS['TL_DCA']['tl_settings']['palettes']['default'] = str_replace('{files_legend', '{recaptcha_legend},recaptchaType;{files_legend', $GLOBALS['TL_DCA']['tl_settings']['palettes']['default']);
 
-$GLOBALS['TL_DCA']['tl_settings']['palettes']['default'] = str_replace('{files_legend', '{recaptcha_legend},recaptchaType,recaptchaPublicKey,recaptchaPrivateKey;{files_legend', $palette);
-$GLOBALS['TL_DCA']['tl_settings']['palettes']['recaptcha3'] = str_replace('{files_legend', '{recaptcha_legend},recaptchaType,recaptcha3GlobalThreshold,recaptchaPublicKey,recaptchaPrivateKey;{files_legend', $palette);
+$GLOBALS['TL_DCA']['tl_settings']['config']['onload_callback'][] = function() {
 
-$GLOBALS['TL_DCA']['tl_settings']['palettes']['__selector__'][] = 'recaptchaType';
+    switch( \Config::get('recaptchaType') ) {
 
+        case 'recaptcha3':
+            $GLOBALS['TL_DCA']['tl_settings']['palettes']['default'] = str_replace('recaptchaType','recaptchaType,recaptcha3GlobalThreshold,recaptchaPublicKey,recaptchaPrivateKey', $GLOBALS['TL_DCA']['tl_settings']['palettes']['default']);
+            break;
+        default:
+            $GLOBALS['TL_DCA']['tl_settings']['palettes']['default'] = str_replace('recaptchaType','recaptchaType,recaptchaPublicKey,recaptchaPrivateKey', $GLOBALS['TL_DCA']['tl_settings']['palettes']['default']);
+            break;
+    }
+};
 
 $GLOBALS['TL_DCA']['tl_settings']['fields'] += [
     'recaptchaType' => [
         'label'             => &$GLOBALS['TL_LANG']['tl_settings']['recaptchaType'],
         'inputType'         => 'select',
-        'options_callback'  => function () 
+        'options_callback'  => function ()
         {
             return [
-                'invisible' => 'reCAPTCHA v2: Invisible', 
-                'recaptcha2' => 'reCAPTCHA v2: Checkbox', 
+                'invisible' => 'reCAPTCHA v2: Invisible',
+                'recaptcha2' => 'reCAPTCHA v2: Checkbox',
                 'recaptcha3' => 'reCAPTCHA v3',
             ];
         },

From 6253da514ac0d7e0da19a9390ed3d6730ee59c63 Mon Sep 17 00:00:00 2001
From: Benny Born <benny.born@numero2.de>
Date: Mon, 24 Feb 2020 12:50:34 +0100
Subject: [PATCH 2/4] Remove "defer" and "async" on inline scripts (fixes #15)

---
 src/Resources/contao/templates/ce_background_recaptcha.html5 | 2 +-
 src/Resources/contao/templates/form_recaptcha.html5          | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/Resources/contao/templates/ce_background_recaptcha.html5 b/src/Resources/contao/templates/ce_background_recaptcha.html5
index 896e220..bdf7cc5 100644
--- a/src/Resources/contao/templates/ce_background_recaptcha.html5
+++ b/src/Resources/contao/templates/ce_background_recaptcha.html5
@@ -1,7 +1,7 @@
 <?php if ($this->recaptchaType != 'recaptcha3') return; ?>
 
 <script src="https://www.google.com/recaptcha/api.js?onload=onRecaptchaLoadCallbackModule<?= $this->id ?>&render=<?= $this->publicKey ?>" defer async></script>
-<script type="text/javascript" defer async>
+<script>
     function onRecaptchaLoadCallbackModule<?= $this->id ?> () {
         grecaptcha.execute('<?= $this->publicKey ?>', {action: '<?= $this->action ?>'})
     }
diff --git a/src/Resources/contao/templates/form_recaptcha.html5 b/src/Resources/contao/templates/form_recaptcha.html5
index 72a8e37..87ba9e9 100755
--- a/src/Resources/contao/templates/form_recaptcha.html5
+++ b/src/Resources/contao/templates/form_recaptcha.html5
@@ -5,7 +5,7 @@
         <p class="error"><?= $this->getErrorAsString() ?></p>
     <?php endif; ?>
 
-    <div class="g-recaptcha" 
+    <div class="g-recaptcha"
         id="recaptcha-<?= $this->strCaptchaKey ?>"
         data-sitekey="<?= $this->publicKey ?>"
         data-callback="onSubmit_<?= $this->strCaptchaKey ?>"
@@ -13,7 +13,7 @@
     ></div>
 
     <script src="https://www.google.com/recaptcha/api.js?onload=onRecaptchaLoadCallbackField<?= $this->strCaptchaKey ?><?php if ($this->recaptchaType == 'recaptcha3'): ?>&render=<?= $this->publicKey ?><?php endif; ?>" defer async></script>
-    <script type="text/javascript" defer async>
+    <script>
         function onRecaptchaLoadCallbackField<?= $this->strCaptchaKey ?> () {
             window.onSubmit_<?= $this->strCaptchaKey ?> = function () { };
 

From afc9821f1bfc7d147112323f85991b8d86fedbf3 Mon Sep 17 00:00:00 2001
From: Benny Born <git@bennyborn.de>
Date: Wed, 26 Feb 2020 19:00:09 +0100
Subject: [PATCH 3/4] fixes #11

---
 src/Resources/contao/dca/tl_form_field.php | 21 +++++++++++++++------
 1 file changed, 15 insertions(+), 6 deletions(-)

diff --git a/src/Resources/contao/dca/tl_form_field.php b/src/Resources/contao/dca/tl_form_field.php
index b0a38f1..ff8f9e2 100755
--- a/src/Resources/contao/dca/tl_form_field.php
+++ b/src/Resources/contao/dca/tl_form_field.php
@@ -1,7 +1,20 @@
 <?php
 
+// alter palette for use with reCaptcha V3
+$GLOBALS['TL_DCA']['tl_form_field']['config']['onload_callback'][] = function() {
+
+    if( \Config::get('recaptchaType') != 'recaptcha3' ) {
+        return;
+    }
+
+    dump($GLOBALS['TL_DCA']['tl_form_field']['palettes']['captcha']);
+
+    $GLOBALS['TL_DCA']['tl_form_field']['palettes']['captcha'] = str_replace('{fconfig_legend},', '{fconfig_legend},recaptcha3_threshold,recaptcha3_action,', $GLOBALS['TL_DCA']['tl_form_field']['palettes']['captcha']);
+    dump($GLOBALS['TL_DCA']['tl_form_field']['palettes']['captcha']);
+};
+
 $GLOBALS['TL_DCA']['tl_form_field']['fields'] += [
-    'recaptch3_threshold' => [
+    'recaptcha3_threshold' => [
         'label'             => &$GLOBALS['TL_LANG']['tl_form_field']['recaptcha3_threshold'],
         'inputType'         => 'text',
         'sql'               => "varchar(8) NOT NULL default ''",
@@ -13,8 +26,4 @@
         'sql'               => "varchar(120) NOT NULL default ''",
         'eval'              => ['tl_class' => 'w50', 'rgxp' => 'recaptcha']
     ],
-];
-
-if (Config::get('recaptchaType') != 'recaptcha3') return;
-
-$GLOBALS['TL_DCA']['tl_form_field']['palettes']['captcha'] = str_replace(',type,label', ',type,label,recaptcha3_threshold,recaptcha3_action', $GLOBALS['TL_DCA']['tl_form_field']['palettes']['captcha']);
+];
\ No newline at end of file

From d0a7f4f16bcf182209877a49978958ed47c4802d Mon Sep 17 00:00:00 2001
From: Benny Born <git@bennyborn.de>
Date: Wed, 26 Feb 2020 19:03:20 +0100
Subject: [PATCH 4/4] removed dumps

---
 src/Resources/contao/dca/tl_form_field.php | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/Resources/contao/dca/tl_form_field.php b/src/Resources/contao/dca/tl_form_field.php
index ff8f9e2..4e18ddb 100755
--- a/src/Resources/contao/dca/tl_form_field.php
+++ b/src/Resources/contao/dca/tl_form_field.php
@@ -7,10 +7,7 @@
         return;
     }
 
-    dump($GLOBALS['TL_DCA']['tl_form_field']['palettes']['captcha']);
-
     $GLOBALS['TL_DCA']['tl_form_field']['palettes']['captcha'] = str_replace('{fconfig_legend},', '{fconfig_legend},recaptcha3_threshold,recaptcha3_action,', $GLOBALS['TL_DCA']['tl_form_field']['palettes']['captcha']);
-    dump($GLOBALS['TL_DCA']['tl_form_field']['palettes']['captcha']);
 };
 
 $GLOBALS['TL_DCA']['tl_form_field']['fields'] += [