#898546: Cleanup the Filter upgrade path, part 1. From: Damien Tournoud --- filter/filter.install | 173 +++++++++++++++++++++++++------------------------ user/user.install | 2 - 2 files changed, 89 insertions(+), 86 deletions(-) diff --git modules/filter/filter.install modules/filter/filter.install index ad4e65b..a9ce6b6 100644 --- modules/filter/filter.install +++ modules/filter/filter.install @@ -157,22 +157,13 @@ function filter_update_dependencies() { */ /** - * Increase the size of {filters}.weight and add {filter_formats}.weight. + * Upgrade the {filter_formats} table and rename it to {filter_format}. */ function filter_update_7000() { - // The list index will be recreated by filter_update_7003(). - db_drop_index('filters', 'list'); - - // Change the weight column of the filter table to normal (ie. non tiny) int. - db_change_field('filters', 'weight', 'weight', array( - 'type' => 'int', - 'not null' => TRUE, - 'default' => 0, - 'description' => 'Weight of filter within format.', - )); + db_rename_table('filter_formats', 'filter_format'); // Add a new filter_format.weight column. - db_add_field('filter_formats', 'weight', array( + db_add_field('filter_format', 'weight', array( 'type' => 'int', 'not null' => TRUE, 'default' => 0, @@ -188,7 +179,7 @@ function filter_update_7000() { * Break out "escape HTML filter" option to its own filter. */ function filter_update_7001() { - $result = db_query("SELECT format FROM {filter_formats}")->fetchCol(); + $result = db_query("SELECT format FROM {filter_format}")->fetchCol(); $insert = db_insert('filters')->fields(array('format', 'module', 'delta', 'weight')); foreach ($result as $format_id) { @@ -208,23 +199,62 @@ function filter_update_7001() { } /** - * Rename {filters} table to {filter} and {filter_formats} table to {filter_format}. - */ -function filter_update_7002() { - db_rename_table('filters', 'filter'); - db_rename_table('filter_formats', 'filter_format'); -} - -/** - * Remove hardcoded numeric deltas from all filters in core. + * Upgrade the {filter} table for core filters. */ function filter_update_7003() { - // Duplicates the filter table since core cannot take care of the potential + // Duplicates the {filters} table since core cannot take care of the potential // contributed module filters. - db_rename_table('filter', 'd6_upgrade_filter'); + db_rename_table('filters', 'd6_upgrade_filter'); // Creates the Drupal 7 filter table. - $schema = filter_schema(); - db_create_table('filter', $schema['filter']); + $filter_table = array( + 'description' => 'Table that maps filters (HTML corrector) to text formats (Filtered HTML).', + 'fields' => array( + 'format' => array( + 'type' => 'int', + 'not null' => TRUE, + 'default' => 0, + 'description' => 'Foreign key: The {filter_format}.format to which this filter is assigned.', + ), + 'module' => array( + 'type' => 'varchar', + 'length' => 64, + 'not null' => TRUE, + 'default' => '', + 'description' => 'The origin module of the filter.', + ), + 'name' => array( + 'type' => 'varchar', + 'length' => 32, + 'not null' => TRUE, + 'default' => '', + 'description' => 'Name of the filter being referenced.', + ), + 'weight' => array( + 'type' => 'int', + 'not null' => TRUE, + 'default' => 0, + 'description' => 'Weight of filter within format.', + ), + 'status' => array( + 'type' => 'int', + 'not null' => TRUE, + 'default' => 0, + 'description' => 'Filter enabled status. (1 = enabled, 0 = disabled)', + ), + 'settings' => array( + 'type' => 'blob', + 'not null' => FALSE, + 'size' => 'big', + 'serialize' => TRUE, + 'description' => 'A serialized array of name value pairs that store the filter settings for the specific format.', + ), + ), + 'primary key' => array('format', 'name'), + 'indexes' => array( + 'list' => array('weight', 'module', 'name'), + ), + ); + db_create_table('filter', $filter_table); // Get an array of the renamed filter deltas, organized by module. $renamed_deltas = array( @@ -244,19 +274,44 @@ function filter_update_7003() { // each record from the old to the new table. foreach ($renamed_deltas as $module => $deltas) { foreach ($deltas as $old_delta => $new_name) { - $query = db_select('d6_upgrade_filter'); - $query->fields('d6_upgrade_filter', array('format', 'weight')); - $query->condition('module', $module); - $query->condition('delta', $old_delta); - $query->distinct(); - $result = $query->execute(); - foreach ($result as $record) { + $query = db_select('d6_upgrade_filter') + ->fields('d6_upgrade_filter', array('format', 'weight')) + ->condition('module', $module) + ->condition('delta', $old_delta) + ->distinct(); + + foreach ($query->execute() as $record) { + // Port the filter settings. + $settings = array(); + if ($new_name == 'filter_html') { + if ($setting = variable_get("allowed_html_{$record->format}", NULL)) { + $settings['allowed_html'] = $setting; + variable_del("allowed_html_{$record->format}"); + } + if ($setting = variable_get("filter_html_help_{$record->format}", NULL)) { + $settings['filter_html_help'] = $setting; + variable_del("filter_html_help_{$record->format}"); + } + if ($setting = variable_get("filter_html_nofollow_{$record->format}", NULL)) { + $settings['filter_html_nofollow'] = $setting; + variable_del("filter_html_nofollow_{$record->format}"); + } + } + elseif ($new_name == 'filter_url') { + if ($setting = variable_get("filter_url_length_{$record->format}", NULL)) { + $settings['filter_url_length'] = $setting; + variable_del("filter_url_length_{$record->format}"); + } + } + db_insert('filter') ->fields(array( 'format' => $record->format, 'module' => $module, 'name' => $new_name, 'weight' => $record->weight, + 'settings' => serialize($settings), + 'status' => 1, )) ->execute(); } @@ -269,49 +324,6 @@ function filter_update_7003() { } /** - * Move filter settings storage into {filter} table. - */ -function filter_update_7004() { - // Enable all existing filters ({filter} contained only enabled previously). - db_update('filter') - ->fields(array('status' => '1')) - ->execute(); - - // Move filter settings from system variables into {filter}.settings. - $filters = db_query("SELECT * FROM {filter} WHERE module = :name", array(':name' => 'filter')); - foreach ($filters as $filter) { - $settings = array(); - if ($filter->name == 'filter_html') { - if ($setting = variable_get("allowed_html_{$filter->format}", NULL)) { - $settings['allowed_html'] = $setting; - variable_del("allowed_html_{$filter->format}"); - } - if ($setting = variable_get("filter_html_help_{$filter->format}", NULL)) { - $settings['filter_html_help'] = $setting; - variable_del("filter_html_help_{$filter->format}"); - } - if ($setting = variable_get("filter_html_nofollow_{$filter->format}", NULL)) { - $settings['filter_html_nofollow'] = $setting; - variable_del("filter_html_nofollow_{$filter->format}"); - } - } - elseif ($filter->name == 'filter_url') { - if ($setting = variable_get("filter_url_length_{$filter->format}", NULL)) { - $settings['filter_url_length'] = $setting; - variable_del("filter_url_length_{$filter->format}"); - } - } - if (!empty($settings)) { - db_update('filter') - ->fields(array('settings' => serialize($settings))) - ->condition('format', $filter->format) - ->condition('name', $filter->name) - ->execute(); - } - } -} - -/** * Integrate text formats with the user permissions system. * * This function converts text format role assignments to use the new text @@ -408,15 +420,6 @@ function filter_update_7008() { * Converts fields that store serialized variables from text to blob. */ function filter_update_7009() { - $spec = array( - 'type' => 'blob', - 'not null' => FALSE, - 'size' => 'big', - 'serialize' => TRUE, - 'description' => 'A serialized array of name value pairs that store the filter settings for the specific format.', - ); - db_change_field('filter', 'settings', 'settings', $spec); - $schema = system_schema_cache_7054(); db_drop_table('cache_filter'); db_create_table('cache_filter', $schema); diff --git modules/user/user.install modules/user/user.install index 27bc5b4..ef7b4f4 100644 --- modules/user/user.install +++ modules/user/user.install @@ -349,7 +349,7 @@ function user_update_dependencies() { // user_update_7006 relies on filter_update_7002. // TODO: move user_update_7006 down below in the upgrade process. $dependencies['user'][7006] = array( - 'filter' => 7002, + 'filter' => 7003, ); // user_update_7013 relies on system_update_7060. $dependencies['user'][7013] = array(