- First commit to issue fork.
- Status changed to Needs review
about 1 year ago 3:11pm 25 June 2023 - πΊπΈUnited States smokris Athens, Ohio, USA
I opened an issue fork consisting of hswong3i's patch from comment #206, plus a commit to remove another stray reference to the removed
$noscript
variable. - Merge request !15Issue #2852269: Integrate Invisible reCAPTCHA option from Google β (Open) created by smokris
- last update
about 1 year ago 2 fail - last update
11 months ago Patch Failed to Apply - π«π·France NuWans
I'm on drupal 10 and recaptcha 3.2, the patch is not applicable on this version. Does anyone use it on this version? Is it planned to adapt?
THANKS - πΊπΈUnited States dchaffin
What is the status of this? I would really like to be able to update to reCaptcha v3. Thanks!
- π³πΏNew Zealand roxflame
@dchaffin, I suggest using this other module while we wait for this to be integrated here
https://www.drupal.org/project/recaptcha_v3 βYou can use it as the primary captcha, and have it fall back to this module if it fails validation.
- π©πͺGermany Anybody Porta Westfalica
@smokris this please needs a rebase.
All others: Once rebased, please test the MR and check if it's working. Can someone help about the documentation, please? (Needs documentation tag)I hope we can then integrate this soon.
For the next time, I think it might make sense to add such new features as (experimental) separate submodules so we don't have to fear breaking something in production for the existing functionality?
- last update
7 months ago 4 pass - π©πͺGermany Anybody Porta Westfalica
@smokris thanks! There's still a TODO in the IS, is that old or still to TBD? Sorry I don't have much time currently, so further community help would speed up the process here for sure.
- Status changed to Needs work
6 months ago 10:19am 22 December 2023 - First commit to issue fork.
- last update
6 months ago 4 pass - π¨π¦Canada OwilliwO Montreal
Sadly, PR 15 diff is not applicable on top of 3.2 stable version.
And the applicable patch from comment #206 is using an undefined variable $noscript (creating a PHP warning).Here is a quite patch I've applied on top of V3.2 + patch #206.
diff --git a/recaptcha.module b/recaptcha.module index b28364a..435ac8b 100644 --- a/recaptcha.module +++ b/recaptcha.module @@ -98,7 +98,6 @@ function recaptcha_captcha($op, $captcha_type = '') { $captcha['form']['recaptcha_widget'] = [ '#markup' => '<div' . new Attribute($attributes) . '></div>', - '#suffix' => $noscript, '#attached' => [ 'html_head' => [ [
Not sure why, but I've not been able to generate a full patch including this variable removal. Sorry about that.
- last update
22 days ago 4 pass