Issue #1958702 by rootatwc: Fixed Aggregator variables to config update is never executed.

6a27d1b0 · catch · ee96cf6b · 6a27d1b0 · 6a27d1b0 · 6a27d1b0
Commit 6a27d1b0 authored Apr 03, 2013 by catch
3 changed files
--- a/core/modules/aggregator/aggregator.install
+++ b/core/modules/aggregator/aggregator.install
@@ -295,7 +295,7 @@ function aggregator_schema() {
 *
 * @ingroup config_upgrade
 */
-function aggregrator_update_8000() {
+function aggregator_update_8000() {
  update_variables_to_config('aggregator.settings', array(
    'aggregator_fetcher' => 'fetcher',
    'aggregator_parser' => 'parser',

--- a/core/modules/system/lib/Drupal/system/Tests/Upgrade/SystemUpgradePathTest.php
+++ b/core/modules/system/lib/Drupal/system/Tests/Upgrade/SystemUpgradePathTest.php
@@ -159,6 +159,16 @@ public function testVariableUpgrade() {
      'child_type' => 'book',
    );

+    $expected_config['aggregator.settings'] = array(
+      'fetcher' => 'test_fetcher',
+      'parser' => 'test_parser',
+      'processors' => array('test_processor'),
+      'items.allowed_html' => '<a>',
+      'items.teaser_length' => 6000,
+      'items.expire' => 10,
+      'source.list_max' => 5,
+    );
+
    foreach ($expected_config as $file => $values) {
      $config = config($file);
      $this->verbose(print_r($config->get(), TRUE));

--- a/core/modules/system/tests/upgrade/drupal-7.system.database.php
+++ b/core/modules/system/tests/upgrade/drupal-7.system.database.php
@@ -135,6 +135,34 @@
    'name' => 'fast_404_html',
    'value' => 's:168:"<!DOCTYPE html><html><head><title>Page Not Found</title></head><body><h1>Page Not Found</h1><p>The requested URL "@path" was not found on this server.</p></body></html>";',
  ))
+  ->values(array(
+    'name' => 'aggregator_fetcher',
+    'value' => 's:12:"test_fetcher";',
+  ))
+  ->values(array(
+    'name' => 'aggregator_parser',
+    'value' => 's:11:"test_parser";',
+  ))
+  ->values(array(
+    'name' => 'aggregator_processors',
+    'value' => 'a:1:{i:0;s:14:"test_processor";}',
+  ))
+  ->values(array(
+    'name' => 'aggregator_allowed_html_tags',
+    'value' => 's:3:"<a>";',
+  ))
+  ->values(array(
+    'name' => 'aggregator_teaser_length',
+    'value' => 'i:6000;',
+  ))
+  ->values(array(
+    'name' => 'aggregator_clear',
+    'value' => 'i:10;',
+  ))
+  ->values(array(
+    'name' => 'aggregator_summary_items',
+    'value' => 'i:5;',
+  ))
  ->execute();

 db_update('variable')