Ignore deactivated users in collaborators user search plugin

Signed-off-by: Thomas Citharel <tcit@tcit.fr>

2 files changed, 8 insertions, 2 deletions

diff --git a/lib/private/Collaboration/Collaborators/UserPlugin.php b/lib/private/Collaboration/Collaborators/UserPlugin.php
index 62f76876031..971b7025564 100644
--- a/lib/private/Collaboration/Collaborators/UserPlugin.php
+++ b/lib/private/Collaboration/Collaborators/UserPlugin.php
@@ -79,7 +79,9 @@ class UserPlugin implements ISearchPlugin {
 			$usersTmp = $this->userManager->searchDisplayName($search, $limit, $offset);
 
 			foreach ($usersTmp as $user) {
-				$users[$user->getUID()] = $user->getDisplayName();
+				if ($user->isEnabled()) { // Don't keep deactivated users
+					$users[$user->getUID()] = $user->getDisplayName();
+				}
 			}
 		}
 
diff --git a/tests/lib/Collaboration/Collaborators/UserPluginTest.php b/tests/lib/Collaboration/Collaborators/UserPluginTest.php
index cfb97de8676..7513f9da5b6 100644
--- a/tests/lib/Collaboration/Collaborators/UserPluginTest.php
+++ b/tests/lib/Collaboration/Collaborators/UserPluginTest.php
@@ -89,7 +89,7 @@ class UserPluginTest extends TestCase {
 		);
 	}
 
-	public function getUserMock($uid, $displayName) {
+	public function getUserMock($uid, $displayName, $enabled = true) {
 		$user = $this->createMock(IUser::class);
 
 		$user->expects($this->any())
@@ -100,6 +100,10 @@ class UserPluginTest extends TestCase {
 			->method('getDisplayName')
 			->willReturn($displayName);
 
+		$user->expects($this->any())
+			->method('isEnabled')
+			->willReturn($enabled);
+
 		return $user;
 	}