The configuration sync directory is now defined in $settings['config_sync_directory'] in settings.php.
The ability to support multiple configuration directories in the $config_directories is deprecated. If you have custom or contributed code that relies on this ability then you need to move your setting either to $settings or another storage - for example state or configuration itself.
As a result of simplifying configuration sync directory configuration we have also deprecated config_get_config_directory(), CONFIG_SYNC_DIRECTORY and drupal_install_config_directories().
Instead of calling config_get_config_directory(CONFIG_SYNC_DIRECTORY) use Settings::get('config_sync_directory').
The constant CONFIG_SYNC_DIRECTORY has no replacement because it is not necessary anymore. The install method drupal_install_config_directories() also has no replacement. It's functionality is implemented as part of \Drupal\Core\Installer\Form\SiteSettingsForm::submitForm().
Updating your settings.php file
Before
$config_directories['sync'] = 'sites/default/files/config_YLZJmmpOqc_KBWbMc2I58ky3-8c7qtg4G-OpSqFClHs5E0NL9YMFgyF4RRTv8IFdl_kAMs_Bdw/sync';
After
$settings['config_sync_directory'] = 'sites/default/files/config_YLZJmmpOqc_KBWbMc2I58ky3-8c7qtg4G-OpSqFClHs5E0NL9YMFgyF4RRTv8IFdl_kAMs_Bdw/sync';
Comments
ShouldThe ability to support
Should
be
?
Not sure of the etiquette in editing someone else's CR!
The branch and version are
The branch and version are reversed for this CR, making it harder to find when looking for all 8.8.x records
Now fixed - thanks!
Now fixed - thanks!
--
Morris Animal Foundation
Need a mention on the 8.8.0 release page
It seems that this change has not been mentioned on the 8.8.0 release page, or maybe I missed something?
We can find this change in the default.settings.php file, but if someone forgot to look at this file, he will miss this major change (and his config sync may be broken?!).
In my memory,In 2020/02, I
In my memory,In 2020/02, I remeber I found this change note on the 8.8.0 release page,
But now, as you see, disappeared,,,,,
Should be put in release notes
I'm also very surprised that this is not in the 8.8.0 release notes.
This change appears to break backwards compatibility, so it doesn't seem like it should even be in a minor version release, let alone ignored in release notes.
All opinions are personal and do not represent anyone but myself.
New feature: Subscribe to all Change records
This is why I have suggested #3094094: Allow subscribing to Change records to prevent confusing situations, where something that used to work, doesn't work any longer.
Adding a positive comment in that issue could be seen as supporting the idea of subscribing to all Change records.