Fix count query when using params.knex

[omeid]

Fixes #121

[daffl]

Great, thank you! Released as v3.0.3

[omeid]

Should note that until knex/knex#51 is addressed, this will break when using with .distinct() :(

[omeid]

A workaround is to use a sub-query when doing distinct, so SELECT * FROM (SELECT .....) and then the count would work fine.

[cjbland]

I don't mean to revive an old PR, but I was hoping @omeid could give a little more details on your workaround? If the table's id column does not exist as part of the distinct clause, the count will not work. This is the issue I'm running into right now because of this line:

feathers-knex/lib/index.js

Line 193 in e7f04f3

.count(`${this.table}.${this.id} as total`);

Here is a snippet from my hook:

const value = params.query['$distinct'];
delete params.query['$distinct'];
context.params.knex =context.service.createQuery(params).clearSelect().distinct(value);