Montreal
Account created on 22 November 2012, over 12 years ago
#

Recent comments

🇨🇦Canada OwilliwO Montreal

Tested and approved on my end too !
Keep it up guys.

  • Drupal 10.3.6
  • Layout Builder ST 8.x-1.0-alpha3
  • PHP 8.1
🇨🇦Canada OwilliwO Montreal

Hi !

I'm facing the same issue, and have been able to reproduce it on a fresh site-install.

  • Drupal 10.3.1
  • Layout Builder enabled
  • Content translation enabled
  • Language are:
    • default: english
    • other: french
  • Layout Builder Symmetric Translation enabled
  • Content moderation enabled (draft, published, archived)
  • Moderation sidebar enabled

I've created a content type which is translatable, has the default body/description field and is using layout builder as full display.
I've allowed users to override the default layout for each node.
Now, when I create a page in english, then add a simple content block (inline) in the layout, publish my page, then translate it, and go to the translation layout edition, I can translate the block the publish the french version.
Until here, I'm fine. I can even edit the french or english layouts, as long as it remain published.

The issues I'm facing start when I create a new draft from my published version (typical usecase: I want to prepare a V2 of my page to publish it later in the evening).
I create a draft from my english (original) version then add a new block in the layout, with some new content.

Then I want to prepare the french V2 of my page, but it's impossible, because when editing the translated layout, seems like I'm working on the published revision, instead of the latest.

This would explain why:

  • I can't see the newly created english block, so I can't prepare its translation
  • if I still try to update the translation, it will only translate the published revision layout

I plan on investigating this a bit more.
Obviously any help would be appreciated.

This is quite a big pain for the edition team on my project: they are not able to prepare both english and french version of an already published page in order to publish both version at the same time. They can "plan" the publication of the new original version, but then they have to set the new translated version ASAP, which can be problematic, especially when several important pages are at stake.

Composer file attached to this comment.

🇨🇦Canada OwilliwO Montreal

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           
🇨🇦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.

🇨🇦Canada OwilliwO Montreal

Hello folks.
Here is a new version of patch #42, that you can apply of new release 1.7

🇨🇦Canada OwilliwO Montreal

Awesome !
Just tested, beta2 version is fully compatible with Drupal 9 & 10.
Thanks for the quick answer and release.

🇨🇦Canada OwilliwO Montreal

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.

Production build 0.71.5 2024