Merge pull request #38305 from nextcloud/backport/38143/stable25

[stable25] Fix generated avatars cache
This commit is contained in:
Simon L 2023-05-16 21:58:12 +02:00 committed by GitHub
commit 88f588f502
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 10 deletions

View file

@ -161,13 +161,8 @@ class AvatarManager implements IAvatarManager {
public function clearCachedAvatars() {
$users = $this->config->getUsersForUserValue('avatar', 'generated', 'true');
foreach ($users as $userId) {
try {
$folder = $this->appData->getFolder($userId);
$folder->delete();
} catch (NotFoundException $e) {
$this->logger->debug("No cache for the user $userId. Ignoring...");
}
$this->config->setUserValue($userId, 'avatar', 'generated', 'false');
// This also bumps the avatar version leading to cache invalidation in browsers
$this->getAvatar($userId)->remove();
}
}

View file

@ -42,7 +42,7 @@ class ClearGeneratedAvatarCache implements IRepairStep {
}
public function getName(): string {
return 'Clear every generated avatar on major updates';
return 'Clear every generated avatar';
}
/**
@ -51,8 +51,9 @@ class ClearGeneratedAvatarCache implements IRepairStep {
private function shouldRun(): bool {
$versionFromBeforeUpdate = $this->config->getSystemValue('version', '0.0.0.0');
// was added to 25.0.0.10
return version_compare($versionFromBeforeUpdate, '25.0.0.10', '<=');
// This job only runs if the server was on a version lower than or equal to 25.0.7 before the upgrade.
// To clear the avatar cache again, bump the version to the currently released version (and change the operator to <= if it's not the master branch) and wait for the next release.
return version_compare($versionFromBeforeUpdate, '25.0.7', '<');
}
public function run(IOutput $output): void {