Problem/Motivation

Every time I run drush cim the views.view.storage shows a change.
Exporting after importing seems to be just a shuffling of keys.

I'll provide a pr shortly.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork storage-3450765

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

nicxvan created an issue. See original summary.

nicxvan’s picture

nicxvan commented
Status: Active » Needs review

I haven't been able to test this on a fresh install.

nicxvan opened merge request !15

mandclu’s picture

mandclu commented

@nicxvan I probably just haven't had enough coffee yet today, but could you add steps to reproduce here?

mandclu’s picture

mandclu commented
Status: Needs review » Postponed (maintainer needs more info)
joshuami’s picture

joshuami
he/him
Primary language English
Location Portland, OR
commented

@mandclu, I just ran into this one with a client. To reproduce, install storage on a newish Drupal site. (One created after the core change that exports config with the current approach to config order.) I looked around for the change that implemented that, but I'm not finding it easily. I think it was around the release of Drupal 10, maybe 10.1.

If you save the storage view at /admin/content/storage, and do a config export, you'll see that it reorders a bunch of config.

Not a huge deal, but if you don't have tests around your config sync status, you can end up with some unexplained config mismatches. It was a head-scratcher for a couple of minutes.

joshuami’s picture

joshuami
he/him
Primary language English
Location Portland, OR
commented
Status: Postponed (maintainer needs more info) » Reviewed & tested by the community

I took a minute and tested the patch on a fresh D11 site.

Without the patch:

  • Install storage module.
  • Export config.
  • Add and commit config to local branch.
  • Edit view /admin/content/storage.
  • Export config and see diff.

With the patch:

  • Install storage module.
  • Export config.
  • Add and commit config to local branch.
  • Edit view /admin/content/storage.
  • Export config and see no changes.

  • mandclu committed 3b5638e5 on 1.4.x authored by nicxvan 
    fix: #3450765 Default view needs to be re exported

By: nicxvan
By:...
mandclu’s picture

mandclu commented
Status: Reviewed & tested by the community » Fixed

Thanks for the work here. Merged in.

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.