DiffArray does not serialize Layout builder section objects causing an error with view display configs managed by layout_builder

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.

Tagging for steps to reproduce this to verify if it's an issue in D10
Tagging for tests to show this issue and that the proposed fix works

The proposed solution doesn't seem to match the MR. Is there still more work to be done there?

Also we will need to update the MR for Drupal 10.1.x please

Thanks!

I was looking at this and don't think I like A) or B), as a utility function I don't think DiffArray should be making changes to form of the inputs and definitely core should not be updated with a hardcoded classname. That seems like maybe normalizing data should be done in StorableConfigBase itself, since it is the one opinionated about only storing arrays. Maybe it should then attempt to do any array conversion/normalization/serialization of objects. Or maybe config_split itself needs to massage everything since it is opinionated about only working with arrays, or maybe layout_builder should make sure all of it's config-able data are arrays to begin with.

I have researched the problem without success but I implemented a VERY HACKY AND UGLY fix for it. Just to have the configuration manager system working again.

I let the solution here just in case is useful for someone.

The patch provided in #11 seems to be problematic as it throws the following error when importing the site's configuration:

TypeError: Drupal\layout_builder\Entity\LayoutBuilderEntityViewDisplayStorage::Drupal\layout_builder\Entity\{closure}(): Argument #1 ($section) must be of type Drupal\layout_builder\Section, array given in Drupal\layout_builder\Entity\LayoutBuilderEntityViewDisplayStorage->Drupal\layout_builder\Entity\{closure}() (line 24 of /app/web/core/modules/layout_builder/src/Entity/LayoutBuilderEntityViewDisplayStorage.php)

Based on the previous patch, I have performed some small adjustments that should do the trick and allow you bypass the mentioned issues.

I found after applying patches #11 and #12 the config export is fixed but the config import started to fail. Here is a single patch combining both patches and adding a fix for the config import

I've rerolled the #13 patch for Drupal 10.3.1.