fix(profile): Set profile config cache directly after writing it to DB

In ProfileManager::getProfileParams we make a lot of indirect calls to ProfileManager::getProfileConfig. The first time we get the call we get into the catch and insert data, and the next time we miss the cache and do the select. There's a possibility of read-after-write issue here (the database r/o-mirror doesn't have yet the inserted config), which leads to conflicts when it tries to reinsert the config. To avoid that, let's put the config value in the configcache directly when it's written.

Signed-off-by: Thomas Citharel <[email protected]>

lib/private/Profile/ProfileManager.php

@@ -347,13 +347,15 @@ public function getProfileConfig(IUser $targetUser, ?IUser $visitingUser): array
 				$this->filterNotStoredProfileConfig($config->getConfigArray()),
 			));
 			$this->configMapper->update($config);
+			$this->configCache[$targetUser->getUID()] = $config;
 			$configArray = $config->getConfigArray();
 		} catch (DoesNotExistException $e) {
 			// Create a new default config if it does not exist
 			$config = new ProfileConfig();
 			$config->setUserId($targetUser->getUID());
 			$config->setConfigArray($defaultProfileConfig);
 			$this->configMapper->insert($config);
+			$this->configCache[$targetUser->getUID()] = $config;
 			$configArray = $config->getConfigArray();
 		}