Skip to content

Commit 8074f7d

Browse files
committed
fix: get actual number of active users
Signed-off-by: Hamza Mahjoubi <hamzamahjoubi221@gmail.com>
1 parent 0ba8603 commit 8074f7d

1 file changed

Lines changed: 10 additions & 8 deletions

File tree

lib/SessionStatistics.php

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -58,15 +58,17 @@ public function getSessionStatistics(): array {
5858
* @param int $offset seconds
5959
*/
6060
private function getNumberOfActiveUsers(int $offset): int {
61-
$query = $this->connection->getQueryBuilder();
62-
$query->select('uid')
63-
->from('authtoken')
64-
->where($query->expr()->gte(
65-
'last_activity',
66-
$query->createNamedParameter($this->timeFactory->getTime() - $offset)
67-
))->groupBy('uid');
61+
$queryBuilder = $this->connection->getQueryBuilder();
62+
$queryBuilder->select('userid')
63+
->from('preferences')
64+
->where($queryBuilder->expr()->eq('appid', $queryBuilder->createNamedParameter('login')))
65+
->andWhere($queryBuilder->expr()->eq('configkey', $queryBuilder->createNamedParameter('lastLogin')))
66+
->andwhere($queryBuilder->expr()->gte(
67+
'configvalue',
68+
$queryBuilder->createNamedParameter($this->timeFactory->getTime() - $offset)
69+
))->groupBy('userid');
6870

69-
$result = $query->executeQuery();
71+
$result = $queryBuilder->executeQuery();
7072
$activeUsers = $result->fetchAll();
7173
$result->closeCursor();
7274

0 commit comments

Comments
 (0)