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