Merge pull request #2117 from acelaya-forks/feature/superfluous-distinct

Remove unneeded DISTINCT from list short URLs query
This commit is contained in:
Alejandro Celaya 2024-05-06 18:54:01 +02:00 committed by GitHub
commit 05ebfccc63
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -42,7 +42,7 @@ class ShortUrlListRepository extends EntitySpecificationRepository implements Sh
$qb = $this->createListQueryBuilder($filtering);
$qb->select(
'DISTINCT s AS shortUrl',
's AS shortUrl',
'(' . $buildVisitsSubQuery('v', excludingBots: false) . ') AS ' . OrderableField::VISITS->value,
'(' . $buildVisitsSubQuery('v2', excludingBots: true) . ') AS ' . OrderableField::NON_BOT_VISITS->value,
// This is added only to have a consistent order by title between database engines