phpBB table prefix

[Eugene Janusov]

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.

[John Hampton]

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?

[Eugene Janusov]

Replying to pacopablo:

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_".

[John Hampton]

Patch to allow specifying of the table prefix for PhpBB tables

[John Hampton]

Replying to esycat:

Replying to pacopablo:

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?

[Eugene Janusov]

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

[John Hampton]

Replying to esycat:

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.

[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_',

[John Hampton]

(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.