Patch is working well on my english/french site !
Thanks a lot.
Here is my "Layout Builder Stack":
Layout Builder (layout_builder) Enabled 10.2.2
Layout Builder Restrictions (layout_builder_restrictions) Enabled 8.x-2.19
Layout Builder Restrictions By Region (layout_builder_restrictions_by_region) Disabled 8.x-2.19
Layout Builder Symmetric Translations (layout_builder_st) Enabled 8.x-1.0-alpha3
Layout Builder Styles (layout_builder_styles) Disabled 2.0.0
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.
Hello folks.
Here is a new version of patch #42, that you can apply of new release 1.7
Awesome !
Just tested, beta2 version is fully compatible with Drupal 9 & 10.
Thanks for the quick answer and release.
OwilliwO → created an issue.
MR diff is not applicable to 8.0-1.0-beta2 version of the module, because of the update on tests/src/Functional/MigrateUpgradeEckTest.php (file existing only in dev version).
I'll stick with the patch in comment #6 until next release.