Fix query for total users

Fix query that counts total number of users to be displayed on the home page Bug: T348210
wikisource · Oct 10, 2023 · 32f4dec · 32f4dec
1 parent 6a39fbc
commit 32f4dec
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/i18n/en.json b/i18n/en.json
@@ -12,7 +12,7 @@
   "logged-in-as": "You are logged in as $1",
   "logout": "Log out",
 
-  "stats-summary": "$1 {{PLURAL:$2|person|people}} have taken part in $3 {{PLURAL:$4|contest|contests}}.",
+  "stats-summary": "$1 {{PLURAL:$2|person|people}} {{PLURAL:$2|has|have}} taken part in $3 {{PLURAL:$4|contest|contests}}.",
 
   "all-contests": "All contests",
   "your-contests": "Your contests",

diff --git a/src/Repository/UserRepository.php b/src/Repository/UserRepository.php
@@ -39,7 +39,7 @@ public function getIdFromUsername( string $username ): string {
 	}
 
 	public function count(): int {
-		$sql = 'SELECT COUNT(*) AS tot FROM users';
+		$sql = 'SELECT COUNT(DISTINCT user_id) AS tot FROM scores';
 		$stmt = $this->db->prepare( $sql );
 		$result = $stmt->executeQuery();
 		return (int)$result->fetchAssociative()['tot'];