Merge pull request #29397 from nextcloud/fix/noid/profile-missing-def…

…-values
nextcloud · Oct 23, 2021 · 7e117da · 7e117da
2 parents 811670a + e26d6f0
commit 7e117da
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 15 deletions.
diff --git a/lib/private/Accounts/AccountManager.php b/lib/private/Accounts/AccountManager.php
@@ -473,15 +473,16 @@ protected function sendEmailVerificationEmail(IUser $user, string $email): bool
 	 * Make sure that all expected data are set
 	 */
 	protected function addMissingDefaultValues(array $userData, array $defaultUserData): array {
-		foreach ($defaultUserData as $i => $value) {
-			// If property doesn't exists, initialize it
-			if (!array_key_exists($i, $userData)) {
-				$userData[$i] = [];
+		foreach ($defaultUserData as $defaultDataItem) {
+			// If property does not exist, initialize it
+			$userDataIndex = array_search($defaultDataItem['name'], array_column($userData, 'name'));
+			if ($userDataIndex === false) {
+				$userData[] = $defaultDataItem;
+				continue;
 			}
 
 			// Merge and extend default missing values
-			$defaultValueIndex = array_search($value['name'], array_column($defaultUserData, 'name'));
-			$userData[$i] = array_merge($defaultUserData[$defaultValueIndex], $userData[$i]);
+			$userData[$userDataIndex] = array_merge($defaultDataItem, $userData[$userDataIndex]);
 		}
 
 		return $userData;

diff --git a/tests/lib/Accounts/AccountManagerTest.php b/tests/lib/Accounts/AccountManagerTest.php
@@ -516,8 +516,6 @@ public function testAddMissingDefaults() {
 				'value' => 'bob',
 				'verified' => IAccountManager::NOT_VERIFIED,
 			],
-			[],
-			[],
 			[
 				'name' => IAccountManager::PROPERTY_EMAIL,
 				'value' => '[email protected]',
@@ -533,23 +531,23 @@ public function testAddMissingDefaults() {
 			],
 
 			[
-				'name' => IAccountManager::PROPERTY_ADDRESS,
-				'value' => '',
-				'scope' => IAccountManager::SCOPE_LOCAL,
+				'name' => IAccountManager::PROPERTY_EMAIL,
+				'value' => '[email protected]',
+				'scope' => IAccountManager::SCOPE_FEDERATED,
 				'verified' => IAccountManager::NOT_VERIFIED,
 			],
 
 			[
-				'name' => IAccountManager::PROPERTY_WEBSITE,
+				'name' => IAccountManager::PROPERTY_ADDRESS,
 				'value' => '',
 				'scope' => IAccountManager::SCOPE_LOCAL,
 				'verified' => IAccountManager::NOT_VERIFIED,
 			],
 
 			[
-				'name' => IAccountManager::PROPERTY_EMAIL,
-				'value' => '[email protected]',
-				'scope' => IAccountManager::SCOPE_FEDERATED,
+				'name' => IAccountManager::PROPERTY_WEBSITE,
+				'value' => '',
+				'scope' => IAccountManager::SCOPE_LOCAL,
 				'verified' => IAccountManager::NOT_VERIFIED,
 			],