Opened 13 years ago

Closed 13 years ago

# phpBB table prefix

Reported by: Owned by: Eugene Janusov John Hampton normal PhpBbAuthPlugin normal database, options 0.11

### Description

Currently the name of phpBB users table is hardcoded in all SQL queries with default prefix. I believe, it should be an option, that allows set this prefix.

### comment:1 follow-up:  2 Changed 13 years ago by John Hampton

Status: new → assigned

I'll admit to not being a phpBB expert, but does phpBB actually allow you to rename or specify which table is the users table? Do you have a specific use case for this?

### comment:2 in reply to:  1 ; follow-up:  3 Changed 13 years ago by Eugene Janusov

You can't change main part of the table names (e. g. "users", "posts"), but prefix "phpbb_" is customizable.

Usually prefix is used in order to names of tables from one project don't overlap with tables from others. But I, for example, have a separate database for forums, therefore during the installation I've set empty prefix, and tables are called just "users" and "posts" without "phpbb_".

### Changed 13 years ago by John Hampton

Patch to allow specifying of the table prefix for PhpBB tables

### comment:3 in reply to:  2 Changed 13 years ago by John Hampton

You can't change main part of the table names (e. g. "users", "posts"), but prefix "phpbb_" is customizable.

Usually prefix is used in order to names of tables from one project don't overlap with tables from others. But I, for example, have a separate database for forums, therefore during the installation I've set empty prefix, and tables are called just "users" and "posts" without "phpbb_".

OK, I just attached a patch that adds the ini option: phpbb_table_prefix to the [account-manager] section. In there you can specify whatever prefix you want, or leave it blank if you don't want. By default I have it set to phpbb_ which should preserve existing behavior.

Will you please test it out and let me know how it works?

### comment:4 follow-up:  5 Changed 13 years ago by Eugene Janusov

Works correctly, thanks. One note — default prefix must be fully in lower case.

### comment:5 in reply to:  4 Changed 13 years ago by John Hampton

Works correctly, thanks. One note — default prefix must be fully in lower case.

Does phpBB lowercase the prefix? Am I guaranteed to never have a mixed case prefix? If so, I can change the patch fairly easily to always force a lowercase prefix.

### comment:6 Changed 13 years ago by Eugene Janusov

No, I mean that default prefix is "phpbb_", not "phpBB_". User might use any case.

I propose change this line in the patch:

+    table_prefix = Option('account-manager', 'phpbb_table_prefix', 'phpBB_',


with:

+    table_prefix = Option('account-manager', 'phpbb_table_prefix', 'phpbb_',


### comment:7 Changed 13 years ago by John Hampton

Resolution: → fixed assigned → closed

(In [4041])

• Fixes #3355 - Indeed the default prefix should have been lowercase. This is fixed in the applied patch. Thanks for the report and the suggestion.

### Modify Ticket

Change Properties