fixed publishing of user properties to lookup-server

Fix of the bug in the lookup_server_connector module to publish public user information to the lookup-server. As described in issue [#25290](#25290)
nextcloud · Jan 24, 2021 · a39fa92 · a39fa92
1 parent 48854f2
commit a39fa92
Showing 1 changed file with 18 additions and 12 deletions.
diff --git a/apps/lookup_server_connector/lib/BackgroundJobs/RetryJob.php b/apps/lookup_server_connector/lib/BackgroundJobs/RetryJob.php
@@ -200,18 +200,24 @@ protected function getUserAccountData(IUser $user): array {
 
 		$data = ['federationId' => $user->getCloudId()];
 		if (!empty($publicData)) {
-			$data['name'] = $publicData[IAccountManager::PROPERTY_DISPLAYNAME]['value'] ?? '';
-			$data['email'] = $publicData[IAccountManager::PROPERTY_EMAIL]['value'] ?? '';
-			$data['address'] = $publicData[IAccountManager::PROPERTY_ADDRESS]['value'] ?? '';
-			$data['website'] = $publicData[IAccountManager::PROPERTY_WEBSITE]['value'] ?? '';
-			$data['twitter'] = $publicData[IAccountManager::PROPERTY_TWITTER]['value'] ?? '';
-			$data['phone'] = $publicData[IAccountManager::PROPERTY_PHONE]['value'] ?? '';
-			$data['twitter_signature'] = $publicData[IAccountManager::PROPERTY_TWITTER]['signature'] ?? '';
-			$data['website_signature'] = $publicData[IAccountManager::PROPERTY_WEBSITE]['signature'] ?? '';
-			$data['verificationStatus'] = [
-				IAccountManager::PROPERTY_WEBSITE => $publicData[IAccountManager::PROPERTY_WEBSITE]['verified'] ?? '',
-				IAccountManager::PROPERTY_TWITTER => $publicData[IAccountManager::PROPERTY_TWITTER]['verified'] ?? '',
-			];
+			$data['name'] = $publicData[IAccountManager::PROPERTY_DISPLAYNAME] ?? '';
+			$data['email'] = $publicData[IAccountManager::PROPERTY_EMAIL] ?? '';
+			$data['address'] = $publicData[IAccountManager::PROPERTY_ADDRESS] ?? '';
+			$data['website'] = $publicData[IAccountManager::PROPERTY_WEBSITE] ?? '';
+			$data['twitter'] = $publicData[IAccountManager::PROPERTY_TWITTER] ?? '';
+			$data['phone'] = $publicData[IAccountManager::PROPERTY_PHONE] ?? '';
+
+			/**
+			/* @todo Signature and verification status are currently removed,
+			/*       since those information are not provided by the Account
+			/*       or AccountProperties class.
+			*/
+			//$data['twitter_signature'] = $publicData[IAccountManager::PROPERTY_TWITTER]['signature'] ?? '';
+			//$data['website_signature'] = $publicData[IAccountManager::PROPERTY_WEBSITE]['signature'] ?? '';
+			//$data['verificationStatus'] = [
+			//	IAccountManager::PROPERTY_WEBSITE => $publicData[IAccountManager::PROPERTY_WEBSITE]['verified'] ?? '',
+			//	IAccountManager::PROPERTY_TWITTER => $publicData[IAccountManager::PROPERTY_TWITTER]['verified'] ?? '',
+			//];
 		}
 
 		return $data;