If I involke pager_query() with 'SELECT DISTINCT u.uid FROM {users}', it will be replaced by 'SELECT count(*) FROM {users}' when counting users. Of course, it is just an example sql statement here. But the counting statement would be better be replaced to 'SELECT COUNT(DISTINCT(u.uid)) FROM {users}'.

Currently, the only way getting the wanted result is to passing extra $count parameters. But it would be better pager_query could do as following:
If 'SELECT DISTINCT' was found
$count_query = 'SELECT COUNT(DISTINCT(first_field))'...

Comments

chx’s picture

Version: 5.0-rc1 » 6.x-dev
Component: page.module » other

features go against 6.x but my intentions are to replace count_query with a subselect. stay tuned.

Pasqualle’s picture

Version: 6.x-dev » 7.x-dev

fixed with dbtng?

Berdir’s picture

Status: Active » Closed (duplicate)

duplicate of #423888: Use subqueries for ->countQuery(), at least for MySQL

(Sorry for pinging the participants, I'm trying to clean up a few old
issues)