Skip to content

Commit

Permalink
Merge pull request #2761 from Nicofuma/ticket/12873
Browse files Browse the repository at this point in the history
[ticket/12873] Test the correct identifier in \phpbb\db\tools

* Nicofuma/ticket/12873:
  [ticket/12873] Add migration to rename the index
  [ticket/12873] Don not touch the existing migrations
  [ticket/12873] Test the good identifier in \phpbb\db\tools
  • Loading branch information
nickvergessen committed Aug 7, 2014
2 parents ab8a197 + 2b3e15c commit 9b36b52
Show file tree
Hide file tree
Showing 3 changed files with 47 additions and 9 deletions.
16 changes: 8 additions & 8 deletions phpBB/install/schemas/schema.json
Expand Up @@ -2397,21 +2397,21 @@
]
},
"KEYS": {
"unq_mtch": [
"UNIQUE",
[
"word_id",
"post_id",
"title_match"
]
],
"word_id": [
"INDEX",
"word_id"
],
"post_id": [
"INDEX",
"post_id"
],
"un_mtch": [
"UNIQUE",
[
"word_id",
"post_id",
"title_match"
]
]
}
},
Expand Down
38 changes: 38 additions & 0 deletions phpBB/phpbb/db/migration/data/v310/rename_too_long_indexes.php
@@ -0,0 +1,38 @@
<?php
/**
*
* This file is part of the phpBB Forum Software package.
*
* @copyright (c) phpBB Limited <https://www.phpbb.com>
* @license GNU General Public License, version 2 (GPL-2.0)
*
* For full copyright and license information, please see
* the docs/CREDITS.txt file.
*
*/

namespace phpbb\db\migration\data\v310;

class rename_too_long_indexes extends \phpbb\db\migration\migration
{
static public function depends_on()
{
return array('\phpbb\db\migration\data\v30x\release_3_0_0');
}

public function update_schema()
{
return array(
'drop_keys' => array(
$this->table_prefix . 'search_wordmatch' => array(
'unq_mtch',
),
),
'add_unique_index' => array(
$this->table_prefix . 'search_wordmatch' => array(
'un_mtch' => array('word_id', 'post_id', 'title_match'),
),
),
);
}
}
2 changes: 1 addition & 1 deletion phpBB/phpbb/db/tools.php
Expand Up @@ -2104,7 +2104,7 @@ function sql_create_unique_index($table_name, $index_name, $column)
$statements = array();

$table_prefix = substr(CONFIG_TABLE, 0, -6); // strlen(config)
if (strlen($table_name . $index_name) - strlen($table_prefix) > 24)
if (strlen($table_name . '_' . $index_name) - strlen($table_prefix) > 24)
{
$max_length = strlen($table_prefix) + 24;
trigger_error("Index name '{$table_name}_$index_name' on table '$table_name' is too long. The maximum is $max_length characters.", E_USER_ERROR);
Expand Down

0 comments on commit 9b36b52

Please sign in to comment.