diff --git a/core/includes/ajax.inc b/core/includes/ajax.inc index ede35da..538c95d 100644 --- a/core/includes/ajax.inc +++ b/core/includes/ajax.inc @@ -516,7 +516,7 @@ function ajax_prepare_response($page_callback_result) { case MENU_SITE_OFFLINE: $commands[] = ajax_command_alert(filter_xss_admin(variable_get('maintenance_mode_message', - t('@site is currently under maintenance. We should be back shortly. Thank you for your patience.', array('@site' => variable_get('site_name', 'Drupal')))))); + t('@site is currently under maintenance. We should be back shortly. Thank you for your patience.', array('@site' => config('system.site')->get('site_name')))))); break; } } diff --git a/core/includes/common.inc b/core/includes/common.inc index 15c54d3..82a8948 100644 --- a/core/includes/common.inc +++ b/core/includes/common.inc @@ -2428,6 +2428,7 @@ function drupal_deliver_html_page($page_callback_result) { // Menu status constants are integers; page content is a string or array. if (is_int($page_callback_result)) { + $site_config = config('system.site'); // @todo: Break these up into separate functions? switch ($page_callback_result) { case MENU_NOT_FOUND: @@ -2444,7 +2445,7 @@ function drupal_deliver_html_page($page_callback_result) { $_GET['destination'] = current_path(); } - $path = drupal_get_normal_path(variable_get('site_404', '')); + $path = drupal_get_normal_path($site_config->get('site_404')); if ($path && $path != current_path()) { // Custom 404 handler. Set the active item in case there are tabs to // display, or other dependencies on the path. @@ -2473,7 +2474,7 @@ function drupal_deliver_html_page($page_callback_result) { $_GET['destination'] = current_path(); } - $path = drupal_get_normal_path(variable_get('site_403', '')); + $path = drupal_get_normal_path($site_config->get('site_403')); if ($path && $path != current_path()) { // Custom 403 handler. Set the active item in case there are tabs to // display or other dependencies on the path. @@ -2496,7 +2497,7 @@ function drupal_deliver_html_page($page_callback_result) { drupal_add_http_header('Status', '503 Service unavailable'); drupal_set_title(t('Site under maintenance')); print theme('maintenance_page', array('content' => filter_xss_admin(variable_get('maintenance_mode_message', - t('@site is currently under maintenance. We should be back shortly. Thank you for your patience.', array('@site' => variable_get('site_name', 'Drupal'))))))); + t('@site is currently under maintenance. We should be back shortly. Thank you for your patience.', array('@site' => $site_config->get('site_name'))))))); break; } } diff --git a/core/includes/gettext.inc b/core/includes/gettext.inc index 4e3b221..979adb2 100644 --- a/core/includes/gettext.inc +++ b/core/includes/gettext.inc @@ -955,7 +955,7 @@ function _locale_export_po_generate($language = NULL, $strings = array(), $heade if (!isset($header)) { if (isset($language)) { - $header = '# ' . $language->name . ' translation of ' . variable_get('site_name', 'Drupal') . "\n"; + $header = '# ' . $language->name . ' translation of ' . config('system.site')->get('site_name') . "\n"; $header .= '# Generated by ' . $user->name . ' <' . $user->mail . ">\n"; $header .= "#\n"; $header .= "msgid \"\"\n"; diff --git a/core/includes/install.core.inc b/core/includes/install.core.inc index eb9dff9..1a47daa 100644 --- a/core/includes/install.core.inc +++ b/core/includes/install.core.inc @@ -1916,8 +1916,11 @@ function install_configure_form_validate($form, &$form_state) { function install_configure_form_submit($form, &$form_state) { global $user; - variable_set('site_name', $form_state['values']['site_name']); - variable_set('site_mail', $form_state['values']['site_mail']); + config('system.site') + ->set('site_name', $form_state['values']['site_name']) + ->set('site_mail', $form_state['values']['site_mail']) + ->save(); + variable_set('date_default_timezone', $form_state['values']['date_default_timezone']); variable_set('site_default_country', $form_state['values']['site_default_country']); diff --git a/core/includes/mail.inc b/core/includes/mail.inc index f58a76e..95a77bd 100644 --- a/core/includes/mail.inc +++ b/core/includes/mail.inc @@ -118,7 +118,11 @@ define('MAIL_LINE_ENDINGS', isset($_SERVER['WINDIR']) || strpos($_SERVER['SERVER * accepted at php-level, which still doesn't guarantee it to be delivered.) */ function drupal_mail($module, $key, $to, $language, $params = array(), $from = NULL, $send = TRUE) { - $default_from = variable_get('site_mail', ini_get('sendmail_from')); + $site_mail = config('system.site')->get('site_mail'); + if (empty($site_mail)) { + $site_mail = ini_get('sendmail_from'); + } + $default_from = $site_mail; // Bundle up the variables into a structured array for altering. $message = array( diff --git a/core/includes/path.inc b/core/includes/path.inc index 0c7e4d4..0635f80 100644 --- a/core/includes/path.inc +++ b/core/includes/path.inc @@ -22,7 +22,7 @@ function drupal_path_initialize() { // If on the front page, resolve to the front page path, including for calls // to current_path() while drupal_get_normal_path() is in progress. if (empty($path)) { - $path = variable_get('site_frontpage', 'user'); + $path = config('system.site')->get('site_frontpage'); _current_path($path); } @@ -298,7 +298,7 @@ function drupal_is_front_page() { $is_front_page = &$drupal_static_fast['is_front_page']; if (!isset($is_front_page)) { - $is_front_page = (current_path() == variable_get('site_frontpage', 'user')); + $is_front_page = (current_path() == config('system.site')->get('site_frontpage')); } return $is_front_page; @@ -329,7 +329,7 @@ function drupal_match_path($path, $patterns) { $replacements = array( '|', '.*', - '\1' . preg_quote(variable_get('site_frontpage', 'user'), '/') . '\2' + '\1' . preg_quote(config('system.site')->get('site_frontpage'), '/') . '\2' ); $patterns_quoted = preg_quote($patterns, '/'); $regexps[$patterns] = '/^(' . preg_replace($to_replace, $replacements, $patterns_quoted) . ')$/'; diff --git a/core/includes/theme.inc b/core/includes/theme.inc index ea1dd1e..719324c 100644 --- a/core/includes/theme.inc +++ b/core/includes/theme.inc @@ -2530,17 +2530,18 @@ function template_preprocess_html(&$variables) { drupal_add_html_head_link(array('rel' => 'shortcut icon', 'href' => drupal_strip_dangerous_protocols($favicon), 'type' => $type)); } + $site_config = config('system.site'); // Construct page title. if (drupal_get_title()) { $head_title = array( 'title' => strip_tags(drupal_get_title()), - 'name' => check_plain(variable_get('site_name', 'Drupal')), + 'name' => check_plain($site_config->get('site_name')), ); } else { - $head_title = array('name' => check_plain(variable_get('site_name', 'Drupal'))); - if (variable_get('site_slogan', '')) { - $head_title['slogan'] = filter_xss_admin(variable_get('site_slogan', '')); + $head_title = array('name' => check_plain($site_config->get('site_name'))); + if ($site_config->get('site_slogan')) { + $head_title['slogan'] = filter_xss_admin($site_config->get('site_slogan')); } } $variables['head_title_array'] = $head_title; @@ -2606,6 +2607,7 @@ function template_preprocess_html(&$variables) { */ function template_preprocess_page(&$variables) { $language_interface = drupal_container()->get(LANGUAGE_TYPE_INTERFACE); + $site_config = config('system.site'); // Move some variables to the top level for themer convenience and template cleanliness. $variables['show_messages'] = $variables['page']['#show_messages']; @@ -2634,8 +2636,8 @@ function template_preprocess_page(&$variables) { $variables['main_menu'] = theme_get_setting('toggle_main_menu') ? menu_main_menu() : array(); $variables['secondary_menu'] = theme_get_setting('toggle_secondary_menu') ? menu_secondary_menu() : array(); $variables['action_links'] = menu_local_actions(); - $variables['site_name'] = (theme_get_setting('toggle_name') ? filter_xss_admin(variable_get('site_name', 'Drupal')) : ''); - $variables['site_slogan'] = (theme_get_setting('toggle_slogan') ? filter_xss_admin(variable_get('site_slogan', '')) : ''); + $variables['site_name'] = (theme_get_setting('toggle_name') ? filter_xss_admin($site_config->get('site_name')) : ''); + $variables['site_slogan'] = (theme_get_setting('toggle_slogan') ? filter_xss_admin($site_config->get('site_slogan')) : ''); $variables['tabs'] = menu_local_tabs(); if ($node = menu_get_object()) { @@ -2817,17 +2819,18 @@ function template_preprocess_maintenance_page(&$variables) { $variables['layout'] = ($variables['layout'] == 'first') ? 'both' : 'second'; } + $site_config = config('system.site'); // Construct page title if (drupal_get_title()) { $head_title = array( 'title' => strip_tags(drupal_get_title()), - 'name' => variable_get('site_name', 'Drupal'), + 'name' => $site_config->get('site_name'), ); } else { - $head_title = array('name' => variable_get('site_name', 'Drupal')); - if (variable_get('site_slogan', '')) { - $head_title['slogan'] = variable_get('site_slogan', ''); + $head_title = array('name' => $site_config->get('site_name')); + if ($site_config->get('site_slogan')) { + $head_title['slogan'] = $site_config->get('site_slogan'); } } @@ -2844,8 +2847,8 @@ function template_preprocess_maintenance_page(&$variables) { $variables['messages'] = $variables['show_messages'] ? theme('status_messages') : ''; $variables['main_menu'] = array(); $variables['secondary_menu'] = array(); - $variables['site_name'] = (theme_get_setting('toggle_name') ? variable_get('site_name', 'Drupal') : ''); - $variables['site_slogan'] = (theme_get_setting('toggle_slogan') ? variable_get('site_slogan', '') : ''); + $variables['site_name'] = (theme_get_setting('toggle_name') ? $site_config->get('site_name') : ''); + $variables['site_slogan'] = (theme_get_setting('toggle_slogan') ? $site_config->get('site_slogan') : ''); $variables['tabs'] = ''; $variables['title'] = drupal_get_title(); diff --git a/core/lib/Drupal/Core/EventSubscriber/MaintenanceModeSubscriber.php b/core/lib/Drupal/Core/EventSubscriber/MaintenanceModeSubscriber.php index daed1c9..4e29088 100644 --- a/core/lib/Drupal/Core/EventSubscriber/MaintenanceModeSubscriber.php +++ b/core/lib/Drupal/Core/EventSubscriber/MaintenanceModeSubscriber.php @@ -39,7 +39,7 @@ class MaintenanceModeSubscriber implements EventSubscriberInterface { // Deliver the 503 page. drupal_maintenance_theme(); drupal_set_title(t('Site under maintenance')); - $content = theme('maintenance_page', array('content' => filter_xss_admin(variable_get('maintenance_mode_message', t('@site is currently under maintenance. We should be back shortly. Thank you for your patience.', array('@site' => variable_get('site_name', 'Drupal'))))))); + $content = theme('maintenance_page', array('content' => filter_xss_admin(variable_get('maintenance_mode_message', t('@site is currently under maintenance. We should be back shortly. Thank you for your patience.', array('@site' => config('system.site')->get('site_name'))))))); $response = new Response('Service unavailable', 503); $response->setContent($content); $event->setResponse($response); diff --git a/core/lib/Drupal/Core/EventSubscriber/PathSubscriber.php b/core/lib/Drupal/Core/EventSubscriber/PathSubscriber.php index 9104544..cf8ef14 100644 --- a/core/lib/Drupal/Core/EventSubscriber/PathSubscriber.php +++ b/core/lib/Drupal/Core/EventSubscriber/PathSubscriber.php @@ -50,7 +50,10 @@ class PathSubscriber extends PathListenerBase implements EventSubscriberInterfac if (empty($path)) { // @todo Temporary hack. Fix when configuration is injectable. - $path = variable_get('site_frontpage', 'user'); + $path = config('system.site')->get('site_frontpage'); + if (empty($path)) { + $path = 'user'; + } } $this->setPath($request, $path); diff --git a/core/lib/Drupal/Core/ExceptionController.php b/core/lib/Drupal/Core/ExceptionController.php index b163395..fb2c5d4 100644 --- a/core/lib/Drupal/Core/ExceptionController.php +++ b/core/lib/Drupal/Core/ExceptionController.php @@ -94,7 +94,7 @@ class ExceptionController { $system_path = $request->attributes->get('system_path'); watchdog('access denied', $system_path, NULL, WATCHDOG_WARNING); - $path = drupal_get_normal_path(variable_get('site_403', '')); + $path = drupal_get_normal_path(config('system.site')->get('site_403')); if ($path && $path != $system_path) { // Keep old path for reference, and to allow forms to redirect to it. if (!isset($_GET['destination'])) { @@ -160,7 +160,7 @@ class ExceptionController { $_GET['destination'] = $system_path; } - $path = drupal_get_normal_path(variable_get('site_404', '')); + $path = drupal_get_normal_path(config('system.site')->get('site_404')); if ($path && $path != $system_path) { // @todo Um, how do I specify an override URL again? Totally not clear. Do // that and sub-call the kernel rather than using meah(). diff --git a/core/modules/aggregator/aggregator.pages.inc b/core/modules/aggregator/aggregator.pages.inc index fcc61b0..7f07328 100644 --- a/core/modules/aggregator/aggregator.pages.inc +++ b/core/modules/aggregator/aggregator.pages.inc @@ -11,7 +11,7 @@ * @see aggregator_menu() */ function aggregator_page_last() { - drupal_add_feed('aggregator/rss', variable_get('site_name', 'Drupal') . ' ' . t('aggregator')); + drupal_add_feed('aggregator/rss', config('system.site')->get('site_name') . ' ' . t('aggregator')); $items = aggregator_load_feed_items('sum'); @@ -61,7 +61,7 @@ function aggregator_page_source_form($form, $form_state, $feed) { * @ingroup forms */ function aggregator_page_category($category) { - drupal_add_feed('aggregator/rss/' . $category['cid'], variable_get('site_name', 'Drupal') . ' ' . t('aggregator - @title', array('@title' => $category['title']))); + drupal_add_feed('aggregator/rss/' . $category['cid'], config('system.site')->get('site_name') . ' ' . t('aggregator - @title', array('@title' => $category['title']))); // It is safe to include the cid in the query because it's loaded from the // database by aggregator_category_load(). @@ -452,7 +452,7 @@ function theme_aggregator_page_rss($variables) { $items .= format_rss_item($feed->ftitle . ': ' . $feed->title, $feed->link, $feed->description, array('pubDate' => date('r', $feed->timestamp))); } - $site_name = variable_get('site_name', 'Drupal'); + $site_name = config('system.site')->get('site_name'); $url = url((isset($category) ? 'aggregator/categories/' . $category->cid : 'aggregator'), array('absolute' => TRUE)); $description = isset($category) ? t('@site_name - aggregated feeds in category @title', array('@site_name' => $site_name, '@title' => $category->title)) : t('@site_name - aggregated feeds', array('@site_name' => $site_name)); @@ -504,7 +504,7 @@ function theme_aggregator_page_opml($variables) { $output = "\n"; $output .= "\n"; $output .= "\n"; - $output .= '' . check_plain(variable_get('site_name', 'Drupal')) . "\n"; + $output .= '' . check_plain(config('system.site')->get('site_name')) . "\n"; $output .= '' . gmdate(DATE_RFC2822, REQUEST_TIME) . "\n"; $output .= "\n"; $output .= "\n"; diff --git a/core/modules/block/lib/Drupal/block/Tests/BlockTest.php b/core/modules/block/lib/Drupal/block/Tests/BlockTest.php index 38e65c4..4ea558a 100644 --- a/core/modules/block/lib/Drupal/block/Tests/BlockTest.php +++ b/core/modules/block/lib/Drupal/block/Tests/BlockTest.php @@ -167,7 +167,7 @@ class BlockTest extends WebTestBase { function testBlockVisibility() { // Enable Node module and change the front page path to 'node'. module_enable(array('node')); - variable_set('site_frontpage', 'node'); + config('system.site')->set('site_frontpage', 'node')->save(); $block = array(); diff --git a/core/modules/contact/contact.install b/core/modules/contact/contact.install index 7ddea45..ea1ae87 100644 --- a/core/modules/contact/contact.install +++ b/core/modules/contact/contact.install @@ -68,11 +68,15 @@ function contact_schema() { * Implements hook_install(). */ function contact_install() { + $site_mail = config('system.site')->get('site_mail'); + if (empty($site_mail)) { + $site_mail = ini_get('sendmail_from'); + } // Insert a default contact category. db_insert('contact') ->fields(array( 'category' => 'Website feedback', - 'recipients' => variable_get('site_mail', ini_get('sendmail_from')), + 'recipients' => $site_mail, 'selected' => 1, 'reply' => '', )) diff --git a/core/modules/contact/contact.module b/core/modules/contact/contact.module index 48c12d9..4840e10c 100644 --- a/core/modules/contact/contact.module +++ b/core/modules/contact/contact.module @@ -170,7 +170,7 @@ function contact_load($cid) { function contact_mail($key, &$message, $params) { $language = $message['language']; $variables = array( - '!site-name' => variable_get('site_name', 'Drupal'), + '!site-name' => config('system.site')->get('site_name'), '!subject' => $params['subject'], '!category' => isset($params['category']['category']) ? $params['category']['category'] : '', '!form-url' => url(current_path(), array('absolute' => TRUE, 'language' => $language)), diff --git a/core/modules/filter/filter.module b/core/modules/filter/filter.module index b7a3059..5c71025 100644 --- a/core/modules/filter/filter.module +++ b/core/modules/filter/filter.module @@ -1277,7 +1277,7 @@ function _filter_html_tips($filter, $format, $long = FALSE) { $output .= '

' . t('This site allows HTML content. While learning all of HTML may feel intimidating, learning how to use a very small number of the most basic HTML "tags" is very easy. This table provides examples for each tag that is enabled on this site.') . '

'; $output .= '

' . t('For more information see W3C\'s HTML Specifications or use your favorite search engine to find other sites that explain HTML.', array('@html-specifications' => 'http://www.w3.org/TR/html/')) . '

'; $tips = array( - 'a' => array(t('Anchors are used to make links to other pages.'), '' . check_plain(variable_get('site_name', 'Drupal')) . ''), + 'a' => array(t('Anchors are used to make links to other pages.'), '' . check_plain(config('system.site')->get('site_name')) . ''), 'br' => array(t('By default line break tags are automatically added, so use this tag to add additional ones. Use of this tag is different because it is not used with an open/close pair like all the others. Use the extra " /" inside the tag to maintain XHTML 1.0 compatibility'), t('Text with
line break')), 'p' => array(t('By default paragraph tags are automatically added, so use this tag to add additional ones.'), '

' . t('Paragraph one.') . '

' . t('Paragraph two.') . '

'), 'strong' => array(t('Strong', array(), array('context' => 'Font weight')), '' . t('Strong', array(), array('context' => 'Font weight')) . ''), diff --git a/core/modules/locale/lib/Drupal/locale/Tests/LocalePathTest.php b/core/modules/locale/lib/Drupal/locale/Tests/LocalePathTest.php index 61fd67f..34facf3 100644 --- a/core/modules/locale/lib/Drupal/locale/Tests/LocalePathTest.php +++ b/core/modules/locale/lib/Drupal/locale/Tests/LocalePathTest.php @@ -25,7 +25,7 @@ class LocalePathTest extends WebTestBase { parent::setUp(array('node', 'locale', 'path')); $this->drupalCreateContentType(array('type' => 'page', 'name' => 'Basic page')); - variable_set('site_frontpage', 'node'); + config('system.site')->set('site_frontpage', 'node')->save(); } /** diff --git a/core/modules/node/node.install b/core/modules/node/node.install index 6f806e9..981fdba 100644 --- a/core/modules/node/node.install +++ b/core/modules/node/node.install @@ -475,7 +475,6 @@ function node_uninstall() { variable_del('node_admin_theme'); variable_del('node_cron_last'); variable_del('node_recent_block_count'); - variable_del('default_nodes_main'); } /** diff --git a/core/modules/node/node.module b/core/modules/node/node.module index 171a74b..0b3d279 100644 --- a/core/modules/node/node.module +++ b/core/modules/node/node.module @@ -2437,7 +2437,7 @@ function node_feed($nids = FALSE, $channel = array()) { $channel_defaults = array( 'version' => '2.0', - 'title' => variable_get('site_name', 'Drupal'), + 'title' => config('system.site')->get('site_name'), 'link' => $base_url, 'description' => config('system.rss-publishing')->get('feed_description'), 'language' => $language_content->langcode @@ -2488,6 +2488,7 @@ function node_view_multiple($nodes, $view_mode = 'teaser', $weight = 0, $langcod * @see node_menu() */ function node_page_default() { + $site_config = config('system.site'); $select = db_select('node', 'n') ->fields('n', array('nid', 'sticky', 'created')) ->condition('promote', 1) @@ -2495,7 +2496,7 @@ function node_page_default() { ->orderBy('sticky', 'DESC') ->orderBy('created', 'DESC') ->extend('Drupal\Core\Database\Query\PagerSelectExtender') - ->limit(variable_get('default_nodes_main', 10)) + ->limit($site_config->get('default_nodes_main')) ->addTag('node_access'); $nids = $select->execute()->fetchCol(); @@ -2505,7 +2506,7 @@ function node_page_default() { $build = node_view_multiple($nodes); // 'rss.xml' is a path, not a file, registered in node_menu(). - drupal_add_feed('rss.xml', variable_get('site_name', 'Drupal') . ' ' . t('RSS')); + drupal_add_feed('rss.xml', $site_config->get('site_name') . ' ' . t('RSS')); $build['pager'] = array( '#theme' => 'pager', '#weight' => 5, @@ -2513,7 +2514,7 @@ function node_page_default() { drupal_set_title(''); } else { - drupal_set_title(t('Welcome to @site-name', array('@site-name' => variable_get('site_name', 'Drupal'))), PASS_THROUGH); + drupal_set_title(t('Welcome to @site-name', array('@site-name' => $site_config->get('site_name'))), PASS_THROUGH); $default_message = '

' . t('No front page content has been created yet.') . '

'; diff --git a/core/modules/overlay/overlay.module b/core/modules/overlay/overlay.module index 7fb7e0b..6829f5b 100644 --- a/core/modules/overlay/overlay.module +++ b/core/modules/overlay/overlay.module @@ -661,7 +661,7 @@ function overlay_overlay_parent_initialize() { // Let the client side know which paths are administrative. $paths = path_get_admin_paths(); foreach ($paths as &$type) { - $type = str_replace('', variable_get('site_frontpage', 'user'), $type); + $type = str_replace('', config('system.site')->get('site_frontpage'), $type); } drupal_add_js(array('overlay' => array('paths' => $paths)), 'setting'); $path_prefixes = array(); diff --git a/core/modules/simpletest/lib/Drupal/simpletest/Tests/SimpleTestTest.php b/core/modules/simpletest/lib/Drupal/simpletest/Tests/SimpleTestTest.php index 8ef2c14..0b670ff 100644 --- a/core/modules/simpletest/lib/Drupal/simpletest/Tests/SimpleTestTest.php +++ b/core/modules/simpletest/lib/Drupal/simpletest/Tests/SimpleTestTest.php @@ -51,7 +51,7 @@ class SimpleTestTest extends WebTestBase { if (!$this->inCURL()) { $this->drupalGet('node'); $this->assertTrue($this->drupalGetHeader('Date'), t('An HTTP header was received.')); - $this->assertTitle(t('Welcome to @site-name | @site-name', array('@site-name' => variable_get('site_name', 'Drupal'))), t('Site title matches.')); + $this->assertTitle(t('Welcome to @site-name | @site-name', array('@site-name' => config('system.site')->get('site_name'))), t('Site title matches.')); $this->assertNoTitle('Foo', t('Site title does not match.')); // Make sure that we are locked out of the installer when prefixing // using the user-agent header. This is an important security check. diff --git a/core/modules/simpletest/lib/Drupal/simpletest/WebTestBase.php b/core/modules/simpletest/lib/Drupal/simpletest/WebTestBase.php index f1a5f95..4afd249 100644 --- a/core/modules/simpletest/lib/Drupal/simpletest/WebTestBase.php +++ b/core/modules/simpletest/lib/Drupal/simpletest/WebTestBase.php @@ -664,7 +664,7 @@ abstract class WebTestBase extends TestBase { // Restore necessary variables. variable_set('install_task', 'done'); - variable_set('site_mail', 'simpletest@example.com'); + config('system.site')->set('site_mail', 'simpletest@example.com')->save(); variable_set('date_default_timezone', date_default_timezone_get()); // Set up English language. diff --git a/core/modules/system/config/system.site.yml b/core/modules/system/config/system.site.yml new file mode 100644 index 0000000..e039269 --- /dev/null +++ b/core/modules/system/config/system.site.yml @@ -0,0 +1,7 @@ +site_name: Drupal +site_mail: '' +site_slogan: '' +default_nodes_main: '10' +site_frontpage: user +site_404: '' +site_403: '' diff --git a/core/modules/system/language.api.php b/core/modules/system/language.api.php index 903de4a..953c07c 100644 --- a/core/modules/system/language.api.php +++ b/core/modules/system/language.api.php @@ -28,11 +28,11 @@ function hook_language_init() { switch (drupal_container()->get(LANGUAGE_TYPE_INTERFACE)->langcode) { case 'it': - $conf['site_name'] = 'Il mio sito Drupal'; + $conf['system.site']['site_name'] = 'Il mio sito Drupal'; break; case 'fr': - $conf['site_name'] = 'Mon site Drupal'; + $conf['system.site']['site_name'] = 'Mon site Drupal'; break; } } diff --git a/core/modules/system/lib/Drupal/system/Tests/Bootstrap/PageCacheTest.php b/core/modules/system/lib/Drupal/system/Tests/Bootstrap/PageCacheTest.php index 6355d54..946fe2d 100644 --- a/core/modules/system/lib/Drupal/system/Tests/Bootstrap/PageCacheTest.php +++ b/core/modules/system/lib/Drupal/system/Tests/Bootstrap/PageCacheTest.php @@ -25,8 +25,10 @@ class PageCacheTest extends WebTestBase { function setUp() { parent::setUp(array('node', 'system_test')); - variable_set('site_name', 'Drupal'); - variable_set('site_frontpage', 'node'); + config('system.site') + ->set('site_name', 'Drupal') + ->set('site_frontpage', 'node') + ->save(); } /** @@ -140,7 +142,7 @@ class PageCacheTest extends WebTestBase { $this->drupalGet(''); $this->assertEqual($this->drupalGetHeader('X-Drupal-Cache'), 'HIT', t('Page was cached.')); $this->assertFalse($this->drupalGetHeader('Content-Encoding'), t('A Content-Encoding header was not sent.')); - $this->assertTitle(t('Welcome to @site-name | @site-name', array('@site-name' => variable_get('site_name', 'Drupal'))), t('Site title matches.')); + $this->assertTitle(t('Welcome to @site-name | @site-name', array('@site-name' => config('system.site')->get('site_name'))), t('Site title matches.')); $this->assertRaw('', t('Page was not compressed.')); } } diff --git a/core/modules/system/lib/Drupal/system/Tests/Common/HttpRequestTest.php b/core/modules/system/lib/Drupal/system/Tests/Common/HttpRequestTest.php index 549c439..d69e2e5 100644 --- a/core/modules/system/lib/Drupal/system/Tests/Common/HttpRequestTest.php +++ b/core/modules/system/lib/Drupal/system/Tests/Common/HttpRequestTest.php @@ -41,7 +41,7 @@ class HttpRequestTest extends WebTestBase { $result = drupal_http_request(url('node', array('absolute' => TRUE))); $this->assertEqual($result->code, 200, t('Fetched page successfully.')); $this->drupalSetContent($result->data); - $this->assertTitle(t('Welcome to @site-name | @site-name', array('@site-name' => variable_get('site_name', 'Drupal'))), t('Site title matches.')); + $this->assertTitle(t('Welcome to @site-name | @site-name', array('@site-name' => config('system.site')->get('site_name'))), t('Site title matches.')); // Test that code and status message is returned. $result = drupal_http_request(url('pagedoesnotexist', array('absolute' => TRUE))); diff --git a/core/modules/system/lib/Drupal/system/Tests/Path/MatchPathTest.php b/core/modules/system/lib/Drupal/system/Tests/Path/MatchPathTest.php index db11056..c1dd421 100644 --- a/core/modules/system/lib/Drupal/system/Tests/Path/MatchPathTest.php +++ b/core/modules/system/lib/Drupal/system/Tests/Path/MatchPathTest.php @@ -31,7 +31,7 @@ class MatchPathTest extends WebTestBase { // Set up a random site front page to test the '' placeholder. $this->front = $this->randomName(); - variable_set('site_frontpage', $this->front); + config('system.site')->set('site_frontpage', $this->front)->save(); // Refresh our static variables from the database. $this->refreshVariables(); } diff --git a/core/modules/system/lib/Drupal/system/Tests/System/AccessDeniedTest.php b/core/modules/system/lib/Drupal/system/Tests/System/AccessDeniedTest.php index 6c17c16..febf1ab 100644 --- a/core/modules/system/lib/Drupal/system/Tests/System/AccessDeniedTest.php +++ b/core/modules/system/lib/Drupal/system/Tests/System/AccessDeniedTest.php @@ -73,7 +73,7 @@ class AccessDeniedTest extends WebTestBase { // Log back in, set the custom 403 page to /user and remove the block $this->drupalLogin($this->admin_user); - variable_set('site_403', 'user'); + config('system.site')->set('site_403', 'user')->save(); $this->drupalPost('admin/structure/block', array('blocks[user_login][region]' => '-1'), t('Save blocks')); // Check that we can log in from the 403 page. diff --git a/core/modules/system/lib/Drupal/system/Tests/System/FrontPageTest.php b/core/modules/system/lib/Drupal/system/Tests/System/FrontPageTest.php index 25fe590..5839f11 100644 --- a/core/modules/system/lib/Drupal/system/Tests/System/FrontPageTest.php +++ b/core/modules/system/lib/Drupal/system/Tests/System/FrontPageTest.php @@ -31,7 +31,7 @@ class FrontPageTest extends WebTestBase { $this->node_path = "node/" . $this->drupalCreateNode(array('promote' => 1))->nid; // Configure 'node' as front page. - variable_set('site_frontpage', 'node'); + config('system.site')->set('site_frontpage', 'node')->save(); // Enable front page logging in system_test.module. variable_set('front_page_output', 1); } diff --git a/core/modules/system/lib/Drupal/system/Tests/System/SiteMaintenanceTest.php b/core/modules/system/lib/Drupal/system/Tests/System/SiteMaintenanceTest.php index f586ca1..2ce7ee2 100644 --- a/core/modules/system/lib/Drupal/system/Tests/System/SiteMaintenanceTest.php +++ b/core/modules/system/lib/Drupal/system/Tests/System/SiteMaintenanceTest.php @@ -27,7 +27,7 @@ class SiteMaintenanceTest extends WebTestBase { parent::setUp(array('node')); // Configure 'node' as front page. - variable_set('site_frontpage', 'node'); + config('system.site')->set('site_frontpage', 'node')->save(); // Create a user allowed to access site in maintenance mode. $this->user = $this->drupalCreateUser(array('access site in maintenance mode')); @@ -48,7 +48,7 @@ class SiteMaintenanceTest extends WebTestBase { $admin_message = t('Operating in maintenance mode. Go online.', array('@url' => url('admin/config/development/maintenance'))); $user_message = t('Operating in maintenance mode.'); - $offline_message = t('@site is currently under maintenance. We should be back shortly. Thank you for your patience.', array('@site' => variable_get('site_name', 'Drupal'))); + $offline_message = t('@site is currently under maintenance. We should be back shortly. Thank you for your patience.', array('@site' => config('system.site')->get('site_name'))); $this->drupalGet(''); $this->assertRaw($admin_message, t('Found the site maintenance mode message.')); diff --git a/core/modules/system/lib/Drupal/system/Tests/System/TokenReplaceTest.php b/core/modules/system/lib/Drupal/system/Tests/System/TokenReplaceTest.php index 0ea8ca9..2fbbf4e 100644 --- a/core/modules/system/lib/Drupal/system/Tests/System/TokenReplaceTest.php +++ b/core/modules/system/lib/Drupal/system/Tests/System/TokenReplaceTest.php @@ -111,13 +111,15 @@ class TokenReplaceTest extends WebTestBase { ); // Set a few site variables. - variable_set('site_name', 'Drupal'); - variable_set('site_slogan', 'Slogan'); + config('system.site') + ->set('site_name', 'Drupal') + ->set('site_slogan', 'Slogan') + ->save(); // Generate and test sanitized tokens. $tests = array(); - $tests['[site:name]'] = check_plain(variable_get('site_name', 'Drupal')); - $tests['[site:slogan]'] = check_plain(variable_get('site_slogan', '')); + $tests['[site:name]'] = check_plain(config('system.site')->get('site_name')); + $tests['[site:slogan]'] = check_plain(config('system.site')->get('site_slogan')); $tests['[site:mail]'] = 'simpletest@example.com'; $tests['[site:url]'] = url('', $url_options); $tests['[site:url-brief]'] = preg_replace(array('!^https?://!', '!/$!'), '', url('', $url_options)); @@ -132,8 +134,8 @@ class TokenReplaceTest extends WebTestBase { } // Generate and test unsanitized tokens. - $tests['[site:name]'] = variable_get('site_name', 'Drupal'); - $tests['[site:slogan]'] = variable_get('site_slogan', ''); + $tests['[site:name]'] = config('system.site')->get('site_name'); + $tests['[site:slogan]'] = config('system.site')->get('site_slogan'); foreach ($tests as $input => $expected) { $output = token_replace($input, array(), array('language' => $language_interface, 'sanitize' => FALSE)); diff --git a/core/modules/system/lib/Drupal/system/Tests/Theme/FunctionsTest.php b/core/modules/system/lib/Drupal/system/Tests/Theme/FunctionsTest.php index cb25c6f..e954c19 100644 --- a/core/modules/system/lib/Drupal/system/Tests/Theme/FunctionsTest.php +++ b/core/modules/system/lib/Drupal/system/Tests/Theme/FunctionsTest.php @@ -98,7 +98,7 @@ class FunctionsTest extends WebTestBase { // Required to verify the "active" class in expected links below, and // because the current path is different when running tests manually via // simpletest.module ('batch') and via the testing framework (''). - _current_path(variable_get('site_frontpage', 'user')); + _current_path(config('system.site')->get('site_frontpage')); // Verify that a list of links is properly rendered. $variables = array(); diff --git a/core/modules/system/lib/Drupal/system/Tests/Theme/ThemeTest.php b/core/modules/system/lib/Drupal/system/Tests/Theme/ThemeTest.php index abd9830..c2d0358 100644 --- a/core/modules/system/lib/Drupal/system/Tests/Theme/ThemeTest.php +++ b/core/modules/system/lib/Drupal/system/Tests/Theme/ThemeTest.php @@ -32,7 +32,7 @@ class ThemeTest extends WebTestBase { function testThemeSuggestions() { // Set the front page as something random otherwise the CLI // test runner fails. - variable_set('site_frontpage', 'nobody-home'); + config('system.site')->set('site_frontpage', 'nobody-home')->save(); $args = array('node', '1', 'edit'); $suggestions = theme_get_suggestions($args, 'page'); $this->assertEqual($suggestions, array('page__node', 'page__node__%', 'page__node__1', 'page__node__edit'), t('Found expected node edit page suggestions')); @@ -76,7 +76,8 @@ class ThemeTest extends WebTestBase { $original_path = _current_path(); // Set the current path to node because theme_get_suggestions() will query // it to see if we are on the front page. - variable_set('site_frontpage', 'node'); + config('system.site')->set('site_frontpage', 'node')->save(); + drupal_static_reset('drupal_is_front_page'); _current_path('node'); $suggestions = theme_get_suggestions(array('node'), 'page'); // Set it back to not annoy the batch runner. diff --git a/core/modules/system/system.admin.inc b/core/modules/system/system.admin.inc index de17cef..cdcc091 100644 --- a/core/modules/system/system.admin.inc +++ b/core/modules/system/system.admin.inc @@ -1465,6 +1465,12 @@ function system_ip_blocking_delete_submit($form, &$form_state) { * @see system_settings_form() */ function system_site_information_settings() { + $site_config = config('system.site'); + $site_mail = $site_config->get('site_mail'); + if (empty($site_mail)) { + $site_mail = ini_get('sendmail_from'); + } + $form['site_information'] = array( '#type' => 'fieldset', '#title' => t('Site details'), @@ -1472,19 +1478,19 @@ function system_site_information_settings() { $form['site_information']['site_name'] = array( '#type' => 'textfield', '#title' => t('Site name'), - '#default_value' => variable_get('site_name', 'Drupal'), + '#default_value' => $site_config->get('site_name'), '#required' => TRUE ); $form['site_information']['site_slogan'] = array( '#type' => 'textfield', '#title' => t('Slogan'), - '#default_value' => variable_get('site_slogan', ''), + '#default_value' => $site_config->get('site_slogan'), '#description' => t("How this is used depends on your site's theme."), ); $form['site_information']['site_mail'] = array( '#type' => 'email', '#title' => t('E-mail address'), - '#default_value' => variable_get('site_mail', ini_get('sendmail_from')), + '#default_value' => $site_mail, '#description' => t("The From address in automated e-mails sent during registration and new password requests, and other notifications. (Use an address ending in your site's domain to help prevent this e-mail being flagged as spam.)"), '#required' => TRUE, ); @@ -1495,17 +1501,17 @@ function system_site_information_settings() { $form['front_page']['site_frontpage'] = array( '#type' => 'textfield', '#title' => t('Default front page'), - '#default_value' => (variable_get('site_frontpage') != 'user' ? drupal_get_path_alias(variable_get('site_frontpage', 'user')) : ''), + '#default_value' => ($site_config->get('site_frontpage') != 'user' ? drupal_get_path_alias($site_config->get('site_frontpage')) : ''), '#size' => 40, '#description' => t('Optionally, specify a relative URL to display as the front page. Leave blank to display the default content feed.'), '#field_prefix' => url(NULL, array('absolute' => TRUE)), ); $form['front_page']['default_nodes_main'] = array( '#type' => 'select', '#title' => t('Number of posts on front page'), - '#default_value' => variable_get('default_nodes_main', 10), + '#default_value' => $site_config->get('default_nodes_main'), '#options' => drupal_map_assoc(array(1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 15, 20, 25, 30)), '#description' => t('The maximum number of posts displayed on overview pages such as the front page.'), - '#access' => (variable_get('site_frontpage') == 'node'), + '#access' => ($site_config->get('site_frontpage') == 'node'), ); $form['error_page'] = array( '#type' => 'fieldset', @@ -1514,7 +1520,7 @@ function system_site_information_settings() { $form['error_page']['site_403'] = array( '#type' => 'textfield', '#title' => t('Default 403 (access denied) page'), - '#default_value' => variable_get('site_403', ''), + '#default_value' => $site_config->get('site_403'), '#size' => 40, '#description' => t('This page is displayed when the requested document is denied to the current user. Leave blank to display a generic "access denied" page.'), '#field_prefix' => url(NULL, array('absolute' => TRUE)), @@ -1522,7 +1528,7 @@ function system_site_information_settings() { $form['error_page']['site_404'] = array( '#type' => 'textfield', '#title' => t('Default 404 (not found) page'), - '#default_value' => variable_get('site_404', ''), + '#default_value' => $site_config->get('site_404'), '#size' => 40, '#description' => t('This page is displayed when no other content matches the requested document. Leave blank to display a generic "page not found" page.'), '#field_prefix' => url(NULL, array('absolute' => TRUE)), @@ -1530,6 +1536,8 @@ function system_site_information_settings() { $form['#validate'][] = 'system_site_information_settings_validate'; + // @todo Replace with configuration system settings form. + $form['#submit'][] = 'system_site_information_settings_submit'; return system_settings_form($form); } @@ -1568,6 +1576,21 @@ function system_site_information_settings_validate($form, &$form_state) { } /** + * Form submission handler for system_site_information_settings(). + */ +function system_site_information_settings_submit($form, &$form_state) { + config('system.site') + ->set('site_name', $form_state['values']['site_name']) + ->set('site_mail', $form_state['values']['site_mail']) + ->set('site_slogan', $form_state['values']['site_slogan']) + ->set('default_nodes_main', $form_state['values']['default_nodes_main']) + ->set('site_frontpage', $form_state['values']['site_frontpage']) + ->set('site_403', $form_state['values']['site_403']) + ->set('site_404', $form_state['values']['site_404']) + ->save(); +} + +/** * Form builder; Cron form. * * @see system_settings_form() @@ -2231,7 +2254,7 @@ function system_site_maintenance_mode() { $form['maintenance_mode_message'] = array( '#type' => 'textarea', '#title' => t('Message to display when in maintenance mode'), - '#default_value' => variable_get('maintenance_mode_message', t('@site is currently under maintenance. We should be back shortly. Thank you for your patience.', array('@site' => variable_get('site_name', 'Drupal')))), + '#default_value' => variable_get('maintenance_mode_message', t('@site is currently under maintenance. We should be back shortly. Thank you for your patience.', array('@site' => config('system.site')->get('site_name')))), ); return system_settings_form($form); diff --git a/core/modules/system/system.api.php b/core/modules/system/system.api.php index 9ad2ba1..b3aab28 100644 --- a/core/modules/system/system.api.php +++ b/core/modules/system/system.api.php @@ -1526,7 +1526,7 @@ function hook_mail_alter(&$message) { $message['send'] = FALSE; return; } - $message['body'][] = "--\nMail sent out from " . variable_get('site_name', t('Drupal')); + $message['body'][] = "--\nMail sent out from " . config('system.site')->get('site_name'); } } @@ -1914,7 +1914,7 @@ function hook_watchdog(array $log_entry) { $to = 'someone@example.com'; $params = array(); $params['subject'] = t('[@site_name] @severity_desc: Alert from your web site', array( - '@site_name' => variable_get('site_name', 'Drupal'), + '@site_name' => config('system.site')->get('site_name'), '@severity_desc' => $severity_list[$log_entry['severity']], )); @@ -1980,7 +1980,7 @@ function hook_mail($key, &$message, $params) { $account = $params['account']; $context = $params['context']; $variables = array( - '%site_name' => variable_get('site_name', 'Drupal'), + '%site_name' => config('system.site')->get('site_name'), '%username' => user_format_name($account), ); if ($context['hook'] == 'taxonomy') { diff --git a/core/modules/system/system.install b/core/modules/system/system.install index 9ba7c4c..c8b0337 100644 --- a/core/modules/system/system.install +++ b/core/modules/system/system.install @@ -1925,6 +1925,23 @@ function system_update_8010() { } /** + * Moves site system settings from variable to config. + * + * @ingroup config_upgrade + */ +function system_update_8011() { + update_variables_to_config('system.site', array( + 'site_name' => 'site_name', + 'site_mail' => 'site_mail', + 'site_slogan' => 'site_slogan', + 'default_nodes_main' => 'default_nodes_main', + 'site_frontpage' => 'site_frontpage', + 'site_403' => 'site_403', + 'site_404' => 'site_404', + )); +} + +/** * @} End of "defgroup updates-7.x-to-8.x". * The next series of updates should start at 9000. */ diff --git a/core/modules/system/system.tokens.inc b/core/modules/system/system.tokens.inc index 2492890..e872f9c 100644 --- a/core/modules/system/system.tokens.inc +++ b/core/modules/system/system.tokens.inc @@ -145,17 +145,17 @@ function system_tokens($type, $tokens, array $data = array(), array $options = a foreach ($tokens as $name => $original) { switch ($name) { case 'name': - $site_name = variable_get('site_name', 'Drupal'); + $site_name = config('system.site')->get('site_name'); $replacements[$original] = $sanitize ? check_plain($site_name) : $site_name; break; case 'slogan': - $slogan = variable_get('site_slogan', ''); + $slogan = config('system.site')->get('site_slogan'); $replacements[$original] = $sanitize ? check_plain($slogan) : $slogan; break; case 'mail': - $replacements[$original] = variable_get('site_mail', ''); + $replacements[$original] = config('system.site')->get('site_mail'); break; case 'url': diff --git a/core/modules/system/tests/menu.test b/core/modules/system/tests/menu.test index dbd94a1..0759a2f 100644 --- a/core/modules/system/tests/menu.test +++ b/core/modules/system/tests/menu.test @@ -248,7 +248,7 @@ class MenuRouterTestCase extends WebTestBase { function testMaintenanceModeLoginPaths() { variable_set('maintenance_mode', TRUE); - $offline_message = t('@site is currently under maintenance. We should be back shortly. Thank you for your patience.', array('@site' => variable_get('site_name', 'Drupal'))); + $offline_message = t('@site is currently under maintenance. We should be back shortly. Thank you for your patience.', array('@site' => config('system.site')->get('site_name'))); $this->drupalGet('node'); $this->assertText($offline_message); $this->drupalGet('menu_login_callback'); @@ -1636,8 +1636,10 @@ class MenuTrailTestCase extends MenuWebTestCase { */ function testCustom403And404Pages() { // Set the custom 403 and 404 pages we will use. - variable_set('site_403', 'menu-test/custom-403-page'); - variable_set('site_404', 'menu-test/custom-404-page'); + config('system.site') + ->set('site_403', 'menu-test/custom-403-page') + ->set('site_404', 'menu-test/custom-404-page') + ->save(); // Define the paths we'll visit to trigger 403 and 404 responses during // this test, and the expected active trail for each case. diff --git a/core/modules/system/tests/upgrade/upgrade.test b/core/modules/system/tests/upgrade/upgrade.test index 97b85be..f593492 100644 --- a/core/modules/system/tests/upgrade/upgrade.test +++ b/core/modules/system/tests/upgrade/upgrade.test @@ -127,10 +127,6 @@ abstract class UpgradePathTestCase extends WebTestBase { // Generate and set a D8-compatible session cookie. $this->prepareD8Session(); - // Restore necessary variables. - // @todo Convert into config('system.site')->set('mail')? - $this->variable_set('site_mail', 'simpletest@example.com'); - drupal_set_time_limit($this->timeLimit); $this->setup = TRUE; } diff --git a/core/modules/taxonomy/taxonomy.pages.inc b/core/modules/taxonomy/taxonomy.pages.inc index c84a6bd..7c81d4c 100644 --- a/core/modules/taxonomy/taxonomy.pages.inc +++ b/core/modules/taxonomy/taxonomy.pages.inc @@ -42,7 +42,7 @@ function taxonomy_term_page(Term $term) { 'term' => taxonomy_term_view($term, 'full'), ); - if ($nids = taxonomy_select_nodes($term->tid, TRUE, variable_get('default_nodes_main', 10))) { + if ($nids = taxonomy_select_nodes($term->tid, TRUE, config('system.site')->get('default_nodes_main'))) { $nodes = node_load_multiple($nids); $build += node_view_multiple($nodes); $build['pager'] = array( @@ -68,7 +68,7 @@ function taxonomy_term_page(Term $term) { */ function taxonomy_term_feed(Term $term) { $channel['link'] = url('taxonomy/term/' . $term->tid, array('absolute' => TRUE)); - $channel['title'] = variable_get('site_name', 'Drupal') . ' - ' . $term->name; + $channel['title'] = config('system.site')->get('site_name') . ' - ' . $term->name; // Only display the description if we have a single term, to avoid clutter and confusion. // HTML will be removed from feed description. $channel['description'] = check_markup($term->description, $term->format, '', TRUE); diff --git a/core/modules/update/update.module b/core/modules/update/update.module index 2293799..83c7c40 100644 --- a/core/modules/update/update.module +++ b/core/modules/update/update.module @@ -521,7 +521,7 @@ function _update_get_cached_available_releases() { function update_mail($key, &$message, $params) { $language = $message['language']; $langcode = $language->langcode; - $message['subject'] .= t('New release(s) available for !site_name', array('!site_name' => variable_get('site_name', 'Drupal')), array('langcode' => $langcode)); + $message['subject'] .= t('New release(s) available for !site_name', array('!site_name' => config('system.site')->get('site_name')), array('langcode' => $langcode)); foreach ($params as $msg_type => $msg_reason) { $message['body'][] = _update_message_text($msg_type, $msg_reason, FALSE, $language); } diff --git a/core/modules/user/user.module b/core/modules/user/user.module index 6a04d7c..a5f2cfa 100644 --- a/core/modules/user/user.module +++ b/core/modules/user/user.module @@ -1879,7 +1879,7 @@ function user_login($form, &$form_state) { ), ); - $form['name']['#description'] = t('Enter your @s username.', array('@s' => variable_get('site_name', 'Drupal'))); + $form['name']['#description'] = t('Enter your @s username.', array('@s' => config('system.site')->get('site_name'))); $form['pass'] = array('#type' => 'password', '#title' => t('Password'), '#description' => t('Enter the password that accompanies your username.'), @@ -3308,7 +3308,11 @@ function _user_mail_notify($op, $account, $language = NULL) { if ($op == 'register_pending_approval') { // If a user registered requiring admin approval, notify the admin, too. // We use the site default language for this. - drupal_mail('user', 'register_pending_approval_admin', variable_get('site_mail', ini_get('sendmail_from')), language_default(), $params); + $site_mail = config('system.site')->get('site_mail'); + if (empty($site_mail)) { + $site_mail = ini_get('sendmail_from'); + } + drupal_mail('user', 'register_pending_approval_admin', $site_mail, language_default(), $params); } } return empty($mail) ? NULL : $mail['result']; diff --git a/core/themes/bartik/template.php b/core/themes/bartik/template.php index 7db1a31..5891de2 100644 --- a/core/themes/bartik/template.php +++ b/core/themes/bartik/template.php @@ -43,6 +43,7 @@ function bartik_process_html(&$variables) { * Implements hook_process_HOOK() for page.tpl.php. */ function bartik_process_page(&$variables) { + $site_config = config('system.site'); // Hook into color.module. if (module_exists('color')) { _color_page_alter($variables); @@ -53,11 +54,11 @@ function bartik_process_page(&$variables) { $variables['hide_site_slogan'] = theme_get_setting('toggle_slogan') ? FALSE : TRUE; if ($variables['hide_site_name']) { // If toggle_name is FALSE, the site_name will be empty, so we rebuild it. - $variables['site_name'] = filter_xss_admin(variable_get('site_name', 'Drupal')); + $variables['site_name'] = filter_xss_admin($site_config->get('site_name')); } if ($variables['hide_site_slogan']) { // If toggle_site_slogan is FALSE, the site_slogan will be empty, so we rebuild it. - $variables['site_slogan'] = filter_xss_admin(variable_get('site_slogan', '')); + $variables['site_slogan'] = filter_xss_admin($site_config->get('site_slogan')); } // Since the title and the shortcut link are both block level elements, // positioning them next to each other is much simpler with a wrapper div. @@ -94,17 +95,18 @@ function bartik_preprocess_maintenance_page(&$variables) { * Implements hook_process_HOOK() for maintenance-page.tpl.php. */ function bartik_process_maintenance_page(&$variables) { + $site_config = config('system.site'); // Always print the site name and slogan, but if they are toggled off, we'll // just hide them visually. $variables['hide_site_name'] = theme_get_setting('toggle_name') ? FALSE : TRUE; $variables['hide_site_slogan'] = theme_get_setting('toggle_slogan') ? FALSE : TRUE; if ($variables['hide_site_name']) { // If toggle_name is FALSE, the site_name will be empty, so we rebuild it. - $variables['site_name'] = filter_xss_admin(variable_get('site_name', 'Drupal')); + $variables['site_name'] = filter_xss_admin($site_config->get('site_name')); } if ($variables['hide_site_slogan']) { // If toggle_site_slogan is FALSE, the site_slogan will be empty, so we rebuild it. - $variables['site_slogan'] = filter_xss_admin(variable_get('site_slogan', '')); + $variables['site_slogan'] = filter_xss_admin($site_config->get('site_slogan')); } } diff --git a/profiles/minimal/minimal.install b/profiles/minimal/minimal.install index 50ef446..2ada0df 100644 --- a/profiles/minimal/minimal.install +++ b/profiles/minimal/minimal.install @@ -79,7 +79,7 @@ function minimal_install() { $query->execute(); // Set front page to "node". - variable_set('site_frontpage', 'node'); + config('system.site')->set('site_frontpage', 'node')->save(); // Allow visitor account creation, but with administrative approval. variable_set('user_register', USER_REGISTER_VISITORS_ADMINISTRATIVE_APPROVAL); diff --git a/profiles/standard/standard.install b/profiles/standard/standard.install index 5c7032a..276fa82 100644 --- a/profiles/standard/standard.install +++ b/profiles/standard/standard.install @@ -206,7 +206,7 @@ function standard_install() { $query->execute(); // Set front page to "node". - variable_set('site_frontpage', 'node'); + config('system.site')->set('site_frontpage', 'node')->save(); // Insert default pre-defined node types into the database. For a complete // list of available node type attributes, refer to the node type API