Merge pull request #30 from 80Quattro/bugfix-double-opt-in-validation

[BUGFIX] PowermailValidator double opt in
studiomitte · Oct 14, 2024 · 91844f1 · 91844f1
2 parents 2bfeeb9 + a3f3544
commit 91844f1
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/Classes/FieldValidator/PowermailV11Validator.php b/Classes/FieldValidator/PowermailV11Validator.php
@@ -49,7 +49,10 @@ protected function isCaptchaCheckToSkip(): bool
     {
         if (property_exists($this, 'flexForm')) {
             $confirmationActive = $this->flexForm['settings']['flexform']['main']['confirmation'] === '1';
-            return $this->getActionName() === 'create' && $confirmationActive;
+            $optinActive = $this->flexForm['settings']['flexform']['main']['optin'] === '1';
+
+            return $this->getActionName() === 'create' && $confirmationActive
+                || $this->getActionName() === 'optinConfirm' && $optinActive;
         }
         return false;
     }

diff --git a/Classes/FieldValidator/PowermailValidator.php b/Classes/FieldValidator/PowermailValidator.php
@@ -62,7 +62,10 @@ protected function isCaptchaCheckToSkip(): bool
     {
         if (property_exists($this, 'flexForm')) {
             $confirmationActive = $this->flexForm['settings']['flexform']['main']['confirmation'] === '1';
-            return $this->getActionName() === 'create' && $confirmationActive;
+            $optinActive = $this->flexForm['settings']['flexform']['main']['optin'] === '1';
+
+            return $this->getActionName() === 'create' && $confirmationActive
+                || $this->getActionName() === 'optinConfirm' && $optinActive;
         }
         return false;
     }

diff --git a/ext_emconf.php b/ext_emconf.php
@@ -17,5 +17,5 @@
         ],
     ],
     'state' => 'beta',
-    'version' => '0.1.7',
+    'version' => '0.1.8',
 ];