diff options
| author | Julius Härtl <jus@bitgrid.net> | 2023-08-10 09:26:12 +0200 |
|---|---|---|
| committer | John Molakvoæ <skjnldsv@users.noreply.github.com> | 2023-08-16 15:44:06 +0200 |
| commit | 6c05eb95eeced16c93bb31769747574698084126 (patch) | |
| tree | 4b130aab4889931d89813d2a6cfdfcf0a0393b11 | |
| parent | f95ed098c56359151432e5640c8b43ce4958cd93 (diff) | |
| download | nextcloud-server-6c05eb95eeced16c93bb31769747574698084126.tar.gz nextcloud-server-6c05eb95eeced16c93bb31769747574698084126.zip | |
fix: Also cleanup version metadata if expiring
Signed-off-by: Julius Härtl <jus@bitgrid.net>
| -rw-r--r-- | apps/files_versions/lib/Storage.php | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/apps/files_versions/lib/Storage.php b/apps/files_versions/lib/Storage.php index 676d94d2fb7..426d5931ba9 100644 --- a/apps/files_versions/lib/Storage.php +++ b/apps/files_versions/lib/Storage.php @@ -913,6 +913,21 @@ class Storage { } foreach ($toDelete as $key => $path) { + // Make sure to cleanup version table relations as expire does not pass deleteVersion + try { + /** @var VersionsMapper $versionsMapper */ + $versionsMapper = \OC::$server->get(VersionsMapper::class); + $file = \OC::$server->get(IRootFolder::class)->getUserFolder($uid)->get($filename); + $pathparts = pathinfo($path); + $timestamp = (int)substr($pathparts['extension'] ?? '', 1); + $versionEntity = $versionsMapper->findVersionForFileId($file->getId(), $timestamp); + if ($versionEntity->getLabel() !== '') { + continue; + } + $versionsMapper->delete($versionEntity); + } catch (DoesNotExistException $e) { + } + \OC_Hook::emit('\OCP\Versions', 'preDelete', ['path' => $path, 'trigger' => self::DELETE_TRIGGER_QUOTA_EXCEEDED]); self::deleteVersion($versionsFileview, $path); \OC_Hook::emit('\OCP\Versions', 'delete', ['path' => $path, 'trigger' => self::DELETE_TRIGGER_QUOTA_EXCEEDED]); |