-
Notifications
You must be signed in to change notification settings - Fork 78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DRAFT: Extend with recaptcha v3 #373
Conversation
@Quehnie @alexander-schranz Does this replace #358? |
if ($this->params->has('ewz_recaptcha.version') | ||
&& 3 == $this->params->get('ewz_recaptcha.version') | ||
) { | ||
$constraint = new \EWZ\Bundle\RecaptchaBundle\Validator\Constraints\IsTrueV3(); | ||
$type = \EWZ\Bundle\RecaptchaBundle\Form\Type\EWZRecaptchaV3Type::class; | ||
unset($options['attr']); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Quehnie This should be done in a compilerpass. There you have access to the ewz_recpatcha.version
even the bundle is registered after the form bundle:
SuluFormBundle/SuluFormBundle.php
Line 26 in b01747b
And then you can set a sulu_form.recaptcha_version
parameter which you use in type_reacptacha.xml
What's in this PR?
The correct way to use EWZ Recaptcha v3 with sulu form bundle
Why?
EWZ Recaptcha V3 was not loaded correctly.
Example Usage
To Do