Only clear known users when we had at least one phonebook entry

Signed-off-by: Joas Schilling <coding@schilljs.com>
nextcloud · Mar 12, 2021 · 3efb2d2 · 3efb2d2
1 parent 6e2f7dd
commit 3efb2d2
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/apps/provisioning_api/lib/Controller/UsersController.php b/apps/provisioning_api/lib/Controller/UsersController.php
@@ -239,9 +239,6 @@ public function searchByPhoneNumbers(string $location, array $search): DataRespo
 		$user = $this->userSession->getUser();
 		$knownTo = $user->getUID();
 
-		// Cleanup all previous entries and only allow new matches
-		$this->knownUserService->deleteKnownTo($knownTo);
-
 		$normalizedNumberToKey = [];
 		foreach ($search as $key => $phoneNumbers) {
 			foreach ($phoneNumbers as $phone) {
@@ -262,6 +259,9 @@ public function searchByPhoneNumbers(string $location, array $search): DataRespo
 			return new DataResponse();
 		}
 
+		// Cleanup all previous entries and only allow new matches
+		$this->knownUserService->deleteKnownTo($knownTo);
+
 		$userMatches = $this->accountManager->searchUsers(IAccountManager::PROPERTY_PHONE, $phoneNumbers);
 
 		if (empty($userMatches)) {