Allow synced Layout override Translations: translating labels and inline blocks

updating a quick fix for the #299 CCF.

Rerolled again.

Leaving the needs upgrade path tests as the current test does not do any assertion before running updates.

CC failure but if it was reroll there was failures in 301

Tried with MR but the fork is outdated and I can't get 11.x version.

Tried to create an interdiff file, also failed.

So this is reroll MR 1504 against 11.x

Fixed issues from PHPStan

Sorry, phpcs issues was missed. Here is a new patch.

The patch #307 now applies successfully, but need to fix/improve new tests, the full report is available in the job

I have posted a patch in Fix multilingual site's layout edit context issue with TypeError: Argument 1 passed to UrlHelper::buildQuery() must be of the type array, null given 🐛 Fix multilingual site's layout edit context issue with TypeError: Argument 1 passed to UrlHelper::buildQuery() must be of the type array, null given Needs work as I was getting this error when trying to access a translation's layout builder in D10.1.2.

I have fixed the missing $args['route_parameters'] (they are availabe in $contextual_links['layout_builder_block_translation'] or $contextual_links['layout_builder_inline_block_translation']) and everything now works.

Maybe someone can check why this patch is required and fix the missing $args['route_parameters'] when _contextual_links_to_id($contextual_links) is called. I am using the 2946333-d10-307.patch

I have fixed the cc failure, added use statement for ClientException and updated elseif condition, Attached patch and interdiff for same

Was previously tagged for tests and upgrade path which still need to happen.

Did not review as there’s more to don

We need an update of 2946333-d10-307.patch for D10.2.x. When issuing composer update to update from 10.1.7 to 10.2.0 I'm getting:

Cannot apply patch Drupal Layout Builder. Allow synced Layout override Translations (https://www.drupal.org/files/issues/202
  3-07-25/2946333-d10-307.patch)!

Watch out: #310 📌 Allow synced Layout override Translations: translating labels and inline blocks Needs work is wrong. I think this issue is a fix for https://www.drupal.org/project/drupal/issues/3101231 🐛 Fix multilingual site's layout edit context issue with TypeError: Argument 1 passed to UrlHelper::buildQuery() must be of the type array, null given Needs work . I'll add the patch there.

Rebased the previous MR on a new MR that is based on 11.x. Fixed some obvious cruft that made its way into the patch file. And updated the deprecation strings. But we still have a lot of needs... tags, so this still needs more work.

heddn → changed the visibility of the branch 9.4.x to hidden.

heddn → changed the visibility of the branch 2946333-allow-synced-layout to hidden.

The latest MR5956 did not apply on D10.2.3, stopping at core/modules/layout_builder/layout_builder.services.yml.
Here is the re-roll for D10.2. as a patch.

Went through coding standards errors and made corrections.

Only one small test error left, have to spin up a vanilla environment with the spark theme enabled as well as this patch going in order to figure it out.

@heddn, this is one of the tests you wrote, one last test fix and it'll be ready to go, can you please have a look?

Left some comments, mostly small stuff.

Leaving needs update path tests though as I didn't see those.

Also such a change seems like will need a change record

I applied the patch 2946333-319.patch by JordiK to D10.2.3 but there is no edit icon when hovering over a block in Layout builder. Also if my patch 3101231-D10-allow-synced-layout-override-translations.patch is not applied then back to the error TypeError: Drupal\Component\Utility\UrlHelper::buildQuery(): Argument #1 ($query) must be of type array, null given

Mithun S → made their first commit to this issue’s fork.

Revised #319 as it cleanly applies for D-10.2.x but was facing an issue with the pencil icon on the layout translation page.

Fixed the issue.

Please review and suggest if any other change is required

I applied the revised patch #327 and still don't get the pencil icon on translated pages when in Layout.

The above is what I get when editing the layout of a translated node on a clean install of D10.2.4. Only the drag/drop icon shows anywhere in the block

Fixed the issue with the note appearing contextual links as mentioned in #325 and #328. To those people who still upload patches in this massive issue: Nobody will see what you changed if you don't at least upload an interdiff along with your patch. But even if you do and there's an MR in the issue, please just use the MR to push your changes, that will make collaboration much easier.

The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

I'm trying to add the 10.2.x fork to my composer.json (sorry never done this before) I have this under repositories:

 {
            "type": "vcs",
            "url": "https://git.drupalcode.org/issue/drupal-2946333.git"
        }

But I don't know what to put under require for the 2946333_3049332_combined-10.2.x branch and would it replace the drupal/core requirement?

So you only need 1 MR pointed at 11.x can close the others

Was this pushed to the core? Is there a way to translate titles added with Layout Builder without applying a patch now?

It looks like the MR for 10.3 is not working properly.
I am using this patch: https://git.drupalcode.org/project/drupal/-/merge_requests/8480.diff which applies fine, but blocks added with layout builder cannot be translated.

What does that patch allow us to translate?

I'm on Drupal 10.3.1 and using layout_builder_st (Layout Builder Symmetric Translations) with patches for Drupal 10 compatibility, but I've noticed some errors with xdebug and ended with this issue.

Am I right that the patch from this issue should resolve the same thing Layout Builder Symmetric Translations is attempted to fix?

If yes, I'd prefer to remove Layout Builder Symmetric Translations and install the patch, but which patch works for 10.3.x?

Subscribe to this issue, I think that Symettric translations should be covered by core, leaving the Asymettric translations in the already stable contrib module.

Patch from https://git.drupalcode.org/project/drupal/-/merge_requests/8480

This patch is causing problems when reverting revisions.

This is because of the getActive call that was added to retrieve the latest version of the block when editing in stead of the saved revision. I believe it was added to prevent EntityChangedConstraint errors, but they were fixed in https://www.drupal.org/project/drupal/issues/3053881 → .

Steps to reproduce this:

1. Create a page with a text block with title AAAA.
2. Update the page and change the text block title to BBBB.
3. Revert back to the first version.
4. The block title is now AAAA. But when you open the edit form of the block, it shows BBBB (the latest revision)

Code that is causing this behavior (in InlineBlock::blockForm()):

    if (!$this->isNew && !$block->isNew() && empty($this->configuration['block_serialized'])) {
      // Get the active block for editing purposes.
      $block = $this->entityRepository->getActive('block_content', $block->id());
    }

I think there might also be other places that this getActive function is called to prevent the error where it is no longer needed.

I would be willing to update the patch, but have not been active in creating it, so I'm not sure about what I might be breaking when removing this code.

@l_vandamme: I think the patch is just a static copy from the MR here. Please adjust it there to proceed with the final fix. Patches can then be downloaded statically from there to use with composer patches for example. Thanks!

layout_builder_st is still not D11 compatible and both maintainers seem inactive and do not repond on PM. Maybe someone should request maintainership until this is fixed?

mysdiir → made their first commit to this issue’s fork.

Added a new 10.3.x backport made with the base of 2946333-11.x-rebase.
MR !10504