Export block UUID with custom Layout Builder blocks

This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.

Can the MR be updated for 10.1 please
Left a comment, but since it's updating the schema believe it will need an update hook and thus test for update hook.

I cound not create a new MR against 10.1.x as I'm not owner of the existing MR.

But I rerolled existing MR against 10.1.x

The last submitted patch, 43: 3180702-d10-43-TEST-ONLY-FAIL.patch, failed testing. View results →

@nginex I rebased the MR (which anyone can do) and @joachim changed the target (which I asked for in Drupal slack)

@tim.plunkett, right, I will take this into account next time, thanks

Previously tagged for update path for the new schema item.

If not needed can we document why not please.

Looking good though!

Thanks!

I'm attaching a patch based on the latest MR that can be applied to Drupal 10.2.x.

Current d10-49 patch is failing with 10.2.3. I am adding the compatible one.

Thanks for sharing the path, it worked for me
https://git.drupalcode.org/project/drupal/-/merge_requests/2383/diffs

I've rerolled the #50 patch for the Drupal 10.3.1 version

Patches #50 and #52 are not including the function ViewsConfigUpdater::needsTidFilterWithMultipleVocabulariesUpdate which is being used on the hook post update.
I'm uploading a reroll against 10.3.1 based on patch #49

As reported in the previous comment, patches #50 and #52 were causing a fatal error, so I was creating a new one based on the patch #49. However, after testing, seems like the patch #49 was incorrect too, because is not loading the entity from uuid on the getEntity method as it's being done in the MR, and therefore the patch #53 is incorrect as well.

I'm uploading a direct reroll from the MR against core 10.3.1 in case anyone needs it.

I have upgraded the patch from #54 in order to allow to the inline block dependency event subscriber to recognize the inline blocks that are referenced with a uuid in the layout.

This is important in order to calculate the correct access checks (\Drupal\block_content\BlockContentAccessControlHandler)

I tired the patch and it works well for exporting the uuid in the default layout builder layout (entity_view_display).

However its not using the blocks UUID to load the inline block. It still wants to load the block based on the block content ID, which can change between environments.

Is there another patch that would load the block based on the UUID instead of the actual ID?

I am having this issue on a Drupal 10.4 site.

is there any progress here? or should I test patch #55?

@aaronchristian : could you fix your issue ?