Remove default tablePrefix tbl_ from db\Connection

[schmunk42]

Why should we have such a default?
And if, because of some reasons yet unknown to me, I'd say it should be named app_.

[mikehaertl]

Agree. Following this logic, we could also prefix all columns with col_. Or even all variables with var_ or all functions with fn_ (you get the idea).

[Sammaye]

tbl_ is a very common, if not defacto, prefix for tables whereas the examples you mention are not.

It stems back from having to tell the difference easily in a console envo between tables and other elements in a list, i.e. views.

[schmunk42]

tbl_ is a very common, if not defacto, prefix for tables

Honestly, I've only seen them in a Yii 1 extension and it was - let's say - a bit uncomfortable.

A good summary is given here IMHO http:https://stackoverflow.com/questions/324163/should-we-use-prefixes-in-our-database-table-naming-conventions

I looked also for tbl_ table prefix on Google ... must be a Yii thing, 3 of the first 5 answers are Yii related 😄

[Sammaye]

Hmm maybe it is something that can be removed, I have just always been
taught, even in uni to use tbl_

On 28 March 2014 16:44, Tobias Munk [email protected] wrote:

tbl_ is a very common, if not defacto, prefix for tables

Honestly, I've only seen them in a Yii 1 extension and it was - let's say

• a bit uncomfortable.

A good summary is given here IMHO
http:https://stackoverflow.com/questions/324163/should-we-use-prefixes-in-our-database-table-naming-conventions

I looked also for tbl_ table prefix on Googlehttps://www.google.de/search?client=safari&rls=en&q=tbl_+table+prefix&ie=UTF-8&oe=UTF-8&gfe_rd=cr&ei=FaU1U_OFLcmK8QfxjIBg... must be a Yii thing, 3 of the first 5 answers are Yii related [image:
😄]

Reply to this email directly or view it on GitHubhttps://github.com//issues/2911#issuecomment-38940827
.

[bryglen]

i dont like it too having tbl prefix but there is a good reason too in my opinion..

when i see tbl_ in my database. I see it this was belong to Yii related stuff, but i prefer to have yii_ as a prefix instead of tbl_ like yii_migrations table instead of tbl_migrations so I know that this table is yii stuffs

[Sammaye]

That is one thing too, I had two apps that pointed to the same db (partially rewritten in Yii1 at the time) and tbl did tell me the difference between old and new.

[mikehaertl]

I still see no really convincing argument why this should be default. This is a kind of hungarian notation. But we don't use this anywhere else so what exactly makes table names so special that we would need it there? We should aim for simple and human readable names by default.

If someone really thinks, he prefers to have the tbl_ prefix, she can still configure it in their Connection component.

[Sammaye]

Yeah it does seem sane to do that

[schmunk42]

@samdark You could just move this setting to the apps.

[samdark]

You mean advanced app?

[schmunk42]

Yes, basic app also, if it has a db connection.

[qiangxue]

I'm fine removing the prefix. If we do so, the apps should not use prefix either for consistency. Can someone prepare a PR for this? All occurrence of tbl_ should be removed.

[Ragazzo]

so how then use prefix ? i know some projects including yupe that actually use prefix, also this is just a good reason to not get yourself in trouble with rdbms reserved words. I dont think we should remove this feature.

[mikehaertl]

@Ragazzo It won't get removed. It's just the default value that will be changed. You can always configure a table prefix in your db component:

'db' => [
    ...
    'tablePrefix' => 'tbl_',
]

[Ragazzo]

Ok )

[samdark]

I'll create a pull request if noone has started it.

[schmunk42]

I haven't so far.

[Sammaye]

I haven't