fix: Only list remnants as disabled if option is enabled

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
nextcloud · May 16, 2024 · 5a154cb · 5a154cb
1 parent bd6989d
commit 5a154cb
Showing 1 changed file with 13 additions and 8 deletions.
diff --git a/apps/user_ldap/lib/User_Proxy.php b/apps/user_ldap/lib/User_Proxy.php
@@ -464,13 +464,18 @@ public function setUserEnabled(string $uid, bool $enabled, callable $queryDataba
 	}
 
 	public function getDisabledUserList(?int $limit = null, int $offset = 0): array {
-		return array_map(
-			fn (OfflineUser $user) => $user->getOCName(),
-			array_slice(
-				$this->deletedUsersIndex->getUsers(),
-				$offset,
-				$limit
-			)
-		);
+		if ((int)$this->getAccess(array_key_first($this->backends) ?? '')->connection->markRemnantsAsDisabled === 1) {
+			/* If first LDAP configuration considers remnants as disabled we consider all of them do */
+			return array_map(
+				fn (OfflineUser $user) => $user->getOCName(),
+				array_slice(
+					$this->deletedUsersIndex->getUsers(),
+					$offset,
+					$limit
+				)
+			);
+		} else {
+			return [];
+		}
 	}
 }