commit 88a3f60015db7a06490da96593d4dc4a8259cc5e Author: Moshe Weitzman Date: Wed Nov 18 15:35:03 2015 -0500 Fix #2607352 The installer needlessly requires to be writable/ diff --git a/core/modules/system/system.install b/core/modules/system/system.install index c9e8175..bf8d736 100644 --- a/core/modules/system/system.install +++ b/core/modules/system/system.install @@ -424,6 +424,11 @@ function system_requirements($phase) { 'title' => t('Temporary files directory'), 'directory' => drupal_realpath('temporary://'), ); + $sync_dir = config_get_config_directory(CONFIG_SYNC_DIRECTORY); + $htaccess_files[$sync_dir . '/.htaccess'] = array( + 'title' => t('Config directory'), + 'directory' => $sync_dir, + ); foreach ($htaccess_files as $htaccess_file => $info) { // Check for the string which was added to the recommended .htaccess file // in the latest security update. @@ -531,23 +536,6 @@ function system_requirements($phase) { } } - // Check the config directory if it is defined in settings.php. If it isn't - // defined, the installer will create a valid config directory later, but - // during runtime we must always display an error. - if (!empty($GLOBALS['config_directories'])) { - foreach ($GLOBALS['config_directories'] as $type => $directory) { - $directories[] = config_get_config_directory($type); - } - } - elseif ($phase != 'install') { - $requirements['config directories'] = array( - 'title' => t('Configuration directories'), - 'value' => t('Not present'), - 'description' => t('Your %file file must define the $config_directories variable as an array containing the name of a directories in which configuration files can be written.', array('%file' => $site_path . '/settings.php')), - 'severity' => REQUIREMENT_ERROR, - ); - } - $requirements['file system'] = array( 'title' => t('File system'), );