# This is a combination of 3 commits.

# This is the 1st commit message:

fix: get actual number of active users

Signed-off-by: Hamza Mahjoubi <hamzamahjoubi221@gmail.com>

# The commit message #2 will be skipped:

# fixup! fix: get actual number of active users
#
# Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

# The commit message #3 will be skipped:

# fixup! fix: get actual number of active users
#
# Signed-off-by: Hamza Mahjoubi <hamzamahjoubi221@gmail.com>

Author: hamza221

lib/SessionStatistics.php

@@ -11,6 +11,7 @@
 namespace OCA\ServerInfo;
 
 use OCP\AppFramework\Utility\ITimeFactory;
+use OCP\DB\QueryBuilder\IQueryBuilder;
 use OCP\IDBConnection;
 
 /**
@@ -58,18 +59,21 @@ public function getSessionStatistics(): array {
 	 * @param int $offset seconds
 	 */
 	private function getNumberOfActiveUsers(int $offset): int {
-		$query = $this->connection->getQueryBuilder();
-		$query->select('uid')
-			->from('authtoken')
-			->where($query->expr()->gte(
-				'last_activity',
-				$query->createNamedParameter($this->timeFactory->getTime() - $offset)
-			))->groupBy('uid');
+		$queryBuilder = $this->connection->getQueryBuilder();
+		$queryBuilder->select($queryBuilder->func()->count('userid'))
+			->from('preferences')
+			->where($queryBuilder->expr()->eq('appid', $queryBuilder->createNamedParameter('login')))
+			->andWhere($queryBuilder->expr()->eq('configkey', $queryBuilder->createNamedParameter('lastLogin')))
+			->andwhere($queryBuilder->expr()->gte(
+				'configvalue',
+				$queryBuilder->createNamedParameter((string)($this->timeFactory->getTime() - $offset)),
+				IQueryBuilder::PARAM_STR,
+			));
 
-		$result = $query->executeQuery();
-		$activeUsers = $result->fetchAll();
+		$result = $queryBuilder->executeQuery();
+		$activeUsers = (int)$result->fetchOne();
 		$result->closeCursor();
 
-		return count($activeUsers);
+		return $activeUsers;
 	}
 }