fix count with sort param

[petersg83]

fix #7138

[alexandrebodin]

LGTM

[robclouth]

I'm pretty sure normal search is broken too. Don't you need to change this line too?

strapi/packages/strapi-connector-bookshelf/lib/queries.js

Line 168 in 6d825fc

const filters = convertRestQueryParams(_.omit(params, '_q'));

[petersg83]

The problem was that the query was doing a sort on a count(*), which has not any sense. So we remove the sort parameter when doing a count(*).
When doing a search, the sort parameter has sense and, I think, should not be removed. So I think there is no need to change this line. But if you find a usecase where it doesn't work, I'll be happy to look into it !

[robclouth]

You're right. Thanks for the fix!