`truncateDb` does not work with objection's `knexSnakeCaseMappers` about knex-db-manager HOT 5 OPEN

Yeah... I think that method that caches DB table names should disable mappers, because it does queries to databases internal tables. The same way that inside knex in migrations mappers are ignored when queries to knex internal tables are made.

As a side note, I think it is a bad idea to use those mappers. They will just increase probability that something in knex/objection/knde-db-manager etc. libraries break and doesn't really give any benefit for developer. Camel / pascal case in db column names are not so hard to manage when you get used to it and saves one from many potentially really hard to debug problems.

from knex-db-manager.

Thanks for the response, @elhigu!

should disable mappers

What do you mean by this? Something like this? https://github.com/knex/knex/blob/master/lib/migrate/Migrator.js#L38

As a side note, I think it is a bad idea to use those mappers.

I agree with this, but for my small project, it has become such a hassle to serialize data to camelCase before sending it out to the UI, which "feels" better to use.

I'll keep your opinion in the back of my head and refactor when I can, thanks again!

from knex-db-manager.

should disable mappers

Oops, I meant this: https://github.com/knex/knex/blob/37d9c3034743705782dce42355abd3df4f0ebf84/lib/util/make-knex.js#L177

from knex-db-manager.

Yep, something like that should do. I'm not sure if actually ew should add some method to knex for allowing to disable them, just for single query... Or maybe knex-db-manager could create another instance of knex using the same pool, but without those snake case wrapper configs. Though I'm not sure if that feature allows to override those configs.

from knex-db-manager.

Alrighty, thanks @elhigu !

I'll try making a PR for this over the weekend 👍

from knex-db-manager.