Due to core #1890784: Refactor configuration import and sync functions config-import is broken.

Attached patch tries to fix this but drush config-import ends with a exception

WD config_import: Drupal\Core\Config\StorageException: Failed to write configuration file: sites/site-x/files/config_hash/active/image.style.medium.yml in Drupal\Core\Config\FileStorage->write()

which is not related to this patch but more to
- #1908440: Relax MTimeProtectedFileStorage permissions for DX, drush integration and world domination

OPEN ISSUES

- needs tests.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

clemens.tolboom’s picture

The patch has some code style fixes too.

Please provide some hints on how to write these tests. Esp. how to work with resources.

moshe weitzman’s picture

Status: Needs review » Needs work

config changed even more recently. i think most of the config commands need a refresh. anyone interested in this should contact folks in #drupal-contribute for help, especially alexpott.

clemens.tolboom’s picture

Just learned about https://drupal.org/project/config_inspector which as some useful code parts I guess.

clemens.tolboom’s picture

Status: Needs work » Needs review
FileSize
5.56 KB

I've tried to fix both config-import and config-edit.

I'm not sure this is how it should be as it seems config goes in bulk. We now must copy the active to staging then start edit the given item. Most is cloned from config_admin_import_form_submit

I also documented constants used by (and removed there) drush_copy_dir() as my IDE shows these.

Needs more work.

moshe weitzman’s picture

Status: Needs review » Closed (duplicate)

Thanks. Lets continue work at https://github.com/drush-ops/drush/issues/28