Skip to main content
Topic: Add setPrefix() to DbTable (Read 1373 times) previous topic - next topic
0 Members and 1 Guest are viewing this topic.

Add setPrefix() to DbTable

Topic says it. Add DbTable::setPrefix() and a property $prefix to match. Then, remove all of the logic that looks for it in the methods. Then, you can remove all of that from the install too.

While I am here, there should also be a separate method for each of those functions to get the query. This way you can test what the query is. For instance DbTable::getCreateTableQuery(). The MySQL and Postgres classes would not implement db_create_table(), just getCreateTableQuery(). Of course, each of the current methods would just be { return $this->db->query('', get*Query(), ['security_override' = true]); }

Re: Add setPrefix() to DbTable

Reply #1

 emanuele keeps this unread.
Bugs creator.
Features destroyer.
Template killer.