diff --git a/phpBB/adm/style/acp_ext_delete_data.html b/phpBB/adm/style/acp_ext_delete_data.html
index 2d21506473..96ae816231 100644
--- a/phpBB/adm/style/acp_ext_delete_data.html
+++ b/phpBB/adm/style/acp_ext_delete_data.html
@@ -11,6 +11,7 @@
{% elseif S_PRE_STEP %}
diff --git a/phpBB/adm/style/acp_ext_enable.html b/phpBB/adm/style/acp_ext_enable.html
index 5319b76d3a..4b2d71cb4c 100644
--- a/phpBB/adm/style/acp_ext_enable.html
+++ b/phpBB/adm/style/acp_ext_enable.html
@@ -11,6 +11,7 @@
{% elseif S_PRE_STEP %}
diff --git a/phpBB/develop/create_schema_files.php b/phpBB/develop/create_schema_files.php
index e7c44a7f34..a8c3fd8647 100644
--- a/phpBB/develop/create_schema_files.php
+++ b/phpBB/develop/create_schema_files.php
@@ -58,6 +58,7 @@ $db_doctrine = $ref->newInstanceWithoutConstructor();
$factory = new \phpbb\db\tools\factory();
$db_tools = $factory->get($db_doctrine, true);
+$db_tools->set_table_prefix($table_prefix);
$tables_data = \Symfony\Component\Yaml\Yaml::parseFile($phpbb_root_path . '/config/default/container/tables.yml');
$tables = [];
diff --git a/phpBB/includes/acp/acp_extensions.php b/phpBB/includes/acp/acp_extensions.php
index e08c2e341b..d5adf62936 100644
--- a/phpBB/includes/acp/acp_extensions.php
+++ b/phpBB/includes/acp/acp_extensions.php
@@ -276,6 +276,11 @@ class acp_extensions
{
$this->template->assign_var('MIGRATOR_ERROR', $e->getLocalisedMessage($this->user));
}
+ catch (\Exception $e)
+ {
+ $this->template->assign_var('MIGRATOR_ERROR', $e->getMessage());
+ $this->template->assign_var('MIGRATOR_ERROR_STACK_TRACE', phpbb_filter_root_path($e->getTraceAsString()));
+ }
$this->tpl_name = 'acp_ext_enable';
@@ -363,6 +368,11 @@ class acp_extensions
{
$this->template->assign_var('MIGRATOR_ERROR', $e->getLocalisedMessage($this->user));
}
+ catch (\Exception $e)
+ {
+ $this->template->assign_var('MIGRATOR_ERROR', $e->getMessage());
+ $this->template->assign_var('MIGRATOR_ERROR_STACK_TRACE', phpbb_filter_root_path($e->getTraceAsString()));
+ }
$this->tpl_name = 'acp_ext_delete_data';
diff --git a/phpBB/includes/functions.php b/phpBB/includes/functions.php
index f725d5f996..72241b52f0 100644
--- a/phpBB/includes/functions.php
+++ b/phpBB/includes/functions.php
@@ -2716,7 +2716,7 @@ function get_backtrace()
// Only show function arguments for include etc.
// Other parameters may contain sensible information
$argument = '';
- if (!empty($trace['args'][0]) && in_array($trace['function'], array('include', 'require', 'include_once', 'require_once')))
+ if (!empty($trace['args'][0]) && in_array($trace['function'], array('include', 'require', 'include_once', 'require_once', 'try_apply')))
{
$argument = htmlspecialchars(phpbb_filter_root_path($trace['args'][0]), ENT_COMPAT);
}
diff --git a/phpBB/install/startup.php b/phpBB/install/startup.php
index b8ceb37342..0cff634795 100644
--- a/phpBB/install/startup.php
+++ b/phpBB/install/startup.php
@@ -107,7 +107,7 @@ function installer_msg_handler($errno, $msg_text, $errfile, $errline): bool
{
/** @var \phpbb\install\helper\iohandler\iohandler_interface $iohandler */
$iohandler = $phpbb_installer_container->get('installer.helper.iohandler');
- $iohandler->add_error_message($msg);
+ $iohandler->add_error_message($msg, get_backtrace());
$iohandler->send_response(true);
exit();
}
diff --git a/phpBB/phpbb/db/doctrine/table_helper.php b/phpBB/phpbb/db/doctrine/table_helper.php
index 935db35838..b163d75ccf 100644
--- a/phpBB/phpbb/db/doctrine/table_helper.php
+++ b/phpBB/phpbb/db/doctrine/table_helper.php
@@ -13,6 +13,7 @@
namespace phpbb\db\doctrine;
use InvalidArgumentException;
+use phpbb\db\tools\doctrine as doctrine_dbtools;
class table_helper
{
@@ -123,6 +124,54 @@ class table_helper
}
}
+ /**
+ * Maps table names to their short names for the purpose of prefixing tables' index names.
+ *
+ * @param array $table_names Table names with prefix to add to the map.
+ * @param string $table_prefix Tables prefix.
+ *
+ * @return array Pairs of table names and their short name representations.
+ * @psalm-return array{string, string}
+ */
+ public static function map_short_table_names(array $table_names = [], string $table_prefix = ''): array
+ {
+ $short_table_names_map = [];
+ foreach ($table_names as $table_name)
+ {
+ $short_table_names_map[$table_name] = self::generate_shortname(doctrine_dbtools::remove_prefix($table_name, $table_prefix));
+ }
+
+ return $short_table_names_map;
+ }
+
+ /**
+ * Generates short table names for the purpose of prefixing tables' index names.
+ *
+ * @param string $table_name Table name without prefix to generate its short name.
+ *
+ * @return string Short table name.
+ */
+ public static function generate_shortname(string $table_name = ''): string
+ {
+ // Only shorten actually long names
+ if (strlen($table_name) > 4)
+ {
+ // Remove vowels
+ $table_name = preg_replace('/([^aeiou_])([aeiou]+)/i', '$1', $table_name);
+
+ // Remove underscores
+ $table_name = str_replace('_', '', $table_name);
+
+ // Remove repeated consonants and their combinations (like 'ss', 'flfl' and similar)
+ $table_name = preg_replace('/(.+)\\1+/i', '$1', $table_name);
+
+ // Restrict short name length to 10 chars
+ $table_name = substr($table_name, 0, 10);
+ }
+
+ return $table_name;
+ }
+
/**
* Private constructor. Call methods of table_helper statically.
*/
diff --git a/phpBB/phpbb/db/migration/data/v400/rename_duplicated_index_names.php b/phpBB/phpbb/db/migration/data/v400/rename_duplicated_index_names.php
new file mode 100644
index 0000000000..805d68ff75
--- /dev/null
+++ b/phpBB/phpbb/db/migration/data/v400/rename_duplicated_index_names.php
@@ -0,0 +1,146 @@
+
+ * @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\v400;
+
+use phpbb\db\migration\migration;
+use phpbb\db\doctrine\table_helper;
+use phpbb\db\tools\doctrine as doctrine_dbtools;
+
+class rename_duplicated_index_names extends migration
+{
+ /**
+ * @var array
+ */
+ protected static $table_keys;
+
+ /**
+ * @var array
+ */
+ protected static $rename_index;
+
+ public static function depends_on()
+ {
+ return [
+ '\phpbb\db\migration\data\v400\storage_track_index',
+ ];
+ }
+
+ public function update_schema()
+ {
+ if (!isset(self::$rename_index))
+ {
+ if (!isset(self::$table_keys))
+ {
+ $this->get_tables_index_names();
+ }
+ $short_table_names = table_helper::map_short_table_names(array_keys(self::$table_keys), $this->table_prefix);
+
+ foreach (self::$table_keys as $table_name => $key_names)
+ {
+ $prefixless_table_name = doctrine_dbtools::remove_prefix($table_name, $this->table_prefix);
+ foreach ($key_names as $key_name)
+ {
+ // If 'old' key name is already new format, do not rename it
+ if (str_starts_with($key_name, $short_table_names[$table_name]))
+ {
+ continue;
+ }
+
+ // If 'old' key name is prefixed by its table name with and/or without table name common prefix
+ // (f.e. 'phpbb_log_log_time'), remove it to prefix with the relevant table's short name
+ $cleaned_key_name = $key_name;
+ foreach ([$table_name, $prefixless_table_name] as $prefix)
+ {
+ $cleaned_key_name = doctrine_dbtools::remove_prefix($cleaned_key_name, $prefix);
+ }
+ $key_name_new = $short_table_names[$table_name] . '_' . $cleaned_key_name;
+
+ self::$rename_index[$table_name][$key_name] = $key_name_new;
+ }
+ }
+ }
+
+ return [
+ 'rename_index' => self::$rename_index,
+ ];
+ }
+
+ public function revert_schema()
+ {
+ $schema = $this->update_schema();
+ array_walk($schema['rename_index'], function (&$index_data, $table_name) {
+ $index_data = array_flip($index_data);
+ });
+
+ return $schema;
+ }
+
+ protected function get_schema()
+ {
+ $self_classname = '\\' . str_replace('/', '\\', self::class);
+ $finder_factory = new \phpbb\finder\factory(null, false, $this->phpbb_root_path, $this->php_ext);
+ $finder = $finder_factory->get();
+ $migrator_classes = $finder->core_path('phpbb/db/migration/data/')->get_classes();
+ $self_class_index = array_search($self_classname, $migrator_classes);
+ unset($migrator_classes[$self_class_index]);
+
+ $schema_generator = new \phpbb\db\migration\schema_generator(
+ $migrator_classes,
+ $this->config,
+ $this->db,
+ $this->db_tools,
+ $this->phpbb_root_path,
+ $this->php_ext,
+ $this->table_prefix,
+ $this->tables
+ );
+
+ return $schema_generator->get_schema();
+ }
+
+ public function get_tables_index_names()
+ {
+ $schema_manager = $this->db_tools->get_connection()->createSchemaManager();
+ $table_names = $schema_manager->listTableNames();
+
+ if (!empty($table_names))
+ {
+ foreach ($table_names as $table_name)
+ {
+ $indices = $schema_manager->listTableIndexes($table_name);
+ $indices = array_keys(array_filter($indices,
+ function (\Doctrine\DBAL\Schema\Index $index)
+ {
+ return !$index->isPrimary();
+ })
+ );
+
+ if (!empty($indices))
+ {
+ self::$table_keys[$table_name] = $indices;
+ }
+ }
+ }
+ else
+ {
+ foreach ($this->get_schema() as $table_name => $table_data)
+ {
+ if (isset($table_data['KEYS']))
+ {
+ self::$table_keys[$table_name] = array_keys($table_data['KEYS']);
+ }
+ }
+ }
+ }
+}
diff --git a/phpBB/phpbb/db/migration/helper.php b/phpBB/phpbb/db/migration/helper.php
index bce2efff51..44cd23462a 100644
--- a/phpBB/phpbb/db/migration/helper.php
+++ b/phpBB/phpbb/db/migration/helper.php
@@ -42,6 +42,7 @@ class helper
'add_primary_keys' => 2, // perform_schema_changes only uses one level, but second is in the function
'add_unique_index' => 2,
'add_index' => 2,
+ 'rename_index' => 1,
);
foreach ($nested_level as $change_type => $data_depth)
diff --git a/phpBB/phpbb/db/migration/migration.php b/phpBB/phpbb/db/migration/migration.php
index 6bfc395e4d..e0ea38ff80 100644
--- a/phpBB/phpbb/db/migration/migration.php
+++ b/phpBB/phpbb/db/migration/migration.php
@@ -72,6 +72,8 @@ abstract class migration implements migration_interface
$this->php_ext = $php_ext;
$this->errors = array();
+
+ $this->db_tools->set_table_prefix($this->table_prefix);
}
/**
diff --git a/phpBB/phpbb/db/migration/schema_generator.php b/phpBB/phpbb/db/migration/schema_generator.php
index ce07c30d45..90cfddbe5a 100644
--- a/phpBB/phpbb/db/migration/schema_generator.php
+++ b/phpBB/phpbb/db/migration/schema_generator.php
@@ -187,6 +187,7 @@ class schema_generator
'add_index' => 'KEYS',
'add_unique_index' => 'KEYS',
'drop_keys' => 'KEYS',
+ 'rename_index' => 'KEYS',
];
$schema_changes = $migration->update_schema();
@@ -206,6 +207,7 @@ class schema_generator
{
case 'add':
case 'change':
+ case 'rename':
$action = function(&$value, $changes, $value_transform = null) {
self::set_all($value, $changes, $value_transform);
};
@@ -347,7 +349,7 @@ class schema_generator
*/
private static function get_value_transform(string $change_type, string $schema_type) : Closure|null
{
- if ($change_type !== 'add')
+ if (!in_array($change_type, ['add', 'rename']))
{
return null;
}
@@ -355,6 +357,23 @@ class schema_generator
switch ($schema_type)
{
case 'index':
+ if ($change_type == 'rename')
+ {
+ return function(&$value, $key, $change) {
+ if (isset($value[$key]))
+ {
+ $data_backup = $value[$key];
+ unset($value[$key]);
+ $value[$change] = $data_backup;
+ unset($data_backup);
+ }
+ else
+ {
+ return null;
+ }
+ };
+ }
+
return function(&$value, $key, $change) {
$value[$key] = ['INDEX', $change];
};
diff --git a/phpBB/phpbb/db/tools/doctrine.php b/phpBB/phpbb/db/tools/doctrine.php
index aaf44aed68..043252b4c5 100644
--- a/phpBB/phpbb/db/tools/doctrine.php
+++ b/phpBB/phpbb/db/tools/doctrine.php
@@ -49,6 +49,11 @@ class doctrine implements tools_interface
*/
private $return_statements;
+ /**
+ * @var string
+ */
+ private $table_prefix;
+
/**
* Database tools constructors.
*
@@ -61,6 +66,14 @@ class doctrine implements tools_interface
$this->connection = $connection;
}
+ /**
+ * {@inheritDoc}
+ */
+ public function get_connection(): Connection
+ {
+ return $this->connection;
+ }
+
/**
* @return AbstractSchemaManager
*
@@ -86,6 +99,14 @@ class doctrine implements tools_interface
return $this->get_schema_manager()->createSchema();
}
+ /**
+ * {@inheritDoc}
+ */
+ public function set_table_prefix($table_prefix): void
+ {
+ $this->table_prefix = $table_prefix;
+ }
+
/**
* {@inheritDoc}
*/
@@ -330,6 +351,19 @@ class doctrine implements tools_interface
);
}
+ /**
+ * {@inheritDoc}
+ */
+ public function sql_rename_index(string $table_name, string $index_name_old, string $index_name_new)
+ {
+ return $this->alter_schema(
+ function (Schema $schema) use ($table_name, $index_name_old, $index_name_new): void
+ {
+ $this->schema_rename_index($schema, $table_name, $index_name_old, $index_name_new);
+ }
+ );
+ }
+
/**
* {@inheritDoc}
*/
@@ -384,6 +418,23 @@ class doctrine implements tools_interface
}
}
+ /**
+ * {@inheritDoc}
+ */
+ public static function add_prefix(string $name, string $prefix): string
+ {
+ return str_ends_with($prefix, '_') ? $prefix . $name : $prefix . '_' . $name;
+ }
+
+ /**
+ * {@inheritDoc}
+ */
+ public static function remove_prefix(string $name, string $prefix = ''): string
+ {
+ $prefix = str_ends_with($prefix, '_') ? $prefix : $prefix . '_';
+ return $prefix && str_starts_with($name, $prefix) ? substr($name, strlen($prefix)) : $name;
+ }
+
/**
* Returns an array of indices for either unique and primary keys, or simple indices.
*
@@ -458,34 +509,26 @@ class doctrine implements tools_interface
*/
protected function alter_schema(callable $callback)
{
- try
+ $current_schema = $this->get_schema();
+ $new_schema = clone $current_schema;
+ call_user_func($callback, $new_schema);
+
+ $comparator = new comparator();
+ $schemaDiff = $comparator->compareSchemas($current_schema, $new_schema);
+ $queries = $schemaDiff->toSql($this->get_schema_manager()->getDatabasePlatform());
+
+ if ($this->return_statements)
{
- $current_schema = $this->get_schema();
- $new_schema = clone $current_schema;
- call_user_func($callback, $new_schema);
-
- $comparator = new comparator();
- $schemaDiff = $comparator->compareSchemas($current_schema, $new_schema);
- $queries = $schemaDiff->toSql($this->get_schema_manager()->getDatabasePlatform());
-
- if ($this->return_statements)
- {
- return $queries;
- }
-
- foreach ($queries as $query)
- {
- // executeQuery() must be used here because $query might return a result set, for instance REPAIR does
- $this->connection->executeQuery($query);
- }
-
- return true;
+ return $queries;
}
- catch (Exception $e)
+
+ foreach ($queries as $query)
{
- // @todo: check if it makes sense to properly handle the exception
- return [$e->getMessage()];
+ // executeQuery() must be used here because $query might return a result set, for instance REPAIR does
+ $this->connection->executeQuery($query);
}
+
+ return true;
}
/**
@@ -553,6 +596,11 @@ class doctrine implements tools_interface
'use_key' => true,
'per_table' => true,
],
+ 'rename_index' => [
+ 'method' => 'schema_rename_index',
+ 'use_key' => true,
+ 'per_table' => true,
+ ],
];
foreach ($mapping as $action => $params)
@@ -610,6 +658,7 @@ class doctrine implements tools_interface
}
$table = $schema->createTable($table_name);
+ $short_table_name = table_helper::generate_shortname(self::remove_prefix($table_name, $this->table_prefix));
$dbms_name = $this->get_schema_manager()->getDatabasePlatform()->getName();
foreach ($table_data['COLUMNS'] as $column_name => $column_data)
@@ -635,6 +684,7 @@ class doctrine implements tools_interface
foreach ($table_data['KEYS'] as $key_name => $key_data)
{
$columns = (is_array($key_data[1])) ? $key_data[1] : [$key_data[1]];
+ $key_name = !str_starts_with($key_name, $short_table_name) ? self::add_prefix($key_name, $short_table_name) : $key_name;
// Supports key columns defined with there length
$columns = array_map(function (string $column)
@@ -667,6 +717,8 @@ class doctrine implements tools_interface
}
/**
+ * Removes a table
+ *
* @param Schema $schema
* @param string $table_name
* @param bool $safe_check
@@ -684,6 +736,8 @@ class doctrine implements tools_interface
}
/**
+ * Adds column to a table
+ *
* @param Schema $schema
* @param string $table_name
* @param string $column_name
@@ -714,6 +768,8 @@ class doctrine implements tools_interface
}
/**
+ * Alters column properties
+ *
* @param Schema $schema
* @param string $table_name
* @param string $column_name
@@ -744,6 +800,8 @@ class doctrine implements tools_interface
}
/**
+ * Alters column properties or adds a column
+ *
* @param Schema $schema
* @param string $table_name
* @param string $column_name
@@ -766,6 +824,8 @@ class doctrine implements tools_interface
}
/**
+ * Removes a column in a table
+ *
* @param Schema $schema
* @param string $table_name
* @param string $column_name
@@ -818,6 +878,8 @@ class doctrine implements tools_interface
}
/**
+ * Creates non-unique index for a table
+ *
* @param Schema $schema
* @param string $table_name
* @param string $index_name
@@ -830,6 +892,8 @@ class doctrine implements tools_interface
{
$columns = (is_array($column)) ? $column : [$column];
$table = $schema->getTable($table_name);
+ $short_table_name = table_helper::generate_shortname(self::remove_prefix($table_name, $this->table_prefix));
+ $index_name = !str_starts_with($index_name, $short_table_name) ? self::add_prefix($index_name, $short_table_name) : $index_name;
if ($safe_check && $table->hasIndex($index_name))
{
@@ -840,6 +904,38 @@ class doctrine implements tools_interface
}
/**
+ * Renames table index
+ *
+ * @param Schema $schema
+ * @param string $table_name
+ * @param string $index_name_old
+ * @param string $index_name_new
+ * @param bool $safe_check
+ *
+ * @throws SchemaException
+ */
+ protected function schema_rename_index(Schema $schema, string $table_name, string $index_name_old, string $index_name_new, bool $safe_check = false): void
+ {
+ $table = $schema->getTable($table_name);
+ $short_table_name = table_helper::generate_shortname(self::remove_prefix($table_name, $this->table_prefix));
+
+ if (!$table->hasIndex($index_name_old))
+ {
+ $index_name_old = !str_starts_with($index_name_old, $short_table_name) ? self::add_prefix($index_name_old, $short_table_name) : self::remove_prefix($index_name_old, $short_table_name);
+ }
+ $index_name_new = !str_starts_with($index_name_new, $short_table_name) ? self::add_prefix($index_name_new, $short_table_name) : $index_name_new;
+
+ if ($safe_check && !$table->hasIndex($index_name_old))
+ {
+ return;
+ }
+
+ $table->renameIndex($index_name_old, $index_name_new);
+ }
+
+ /**
+ * Creates unique (non-primary) index for a table
+ *
* @param Schema $schema
* @param string $table_name
* @param string $index_name
@@ -852,6 +948,8 @@ class doctrine implements tools_interface
{
$columns = (is_array($column)) ? $column : [$column];
$table = $schema->getTable($table_name);
+ $short_table_name = table_helper::generate_shortname(self::remove_prefix($table_name, $this->table_prefix));
+ $index_name = !str_starts_with($index_name, $short_table_name) ? self::add_prefix($index_name, $short_table_name) : $index_name;
if ($safe_check && $table->hasIndex($index_name))
{
@@ -862,6 +960,8 @@ class doctrine implements tools_interface
}
/**
+ * Removes table index
+ *
* @param Schema $schema
* @param string $table_name
* @param string $index_name
@@ -872,6 +972,12 @@ class doctrine implements tools_interface
protected function schema_index_drop(Schema $schema, string $table_name, string $index_name, bool $safe_check = false): void
{
$table = $schema->getTable($table_name);
+ $short_table_name = table_helper::generate_shortname(self::remove_prefix($table_name, $this->table_prefix));
+
+ if (!$table->hasIndex($index_name))
+ {
+ $index_name = !str_starts_with($index_name, $short_table_name) ? self::add_prefix($index_name, $short_table_name) : self::remove_prefix($index_name, $short_table_name);
+ }
if ($safe_check && !$table->hasIndex($index_name))
{
@@ -882,6 +988,8 @@ class doctrine implements tools_interface
}
/**
+ * Creates primary key for a table
+ *
* @param $column
* @param Schema $schema
* @param string $table_name
diff --git a/phpBB/phpbb/db/tools/tools_interface.php b/phpBB/phpbb/db/tools/tools_interface.php
index 0b8cefd8dd..ceb1bb384d 100644
--- a/phpBB/phpbb/db/tools/tools_interface.php
+++ b/phpBB/phpbb/db/tools/tools_interface.php
@@ -165,6 +165,17 @@ interface tools_interface
*/
public function sql_create_index(string $table_name, string $index_name, $column);
+ /**
+ * Rename index
+ *
+ * @param string $table_name Table to modify
+ * @param string $index_name_old Name of the index to rename
+ * @param string $index_name_new New name of the index being renamed
+ *
+ * @return bool|string[] True if the statements have been executed
+ */
+ public function sql_rename_index(string $table_name, string $index_name_old, string $index_name_new);
+
/**
* Drop Index
*
@@ -215,4 +226,41 @@ interface tools_interface
* @return void
*/
public function sql_truncate_table(string $table_name): void;
+
+ /**
+ * Gets current Doctrine DBAL connection
+ *
+ * @return \Doctrine\DBAL\Connection
+ */
+ public function get_connection(): \Doctrine\DBAL\Connection;
+
+ /**
+ * Adds prefix to a string if needed
+ *
+ * @param string $name Table name with tables prefix
+ * @param string $prefix Prefix to add
+ *
+ * @return string Prefixed name
+ */
+ public static function add_prefix(string $name, string $prefix): string;
+
+ /**
+ * Removes prefix from string if exists. If prefix is empty,
+ * the first part of the string ending with underscore will be removed.
+ *
+ * @param string $name String to remove the prefix from
+ * @param string $prefix Prefix to remove
+ *
+ * @return string Prefixless string
+ */
+ public static function remove_prefix(string $name, string $prefix = ''): string;
+
+ /**
+ * Sets table prefix
+ *
+ * @param string $table_prefix String to test vs prefix
+ *
+ * @return void
+ */
+ public function set_table_prefix(string $table_prefix): void;
}
diff --git a/phpBB/phpbb/install/helper/database.php b/phpBB/phpbb/install/helper/database.php
index ddf4a503b7..91fd9c7aa8 100644
--- a/phpBB/phpbb/install/helper/database.php
+++ b/phpBB/phpbb/install/helper/database.php
@@ -393,6 +393,7 @@ class database
$doctrine_db = connection_factory::get_connection_from_params($dbms, $dbhost, $dbuser, $dbpass, $dbname, $dbport);
$db_tools_factory = new \phpbb\db\tools\factory();
$db_tools = $db_tools_factory->get($doctrine_db);
+ $db_tools->set_table_prefix($table_prefix);
$tables = $db_tools->sql_list_tables();
$tables = array_map('strtolower', $tables);
$table_intersect = array_intersect($tables, $table_ary);
diff --git a/phpBB/phpbb/install/installer.php b/phpBB/phpbb/install/installer.php
index 23ab5df919..3664b39f2d 100644
--- a/phpBB/phpbb/install/installer.php
+++ b/phpBB/phpbb/install/installer.php
@@ -280,7 +280,9 @@ class installer
}
catch (\Exception $e)
{
- $this->iohandler->add_error_message($e->getMessage());
+ $stack_trace = phpbb_filter_root_path(str_replace("\n", '
', $e->getTraceAsString()));
+ $message = $e->getMessage();
+ $this->iohandler->add_error_message($message, $stack_trace);
$this->iohandler->send_response(true);
$fail_cleanup = true;
}
diff --git a/phpBB/phpbb/install/module/install_database/task/add_tables.php b/phpBB/phpbb/install/module/install_database/task/add_tables.php
index 0d7e81b9e8..dfe276b6c9 100644
--- a/phpBB/phpbb/install/module/install_database/task/add_tables.php
+++ b/phpBB/phpbb/install/module/install_database/task/add_tables.php
@@ -98,6 +98,7 @@ class add_tables extends task_base
$this->schema_file_path = $phpbb_root_path . 'store/schema.json';
$this->table_prefix = $this->config->get('table_prefix');
$this->change_prefix = $this->config->get('change_table_prefix', true);
+ $this->db_tools->set_table_prefix($this->table_prefix);
parent::__construct(true);
}
diff --git a/phpBB/phpbb/install/module/install_database/task/create_schema_file.php b/phpBB/phpbb/install/module/install_database/task/create_schema_file.php
index e75bca65dc..20615c134a 100644
--- a/phpBB/phpbb/install/module/install_database/task/create_schema_file.php
+++ b/phpBB/phpbb/install/module/install_database/task/create_schema_file.php
@@ -145,6 +145,7 @@ class create_schema_file extends \phpbb\install\task_base
$migrator_classes = $finder->core_path('phpbb/db/migration/data/')->get_classes();
$factory = new \phpbb\db\tools\factory();
$db_tools = $factory->get($this->db_doctrine, true);
+ $db_tools->set_table_prefix($table_prefix);
$tables_data = \Symfony\Component\Yaml\Yaml::parseFile($this->phpbb_root_path . '/config/default/container/tables.yml');
$tables = [];
foreach ($tables_data['parameters'] as $parameter => $table)
diff --git a/tests/dbal/auto_increment_test.php b/tests/dbal/auto_increment_test.php
index e238859afc..74c34c934f 100644
--- a/tests/dbal/auto_increment_test.php
+++ b/tests/dbal/auto_increment_test.php
@@ -28,10 +28,12 @@ class phpbb_dbal_auto_increment_test extends phpbb_database_test_case
{
parent::setUp();
+ $table_prefix = 'prefix_';
$this->db = $this->new_dbal();
$this->db_doctrine = $this->new_doctrine_dbal();
$factory = new \phpbb\db\tools\factory();
$this->tools = $factory->get($this->db_doctrine);
+ $this->tools->set_table_prefix($table_prefix);
$this->table_data = array(
'COLUMNS' => array(
diff --git a/tests/dbal/db_tools_test.php b/tests/dbal/db_tools_test.php
index 73543ed97d..5bd0705daa 100644
--- a/tests/dbal/db_tools_test.php
+++ b/tests/dbal/db_tools_test.php
@@ -35,12 +35,15 @@ class phpbb_dbal_db_tools_test extends phpbb_database_test_case
protected function setUp(): void
{
+
parent::setUp();
+ $table_prefix = 'prefix_';
$this->db = $this->new_dbal();
$this->doctrine_db = $this->new_doctrine_dbal();
$factory = new \phpbb\db\tools\factory();
$this->tools = $factory->get($this->doctrine_db);
+ $this->tools->set_table_prefix($table_prefix);
$this->table_data = array(
'COLUMNS' => array(
@@ -232,23 +235,25 @@ class phpbb_dbal_db_tools_test extends phpbb_database_test_case
public function test_column_change_with_index()
{
+ $short_table_name = \phpbb\db\doctrine\table_helper::generate_shortname('table_name');
+
// Create column
$this->assertFalse($this->tools->sql_column_exists('prefix_table_name', 'c_bug_12012'));
$this->assertTrue($this->tools->sql_column_add('prefix_table_name', 'c_bug_12012', array('DECIMAL', 0)));
$this->assertTrue($this->tools->sql_column_exists('prefix_table_name', 'c_bug_12012'));
// Create index over the column
- $this->assertFalse($this->tools->sql_index_exists('prefix_table_name', 'i_bug_12012'));
+ $this->assertFalse($this->tools->sql_index_exists('prefix_table_name', $short_table_name . '_i_bug_12012'));
$this->assertTrue($this->tools->sql_create_index('prefix_table_name', 'i_bug_12012', array('c_bug_12012', 'c_bool')));
- $this->assertTrue($this->tools->sql_index_exists('prefix_table_name', 'i_bug_12012'));
+ $this->assertTrue($this->tools->sql_index_exists('prefix_table_name', $short_table_name . '_i_bug_12012'));
// Change type from int to string
$this->assertTrue($this->tools->sql_column_change('prefix_table_name', 'c_bug_12012', array('VCHAR:100', '')));
// Remove the index
- $this->assertTrue($this->tools->sql_index_exists('prefix_table_name', 'i_bug_12012'));
+ $this->assertTrue($this->tools->sql_index_exists('prefix_table_name', $short_table_name . '_i_bug_12012'));
$this->assertTrue($this->tools->sql_index_drop('prefix_table_name', 'i_bug_12012'));
- $this->assertFalse($this->tools->sql_index_exists('prefix_table_name', 'i_bug_12012'));
+ $this->assertFalse($this->tools->sql_index_exists('prefix_table_name', $short_table_name . '_i_bug_12012'));
// Remove the column
$this->assertTrue($this->tools->sql_column_exists('prefix_table_name', 'c_bug_12012'));
@@ -301,19 +306,21 @@ class phpbb_dbal_db_tools_test extends phpbb_database_test_case
public function test_column_remove_with_index()
{
+ $short_table_name = \phpbb\db\doctrine\table_helper::generate_shortname('table_name');
+
// Create column
$this->assertFalse($this->tools->sql_column_exists('prefix_table_name', 'c_bug_12012_2'));
$this->assertTrue($this->tools->sql_column_add('prefix_table_name', 'c_bug_12012_2', array('UINT', 4)));
$this->assertTrue($this->tools->sql_column_exists('prefix_table_name', 'c_bug_12012_2'));
// Create index over the column
- $this->assertFalse($this->tools->sql_index_exists('prefix_table_name', 'bug_12012_2'));
+ $this->assertFalse($this->tools->sql_index_exists('prefix_table_name', $short_table_name . '_bug_12012_2'));
$this->assertTrue($this->tools->sql_create_index('prefix_table_name', 'bug_12012_2', array('c_bug_12012_2', 'c_bool')));
- $this->assertTrue($this->tools->sql_index_exists('prefix_table_name', 'bug_12012_2'));
+ $this->assertTrue($this->tools->sql_index_exists('prefix_table_name', $short_table_name . '_bug_12012_2'));
- $this->assertFalse($this->tools->sql_index_exists('prefix_table_name', 'bug_12012_3'));
+ $this->assertFalse($this->tools->sql_index_exists('prefix_table_name', $short_table_name . '_bug_12012_3'));
$this->assertTrue($this->tools->sql_create_index('prefix_table_name', 'bug_12012_3', array('c_bug_12012_2')));
- $this->assertTrue($this->tools->sql_index_exists('prefix_table_name', 'bug_12012_3'));
+ $this->assertTrue($this->tools->sql_index_exists('prefix_table_name', $short_table_name . '_bug_12012_3'));
// Remove the column
$this->assertTrue($this->tools->sql_column_exists('prefix_table_name', 'c_bug_12012_2'));
@@ -443,24 +450,24 @@ class phpbb_dbal_db_tools_test extends phpbb_database_test_case
public function test_index_exists()
{
- $this->assertTrue($this->tools->sql_index_exists('prefix_table_name', 'i_simple'));
+ $this->assertTrue($this->tools->sql_index_exists('prefix_table_name', \phpbb\db\doctrine\table_helper::generate_shortname('table_name') . '_i_simple'));
}
public function test_unique_index_exists()
{
- $this->assertTrue($this->tools->sql_unique_index_exists('prefix_table_name', 'i_uniq'));
+ $this->assertTrue($this->tools->sql_unique_index_exists('prefix_table_name', \phpbb\db\doctrine\table_helper::generate_shortname('table_name') . '_i_uniq'));
}
public function test_create_index_against_index_exists()
{
$this->tools->sql_create_index('prefix_table_name', 'fookey', array('c_timestamp', 'c_decimal'));
- $this->assertTrue($this->tools->sql_index_exists('prefix_table_name', 'fookey'));
+ $this->assertTrue($this->tools->sql_index_exists('prefix_table_name', \phpbb\db\doctrine\table_helper::generate_shortname('table_name') . '_fookey'));
}
public function test_create_unique_index_against_unique_index_exists()
{
$this->tools->sql_create_unique_index('prefix_table_name', 'i_uniq_ts_id', array('c_timestamp', 'c_id'));
- $this->assertTrue($this->tools->sql_unique_index_exists('prefix_table_name', 'i_uniq_ts_id'));
+ $this->assertTrue($this->tools->sql_unique_index_exists('prefix_table_name', \phpbb\db\doctrine\table_helper::generate_shortname('table_name') . '_i_uniq_ts_id'));
}
public function test_create_int_default_null()
@@ -493,27 +500,28 @@ class phpbb_dbal_db_tools_test extends phpbb_database_test_case
$table_suffix = str_repeat('a', 25 - strlen($table_prefix));
$table_name = $table_prefix . $table_suffix;
+ $short_table_name = \phpbb\db\doctrine\table_helper::generate_shortname($table_suffix);
$this->tools->sql_create_table($table_name, $this->table_data);
// Index name and table suffix and table prefix have > maximum index length chars in total.
// Index name and table suffix have <= maximum index length chars in total.
$long_index_name = str_repeat('i', $max_index_length - strlen($table_suffix));
- $this->assertFalse($this->tools->sql_index_exists($table_name, $long_index_name));
+ $this->assertFalse($this->tools->sql_index_exists($table_name, $short_table_name . '_' . $long_index_name));
$this->assertTrue($this->tools->sql_create_index($table_name, $long_index_name, array('c_timestamp')));
- $this->assertTrue($this->tools->sql_index_exists($table_name, $long_index_name));
+ $this->assertTrue($this->tools->sql_index_exists($table_name, $short_table_name . '_' . $long_index_name));
// Index name and table suffix have > maximum index length chars in total.
$very_long_index_name = str_repeat('i', $max_index_length);
- $this->assertFalse($this->tools->sql_index_exists($table_name, $very_long_index_name));
+ $this->assertFalse($this->tools->sql_index_exists($table_name, $short_table_name . '_' . $very_long_index_name));
$this->assertTrue($this->tools->sql_create_index($table_name, $very_long_index_name, array('c_timestamp')));
- $this->assertTrue($this->tools->sql_index_exists($table_name, $very_long_index_name));
+ $this->assertTrue($this->tools->sql_index_exists($table_name, $short_table_name . '_' . $very_long_index_name));
$this->tools->sql_table_drop($table_name);
// Index name has > maximum index length chars - that should not be possible.
$too_long_index_name = str_repeat('i', $max_index_length + 1);
- $this->assertFalse($this->tools->sql_index_exists('prefix_table_name', $too_long_index_name));
+ $this->assertFalse($this->tools->sql_index_exists('prefix_table_name', $short_table_name . '_' . $too_long_index_name));
$this->setExpectedTriggerError(E_USER_ERROR); // TODO: Do we want to keep this limitation, if yes reimplement the user check
/* https://github.com/phpbb/phpbb/blob/aee5e373bca6cd20d44b99585d3b758276a2d7e6/phpBB/phpbb/db/tools/tools.php#L1488-L1517 */
$this->tools->sql_create_index('prefix_table_name', $too_long_index_name, array('c_timestamp'));
diff --git a/tests/dbal/migration/schema_index.php b/tests/dbal/migration/schema_index.php
new file mode 100644
index 0000000000..1ea8eb7c78
--- /dev/null
+++ b/tests/dbal/migration/schema_index.php
@@ -0,0 +1,65 @@
+
+* @license GNU General Public License, version 2 (GPL-2.0)
+*
+* For full copyright and license information, please see
+* the docs/CREDITS.txt file.
+*
+*/
+
+class phpbb_dbal_migration_schema_index extends \phpbb\db\migration\migration
+{
+ function update_schema()
+ {
+ return [
+ 'add_tables' => [
+ $this->table_prefix . 'foobar1' => [
+ 'COLUMNS' => [
+ 'user_id' => ['UINT', 0],
+ 'username' => ['VCHAR:50', 0],
+ ],
+ 'KEYS' => [
+ 'tstidx_user_id' => ['UNIQUE', 'user_id'],
+ 'tstidx_username' => ['INDEX', 'username'],
+ ],
+ ],
+ $this->table_prefix . 'foobar2' => [
+ 'COLUMNS' => [
+ 'ban_userid' => ['ULINT', 0],
+ 'ban_ip' => ['VCHAR:40', ''],
+ 'ban_reason' => ['VCHAR:100', ''],
+ ],
+ 'KEYS' => [
+ 'tstidx_ban_userid' => ['UNIQUE', 'ban_userid'],
+ 'tstidxban_data' => ['INDEX', ['ban_ip', 'ban_reason']],
+ ],
+ ],
+ ],
+
+ 'rename_index' => [
+ $this->table_prefix . 'foobar1' => [
+ 'tstidx_user_id' => 'fbr1_user_id',
+ 'tstidx_username' => 'fbr1_username',
+ ],
+ $this->table_prefix . 'foobar2' => [
+ 'tstidx_ban_userid' => 'fbr2_ban_userid',
+ 'tstidxban_data' => 'fbr2_ban_data',
+ ],
+ ],
+ ];
+ }
+
+ function revert_schema()
+ {
+ return [
+ 'drop_tables' => [
+ $this->table_prefix . 'foobar1',
+ $this->table_prefix . 'foobar2',
+ ],
+ ];
+ }
+}
diff --git a/tests/dbal/migrator_test.php b/tests/dbal/migrator_test.php
index 5d16fa63ee..f4b80fc5e9 100644
--- a/tests/dbal/migrator_test.php
+++ b/tests/dbal/migrator_test.php
@@ -24,6 +24,7 @@ require_once __DIR__ . '/migration/revert_table_with_dependency.php';
require_once __DIR__ . '/migration/fail.php';
require_once __DIR__ . '/migration/installed.php';
require_once __DIR__ . '/migration/schema.php';
+require_once __DIR__ . '/migration/schema_index.php';
class phpbb_dbal_migrator_test extends phpbb_database_test_case
{
@@ -52,12 +53,15 @@ class phpbb_dbal_migrator_test extends phpbb_database_test_case
protected function setUp(): void
{
+ global $table_prefix;
+
parent::setUp();
$this->db = $this->new_dbal();
$this->doctrine_db = $this->new_doctrine_dbal();
$factory = new \phpbb\db\tools\factory();
$this->db_tools = $factory->get($this->doctrine_db);
+ $this->db_tools->set_table_prefix($table_prefix);
$this->config = new \phpbb\config\db($this->db, new phpbb_mock_cache, 'phpbb_config');
@@ -416,4 +420,86 @@ class phpbb_dbal_migrator_test extends phpbb_database_test_case
$this->assertFalse($this->db_tools->sql_column_exists('phpbb_config', 'test_column1'));
$this->assertFalse($this->db_tools->sql_table_exists('phpbb_foobar'));
}
+
+ public function test_rename_index()
+ {
+ $this->migrator->set_migrations(array('phpbb_dbal_migration_schema_index'));
+
+ while (!$this->migrator->finished())
+ {
+ $this->migrator->update();
+ }
+
+ $this->assertTrue($this->db_tools->sql_unique_index_exists('phpbb_foobar1', 'fbr1_user_id'));
+ $this->assertTrue($this->db_tools->sql_index_exists('phpbb_foobar1', 'fbr1_username'));
+ $this->assertTrue($this->db_tools->sql_unique_index_exists('phpbb_foobar2', 'fbr2_ban_userid'));
+ $this->assertTrue($this->db_tools->sql_index_exists('phpbb_foobar2', 'fbr2_ban_data'));
+
+ while ($this->migrator->migration_state('phpbb_dbal_migration_schema_index'))
+ {
+ $this->migrator->revert('phpbb_dbal_migration_schema_index');
+ }
+
+ $this->assertFalse($this->db_tools->sql_table_exists('phpbb_foobar1'));
+ $this->assertFalse($this->db_tools->sql_table_exists('phpbb_foobar2'));
+ }
+
+ public function test_schema_generator(): array
+ {
+ global $phpbb_root_path, $phpEx;
+
+ $finder_factory = new \phpbb\finder\factory(null, false, $phpbb_root_path, $phpEx);
+ $finder = $finder_factory->get();
+ $migrator_classes = $finder->core_path('phpbb/db/migration/data/')->get_classes();
+
+ $schema_generator = new \phpbb\db\migration\schema_generator(
+ $migrator_classes,
+ $this->config,
+ $this->db,
+ $this->db_tools,
+ $phpbb_root_path,
+ $phpEx,
+ 'phpbb_',
+ self::get_core_tables()
+ );
+ $db_table_schema = $schema_generator->get_schema();
+
+ $this->assertNotEmpty($db_table_schema);
+
+ return $db_table_schema;
+ }
+
+ /**
+ * @depends test_schema_generator
+ */
+ public function test_table_indexes(array $db_table_schema)
+ {
+ $table_keys = [];
+ foreach ($db_table_schema as $table_name => $table_data)
+ {
+ if (isset($table_data['KEYS']))
+ {
+ foreach ($table_data['KEYS'] as $key_name => $key_data)
+ {
+ $table_keys[$table_name][] = $key_name;
+ }
+ }
+ }
+
+ $this->assertNotEmpty($table_keys);
+
+ $table_names = array_merge(array_keys($db_table_schema), ['phpbb_custom_table']);
+ $short_table_names = \phpbb\db\doctrine\table_helper::map_short_table_names($table_names, 'phpbb_');
+ $this->assertEquals('phpbb_custom_table', array_search(\phpbb\db\doctrine\table_helper::generate_shortname('custom_table'), $short_table_names));
+ $this->assertEquals($short_table_names['phpbb_custom_table'], \phpbb\db\doctrine\table_helper::generate_shortname('custom_table'));
+
+ foreach ($table_keys as $table_name => $key_names)
+ {
+ $index_prefix = $short_table_names[$table_name] . '_';
+ foreach ($key_names as $key_name)
+ {
+ $this->assertEquals(0, strpos($key_name, $index_prefix), "$key_name does not contain $index_prefix");
+ }
+ }
+ }
}
diff --git a/tests/extension/manager_test.php b/tests/extension/manager_test.php
index 73ac5a51de..e659d37a51 100644
--- a/tests/extension/manager_test.php
+++ b/tests/extension/manager_test.php
@@ -154,6 +154,7 @@ class phpbb_extension_manager_test extends phpbb_database_test_case
{
$phpbb_root_path = __DIR__ . './../../phpBB/';
$php_ext = 'php';
+ $table_prefix = 'phpbb_';
$config = new \phpbb\config\config(array('version' => PHPBB_VERSION));
$db = $this->new_dbal();
@@ -162,7 +163,7 @@ class phpbb_extension_manager_test extends phpbb_database_test_case
$factory = new \phpbb\db\tools\factory();
$finder_factory = new \phpbb\finder\factory(null, false, $phpbb_root_path, $php_ext);
$db_tools = $factory->get($db_doctrine);
- $table_prefix = 'phpbb_';
+ $db_tools->set_table_prefix($table_prefix);
$container = new phpbb_mock_container_builder();
diff --git a/tests/extension/metadata_manager_test.php b/tests/extension/metadata_manager_test.php
index 095efd3a69..b32b67c124 100644
--- a/tests/extension/metadata_manager_test.php
+++ b/tests/extension/metadata_manager_test.php
@@ -37,6 +37,7 @@ class phpbb_extension_metadata_manager_test extends phpbb_database_test_case
{
parent::setUp();
+ $this->table_prefix = 'phpbb_';
$this->config = new \phpbb\config\config(array(
'version' => '3.1.0',
));
@@ -45,13 +46,13 @@ class phpbb_extension_metadata_manager_test extends phpbb_database_test_case
$phpbb_dispatcher = new phpbb_mock_event_dispatcher();
$factory = new \phpbb\db\tools\factory();
$this->db_tools = $factory->get($this->db_doctrine);
+ $this->db_tools->set_table_prefix($this->table_prefix);
$finder_factory = $this->createMock('\phpbb\finder\factory');
$this->phpbb_root_path = __DIR__ . '/';
$this->phpEx = 'php';
$this->cache = new \phpbb\cache\service(new phpbb_mock_cache(), $this->config, $this->db, $phpbb_dispatcher, $this->phpbb_root_path, $this->phpEx);
- $this->table_prefix = 'phpbb_';
$container = new phpbb_mock_container_builder();
$cache_path = $this->phpbb_root_path . 'cache/twig';
diff --git a/tests/migrations/migrations_check_config_added_test.php b/tests/migrations/migrations_check_config_added_test.php
index 1472f9e370..5cde980cde 100644
--- a/tests/migrations/migrations_check_config_added_test.php
+++ b/tests/migrations/migrations_check_config_added_test.php
@@ -55,6 +55,10 @@ class migrations_check_config_added_test extends phpbb_test_case
{
global $phpbb_root_path, $phpEx;
+ $this->table_prefix = 'phpbb_';
+ $this->phpbb_root_path = $phpbb_root_path;
+ $this->php_ext = $phpEx;
+
$this->config = new \phpbb\config\config([
'search_type' => '\phpbb\search\fulltext_mysql',
]);
@@ -63,9 +67,7 @@ class migrations_check_config_added_test extends phpbb_test_case
$this->db_doctrine = $this->createMock(\Doctrine\DBAL\Connection::class);
$factory = new \phpbb\db\tools\factory();
$this->db_tools = $factory->get($this->db_doctrine);
- $this->table_prefix = 'phpbb_';
- $this->phpbb_root_path = $phpbb_root_path;
- $this->php_ext = $phpEx;
+ $this->db_tools->set_table_prefix($this->table_prefix);
$tools = [
new \phpbb\db\migration\tool\config($this->config),
diff --git a/tests/migrator/get_callable_from_step_test.php b/tests/migrator/get_callable_from_step_test.php
index 1200568046..f8a92c742a 100644
--- a/tests/migrator/get_callable_from_step_test.php
+++ b/tests/migrator/get_callable_from_step_test.php
@@ -23,6 +23,8 @@ class get_callable_from_step_test extends phpbb_database_test_case
$db = $this->new_dbal();
$db_doctrine = $this->new_doctrine_dbal();
$factory = new \phpbb\db\tools\factory();
+ $db_tools = $factory->get($db_doctrine);
+ $db_tools->set_table_prefix($table_prefix);
$user = $this->getMockBuilder('\phpbb\user')->disableOriginalConstructor()->getMock();
$user->ip = '127.0.0.1';
$module_manager = new \phpbb\module\module_manager(
@@ -38,7 +40,7 @@ class get_callable_from_step_test extends phpbb_database_test_case
new phpbb_mock_container_builder(),
new \phpbb\config\config(array()),
$db,
- $factory->get($db_doctrine),
+ $db_tools,
'phpbb_migrations',
$phpbb_root_path,
$php_ext,
diff --git a/tests/migrator/schema_generator_test.php b/tests/migrator/schema_generator_test.php
index 04c73f925e..943029304f 100644
--- a/tests/migrator/schema_generator_test.php
+++ b/tests/migrator/schema_generator_test.php
@@ -30,14 +30,16 @@ class schema_generator_test extends phpbb_test_case
parent::setUp();
+ $this->table_prefix = 'phpbb_';
+ $this->phpbb_root_path = $phpbb_root_path;
+ $this->php_ext = $phpEx;
+
$this->config = new \phpbb\config\config(array());
$this->db = new \phpbb\db\driver\sqlite3();
$this->doctrine_db = \phpbb\db\doctrine\connection_factory::get_connection(new phpbb_mock_config_php_file());
$factory = new \phpbb\db\tools\factory();
$this->db_tools = $factory->get($this->doctrine_db);
- $this->table_prefix = 'phpbb_';
- $this->phpbb_root_path = $phpbb_root_path;
- $this->php_ext = $phpEx;
+ $this->db_tools->set_table_prefix($this->table_prefix);
}
protected function get_schema_generator(array $class_names)
diff --git a/tests/test_framework/phpbb_database_test_case.php b/tests/test_framework/phpbb_database_test_case.php
index 09de77bc35..76e77add44 100644
--- a/tests/test_framework/phpbb_database_test_case.php
+++ b/tests/test_framework/phpbb_database_test_case.php
@@ -88,6 +88,7 @@ abstract class phpbb_database_test_case extends TestCase
$doctrine = \phpbb\db\doctrine\connection_factory::get_connection(new phpbb_mock_config_php_file());
$factory = new \phpbb\db\tools\factory();
$db_tools = $factory->get($doctrine, true);
+ $db_tools->set_table_prefix($table_prefix);
$schema_generator = new \phpbb\db\migration\schema_generator($classes, new \phpbb\config\config(array()), $db, $db_tools, $phpbb_root_path, $phpEx, $table_prefix, self::get_core_tables());
file_put_contents(self::$schema_file, json_encode($schema_generator->get_schema()));
diff --git a/tests/test_framework/phpbb_database_test_connection_manager.php b/tests/test_framework/phpbb_database_test_connection_manager.php
index db73248f6c..8759a1f8c0 100644
--- a/tests/test_framework/phpbb_database_test_connection_manager.php
+++ b/tests/test_framework/phpbb_database_test_connection_manager.php
@@ -327,6 +327,8 @@ class phpbb_database_test_connection_manager
*/
protected function load_schema_from_file($directory, \phpbb\db\driver\driver_interface $db, \Doctrine\DBAL\Connection $doctrine)
{
+ global $table_prefix;
+
$schema = $this->dbms['SCHEMA'];
if ($this->config['dbms'] == 'phpbb\db\driver\mysql')
@@ -363,7 +365,7 @@ class phpbb_database_test_connection_manager
}
else
{
- global $phpbb_root_path, $phpEx, $table_prefix;
+ global $phpbb_root_path, $phpEx;
$finder = new \phpbb\finder\finder(null, false, $phpbb_root_path, $phpEx);
$classes = $finder->core_path('phpbb/db/migration/data/')
@@ -373,6 +375,7 @@ class phpbb_database_test_connection_manager
$doctrine = \phpbb\db\doctrine\connection_factory::get_connection(new phpbb_mock_config_php_file());
$factory = new \phpbb\db\tools\factory();
$db_tools = $factory->get($doctrine, true);
+ $db_tools->set_table_prefix($table_prefix);
$tables = phpbb_database_test_case::get_core_tables();
$schema_generator = new \phpbb\db\migration\schema_generator($classes, new \phpbb\config\config(array()), $db, $db_tools, $phpbb_root_path, $phpEx, $table_prefix, $tables);
@@ -381,6 +384,7 @@ class phpbb_database_test_connection_manager
$factory = new \phpbb\db\tools\factory();
$db_tools = $factory->get($doctrine);
+ $db_tools->set_table_prefix($table_prefix);
foreach ($db_table_schema as $table_name => $table_data)
{
$db_tools->sql_create_table(
diff --git a/tests/test_framework/phpbb_functional_test_case.php b/tests/test_framework/phpbb_functional_test_case.php
index 2ade7533cc..ebe7c15f81 100644
--- a/tests/test_framework/phpbb_functional_test_case.php
+++ b/tests/test_framework/phpbb_functional_test_case.php
@@ -286,6 +286,7 @@ class phpbb_functional_test_case extends phpbb_test_case
$factory = new \phpbb\db\tools\factory();
$finder_factory = new \phpbb\finder\factory(null, false, $phpbb_root_path, $phpEx);
$db_tools = $factory->get($db_doctrine);
+ $db_tools->set_table_prefix(self::$config['table_prefix']);
$container = new phpbb_mock_container_builder();
$migrator = new \phpbb\db\migrator(