diff --git a/core/modules/file/config/file.settings.yml b/core/modules/file/config/file.settings.yml new file mode 100644 index 0000000..6372c80 --- /dev/null +++ b/core/modules/file/config/file.settings.yml @@ -0,0 +1,6 @@ +description: + type: 'textfield' + length: 128 +icon: + directory: 'icons' + diff --git a/core/modules/file/file.field.inc b/core/modules/file/file.field.inc index 9c37a32..bb1c441 100644 --- a/core/modules/file/file.field.inc +++ b/core/modules/file/file.field.inc @@ -572,11 +572,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 ca6e1fd..27a2438 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 6694598..34b6e90 100644 --- a/core/modules/file/file.module +++ b/core/modules/file/file.module @@ -1238,7 +1238,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 */ @@ -1277,7 +1278,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 */ @@ -1297,7 +1299,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. @@ -1316,7 +1318,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 @@ -1325,7 +1327,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 = drupal_get_path('module', 'file') . '/' . config('file.settings')->get('icon.directory'); } // If there's an icon matching the exact mimetype, go for it.