From bb8ab8ddc427e514a0d52b243bd7bdfbc091e4e3 Mon Sep 17 00:00:00 2001
From: Joas Schilling <coding@schilljs.com>
Date: Mon, 22 Aug 2022 14:14:47 +0200
Subject: [PATCH 1/2] Fix plural usage in LDAP wizard

Signed-off-by: Joas Schilling <coding@schilljs.com>
---
 apps/user_ldap/lib/Wizard.php | 33 ++++++++++++++++++++-------------
 1 file changed, 20 insertions(+), 13 deletions(-)

diff --git a/apps/user_ldap/lib/Wizard.php b/apps/user_ldap/lib/Wizard.php
index ae9546be08b90..da2b92d98738f 100644
--- a/apps/user_ldap/lib/Wizard.php
+++ b/apps/user_ldap/lib/Wizard.php
@@ -138,7 +138,7 @@ public function countGroups() {
 		$filter = $this->configuration->ldapGroupFilter;
 
 		if (empty($filter)) {
-			$output = self::$l->n('%s group found', '%s groups found', 0, [0]);
+			$output = self::$l->n('%n group found', '%n groups found', 0);
 			$this->result->addChange('ldap_group_count', $output);
 			return $this->result;
 		}
@@ -152,12 +152,16 @@ public function countGroups() {
 			}
 			return false;
 		}
-		$output = self::$l->n(
-			'%s group found',
-			'%s groups found',
-			$groupsTotal,
-			[$this->formatCountResult($groupsTotal)]
-		);
+
+		if ($groupsTotal > 1000) {
+			$output = self::$l->t('> 1000 groups found');
+		} else {
+			$output = self::$l->n(
+				'%n group found',
+				'%n groups found',
+				$groupsTotal
+			);
+		}
 		$this->result->addChange('ldap_group_count', $output);
 		return $this->result;
 	}
@@ -170,12 +174,15 @@ public function countUsers() {
 		$filter = $this->access->getFilterForUserCount();
 
 		$usersTotal = $this->countEntries($filter, 'users');
-		$output = self::$l->n(
-			'%s user found',
-			'%s users found',
-			$usersTotal,
-			[$this->formatCountResult($usersTotal)]
-		);
+		if ($usersTotal > 1000) {
+			$output = self::$l->t('> 1000 users found');
+		} else {
+			$output = self::$l->n(
+				'%n user found',
+				'%n users found',
+				$usersTotal
+			);
+		}
 		$this->result->addChange('ldap_user_count', $output);
 		return $this->result;
 	}

From d40ae6fd57bf2d6645ce3a73078c7221f3c03107 Mon Sep 17 00:00:00 2001
From: Joas Schilling <coding@schilljs.com>
Date: Mon, 22 Aug 2022 14:15:32 +0200
Subject: [PATCH 2/2] Remove unused method

Signed-off-by: Joas Schilling <coding@schilljs.com>
---
 apps/user_ldap/lib/Wizard.php | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/apps/user_ldap/lib/Wizard.php b/apps/user_ldap/lib/Wizard.php
index da2b92d98738f..e85e65a7d70f9 100644
--- a/apps/user_ldap/lib/Wizard.php
+++ b/apps/user_ldap/lib/Wizard.php
@@ -120,20 +120,6 @@ public function countEntries(string $filter, string $type): int {
 		return (int)$result;
 	}
 
-	/**
-	 * formats the return value of a count operation to the string to be
-	 * inserted.
-	 *
-	 * @param int $count
-	 * @return string
-	 */
-	private function formatCountResult(int $count): string {
-		if ($count > 1000) {
-			return '> 1000';
-		}
-		return (string)$count;
-	}
-
 	public function countGroups() {
 		$filter = $this->configuration->ldapGroupFilter;