Closed (fixed)
Project:
Drupal core
Version:
8.4.x-dev
Component:
documentation
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
5 Oct 2017 at 08:00 UTC
Updated:
2 Nov 2017 at 16:39 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
alexpottHere's a patch
Comment #3
alexpottRerolled.
Comment #4
borisson_In
install_ensure_config_directorywe were also doing:
file_prepare_directory($config_directory, FILE_CREATE_DIRECTORY | FILE_MODIFY_PERMISSIONS);So that means that functionally we're doing the same thing, looks very good.
Comment #5
xjmConfirmed that this is the last reference:
The change record in https://www.drupal.org/node/2501187 doesn't mention this function and only refers to the long-gone active directory. It looks like the CR reference was added in #2873750: Add Change record to @deprecated for install_ensure_config_directory(), but let's either update the change record appropriately or switch to a better one if there's a better one?
Comment #6
alexpottI've added the deprecation of the function to the existing CR. The code that was already on the CR is the same code that you would need to replace the call to
install_ensure_config_directory().Comment #9
xjmCommitted and pushed to 8.5.x and cherry-picked to 8.4.x. Thanks!