diff --git a/core/modules/file/config/file.settings.yml b/core/modules/file/config/file.settings.yml new file mode 100644 index 0000000000000000000000000000000000000000..e65227724a24c667c810e1fdc4159601f9eb2093 --- /dev/null +++ b/core/modules/file/config/file.settings.yml @@ -0,0 +1,6 @@ +description: + type: 'textfield' + length: 128 +icon: + directory: 'core/modules/file/icons' + diff --git a/core/modules/file/file.field.inc b/core/modules/file/file.field.inc index eb568893f109804707128b6c77c6d4ec74d7c6f6..a43849d4c3076ff6e1914fc551c7251ab8141c44 100644 --- a/core/modules/file/file.field.inc +++ b/core/modules/file/file.field.inc @@ -427,11 +427,12 @@ function file_field_widget_process($element, &$form_state, $form) { // Add the description field if enabled. if (!empty($instance['settings']['description_field']) && $item['fid']) { + $config = config('file.settings'); $element['description'] = array( - '#type' => variable_get('file_description_type', 'textfield'), + '#type' => $config->get('description.type'), '#title' => t('Description'), '#value' => isset($item['description']) ? $item['description'] : '', - '#maxlength' => variable_get('file_description_length', 128), + '#maxlength' => $config->get('description.length'), '#description' => t('The description may be used as the label of the link to the file.'), ); } diff --git a/core/modules/file/file.install b/core/modules/file/file.install index ca6e1fdfb2c0d26b26d36d74ace8f3922a4be68a..27a243825bf658c0d40a90ff5880e3d4db8830b7 100644 --- a/core/modules/file/file.install +++ b/core/modules/file/file.install @@ -232,3 +232,16 @@ function file_requirements($phase) { return $requirements; } + +/** +* Converts default_file_main variable to config. +* +* @ingroup config_upgrade +*/ +function file_update_8000() { + update_variables_to_config('file.settings', array( + 'file_description_type' => 'description.type', + 'file_description_length'=>'description.length', + 'file_icon_directory'=>'icon.directory', + )); +} diff --git a/core/modules/file/file.module b/core/modules/file/file.module index b61e52f53a1f1668c1a00c98f0d7a77c4d86174a..fd2b269ad33b907932de300d90d395bc0923e98f 100644 --- a/core/modules/file/file.module +++ b/core/modules/file/file.module @@ -1231,7 +1231,8 @@ function file_managed_file_pre_render($element) { * An associative array containing: * - file: A file object to which the link will be created. * - icon_directory: (optional) A path to a directory of icons to be used for - * files. Defaults to the value of the "file_icon_directory" variable. + * files. Defaults to the value of the "icon.directory" + * variable. * * @ingroup themeable */ @@ -1270,7 +1271,8 @@ function theme_file_link($variables) { * An associative array containing: * - file: A file entity for which to make an icon. * - icon_directory: (optional) A path to a directory of icons to be used for - * files. Defaults to the value of the "file_icon_directory" variable. + * files. Defaults to the value of the "icon.directory" + * variable. * * @ingroup themeable */ @@ -1290,7 +1292,7 @@ function theme_file_icon($variables) { * A file entity. * @param $icon_directory * (optional) A path to a directory of icons to be used for files. Defaults to - * the value of the "file_icon_directory" variable. + * the value of the "icon.directory" variable. * * @return * A URL string to the icon, or FALSE if an appropriate icon cannot be found. @@ -1309,7 +1311,7 @@ function file_icon_url(File $file, $icon_directory = NULL) { * A file entity. * @param $icon_directory * (optional) A path to a directory of icons to be used for files. Defaults to - * the value of the "file_icon_directory" variable. + * the value of the "icon.directory" variable. * * @return * A string to the icon as a local path, or FALSE if an appropriate icon could @@ -1318,7 +1320,7 @@ function file_icon_url(File $file, $icon_directory = NULL) { function file_icon_path(File $file, $icon_directory = NULL) { // Use the default set of icons if none specified. if (!isset($icon_directory)) { - $icon_directory = variable_get('file_icon_directory', drupal_get_path('module', 'file') . '/icons'); + $icon_directory = config('file.settings')->get('icon.directory'); } // If there's an icon matching the exact mimetype, go for it.