From 6be9c0a97462272674cd6892aee00df752fc99b8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20Schie=C3=9Fle?= <schiessle@owncloud.com>
Date: Fri, 11 Jan 2013 11:12:32 +0100
Subject: [PATCH] some code clean-up
---
apps/files_versions/lib/versions.php | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/apps/files_versions/lib/versions.php b/apps/files_versions/lib/versions.php
index fc8adf646a..7b5f4755b2 100644
--- a/apps/files_versions/lib/versions.php
+++ b/apps/files_versions/lib/versions.php
@@ -89,7 +89,11 @@ class Storage {
$versionsSize += $users_view->filesize('files'.$filename);
// expire old revisions if necessary
- self::expire($filename, $versionsSize);
+ $newSize = self::expire($filename, $versionsSize);
+
+ if ( $newSize != $versionsSize ) {
+ \OCP\Config::setAppValue('files_versions', 'size', $versionsSize);
+ }
}
}
@@ -329,7 +333,7 @@ class Storage {
/**
* @brief Erase a file's versions which exceed the set quota
*/
- public static function expire($filename, $versionsSize = null) {
+ private static function expire($filename, $versionsSize = null) {
if(\OCP\Config::getSystemValue('files_versions', Storage::DEFAULTENABLED)=='true') {
list($uid, $filename) = self::getUidAndFilename($filename);
$versions_fileview = new \OC_FilesystemView('/'.$uid.'/files_versions');
@@ -418,7 +422,9 @@ class Storage {
if ($i = $numOfVersions-2) break; // keep at least the last version
}
- \OCP\Config::setAppValue('files_versions', 'size', $versionsSize);
+ return $versionsSize;
}
+
+ return false;
}
}
--
GitLab