Problem/Motivation

While compiling config actions, SiteExporter uses NestedArray::mergeDeep(), which preserves integer keys. When merging indexed arrays -- in particular, user permissions -- that means all existing permissions are repeated, which if you export the recipe multiple times, recipe.yml can grow at an exponential rate.

Proposed resolution

Don't bother with a deep merge; just fully overwrite the full config action for a particular config object or entity.

Issue fork drupal_cms-3577153

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

phenaproxima created an issue. See original summary.

phenaproxima opened merge request !833

  • phenaproxima committed 79820be6 on 2.x 
    fix (drupal_cms_helper): #3577153 SiteExporter repeats indexed arrays,...

  • phenaproxima committed 7edee58e on 2.0.x 
    fix (drupal_cms_helper): #3577153 SiteExporter repeats indexed arrays,...
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Active » Fixed

A fix worth making quickly. Merged into 2.x and cherry-picked to 2.0.x.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.